From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1750990AbeFAGsD (ORCPT ); Fri, 1 Jun 2018 02:48:03 -0400 Received: from lelnx194.ext.ti.com ([198.47.27.80]:51587 "EHLO lelnx194.ext.ti.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750750AbeFAGsB (ORCPT ); Fri, 1 Jun 2018 02:48:01 -0400 Subject: Re: [PATCH v2 3/6] clk: ti: dra7: Add clkctrl clock data for the mcan clocks To: Tero Kristo , Rob Herring CC: , , , , , , , References: <20180530141133.3711-1-faiz_abbas@ti.com> <20180530141133.3711-4-faiz_abbas@ti.com> <20180531040350.GA25132@rob-hp-laptop> <678ef467-a235-91db-8bae-f249ad98eac8@ti.com> From: Faiz Abbas Message-ID: Date: Fri, 1 Jun 2018 12:18:37 +0530 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.7.0 MIME-Version: 1.0 In-Reply-To: <678ef467-a235-91db-8bae-f249ad98eac8@ti.com> Content-Type: text/plain; charset="utf-8" Content-Language: en-US Content-Transfer-Encoding: 8bit X-EXCLAIMER-MD-CONFIG: e1e8a2fd-e40a-4ac6-ac9b-f7e9cc9ee180 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi, On Thursday 31 May 2018 06:59 PM, Tero Kristo wrote: > On 31/05/18 13:14, Faiz Abbas wrote: >> Hi, >> >> On Thursday 31 May 2018 09:33 AM, Rob Herring wrote: >>> On Wed, May 30, 2018 at 07:41:30PM +0530, Faiz Abbas wrote: >>>> Add clkctrl data for the m_can clocks and register it within the >> ... >>>>   diff --git a/include/dt-bindings/clock/dra7.h >>>> b/include/dt-bindings/clock/dra7.h >>>> index 5e1061b15aed..d7549c57cac3 100644 >>>> --- a/include/dt-bindings/clock/dra7.h >>>> +++ b/include/dt-bindings/clock/dra7.h >>>> @@ -168,5 +168,6 @@ >>>>   #define DRA7_COUNTER_32K_CLKCTRL    DRA7_CLKCTRL_INDEX(0x50) >>>>   #define DRA7_UART10_CLKCTRL    DRA7_CLKCTRL_INDEX(0x80) >>>>   #define DRA7_DCAN1_CLKCTRL    DRA7_CLKCTRL_INDEX(0x88) >>>> +#define DRA7_ADC_CLKCTRL    DRA7_CLKCTRL_INDEX(0xa0) >>> >>> ADC and mcan are the same thing? >>> >> >> The register to control MCAN clocks is called ADC_CLKCTRL, Yes. > > Is there any reason for this or is that just a documentation bug? > Looks like they meant to have an ADC in dra74 or dra72 but decided against it and then many years later used the same registers for MCAN instead. You can see ADC_CLKCTRL exists in dra72 TRM but is explicitly disabled. http://www.ti.com/lit/ug/spruic2b/spruic2b.pdf pg:1524 Thanks, Faiz