Message ID | 1487137656-4006-1-git-send-email-yuantian.tang@nxp.com (mailing list archive) |
---|---|
State | New, archived |
Headers | show |
PING! Regards, Yuantian > -----Original Message----- > From: yuantian.tang@nxp.com [mailto:yuantian.tang@nxp.com] > Sent: Wednesday, February 15, 2017 1:48 PM > To: mturquette@baylibre.com > Cc: sboyd@codeaurora.org; robh+dt@kernel.org; mark.rutland@arm.com; > linux-clk@vger.kernel.org; devicetree@vger.kernel.org; linux- > kernel@vger.kernel.org; linux-arm-kernel@lists.infradead.org; Y.T. Tang > <yuantian.tang@nxp.com>; Scott Wood <oss@buserror.net>; Y.T. Tang > <yuantian.tang@nxp.com> > Subject: [PATCH 1/2] dt-bindings: qoriq-clock: Add coreclk > > From: Tang Yuantian <Yuantian.Tang@nxp.com> > > ls1012a has separate input root clocks for core PLLs versus the platform PLL, > with the latter described as sysclk in the hw docs. > Update the qoriq-clock binding to allow a second input clock, named > "coreclk". If present, this clock will be used for the core PLLs. > > Signed-off-by: Scott Wood <oss@buserror.net> > Signed-off-by: Tang Yuantian <yuantian.tang@nxp.com> > --- > Documentation/devicetree/bindings/clock/qoriq-clock.txt | 6 ++++++ > 1 file changed, 6 insertions(+) > > diff --git a/Documentation/devicetree/bindings/clock/qoriq-clock.txt > b/Documentation/devicetree/bindings/clock/qoriq-clock.txt > index df9cb5a..97a9666 100644 > --- a/Documentation/devicetree/bindings/clock/qoriq-clock.txt > +++ b/Documentation/devicetree/bindings/clock/qoriq-clock.txt > @@ -55,6 +55,11 @@ Optional properties: > - clocks: If clock-frequency is not specified, sysclk may be provided > as an input clock. Either clock-frequency or clocks must be > provided. > + A second input clock, called "coreclk", may be provided if > + core PLLs are based on a different input clock from the > + platform PLL. > +- clock-names: Required if a coreclk is present. Valid names are > + "sysclk" and "coreclk". > > 2. Clock Provider > > @@ -71,6 +76,7 @@ second cell is the clock index for the specified type. > 2 hwaccel index (n in CLKCGnHWACSR) > 3 fman 0 for fm1, 1 for fm2 > 4 platform pll 0=pll, 1=pll/2, 2=pll/3, 3=pll/4 > + 5 coreclk must be 0 > > 3. Example > > -- > 2.1.0.27.g96db324
On Wed, Feb 15, 2017 at 01:47:35PM +0800, yuantian.tang@nxp.com wrote: > From: Tang Yuantian <Yuantian.Tang@nxp.com> > > ls1012a has separate input root clocks for core PLLs versus the platform > PLL, with the latter described as sysclk in the hw docs. > Update the qoriq-clock binding to allow a second input clock, named > "coreclk". If present, this clock will be used for the core PLLs. > > Signed-off-by: Scott Wood <oss@buserror.net> > Signed-off-by: Tang Yuantian <yuantian.tang@nxp.com> > --- > Documentation/devicetree/bindings/clock/qoriq-clock.txt | 6 ++++++ > 1 file changed, 6 insertions(+) The change looks fine, but sounds like Scott should remain the author (or agree he shouldn't be). > > diff --git a/Documentation/devicetree/bindings/clock/qoriq-clock.txt b/Documentation/devicetree/bindings/clock/qoriq-clock.txt > index df9cb5a..97a9666 100644 > --- a/Documentation/devicetree/bindings/clock/qoriq-clock.txt > +++ b/Documentation/devicetree/bindings/clock/qoriq-clock.txt > @@ -55,6 +55,11 @@ Optional properties: > - clocks: If clock-frequency is not specified, sysclk may be provided > as an input clock. Either clock-frequency or clocks must be > provided. > + A second input clock, called "coreclk", may be provided if > + core PLLs are based on a different input clock from the > + platform PLL. > +- clock-names: Required if a coreclk is present. Valid names are > + "sysclk" and "coreclk". > > 2. Clock Provider > > @@ -71,6 +76,7 @@ second cell is the clock index for the specified type. > 2 hwaccel index (n in CLKCGnHWACSR) > 3 fman 0 for fm1, 1 for fm2 > 4 platform pll 0=pll, 1=pll/2, 2=pll/3, 3=pll/4 > + 5 coreclk must be 0 > > 3. Example > > -- > 2.1.0.27.g96db324 >
Hi Rob, > -----Original Message----- > From: Rob Herring [mailto:robh@kernel.org] > Sent: Tuesday, February 28, 2017 1:19 AM > To: Y.T. Tang <yuantian.tang@nxp.com> > Cc: mturquette@baylibre.com; sboyd@codeaurora.org; > mark.rutland@arm.com; linux-clk@vger.kernel.org; > devicetree@vger.kernel.org; linux-kernel@vger.kernel.org; linux-arm- > kernel@lists.infradead.org; Scott Wood <oss@buserror.net> > Subject: Re: [PATCH 1/2] dt-bindings: qoriq-clock: Add coreclk > > On Wed, Feb 15, 2017 at 01:47:35PM +0800, yuantian.tang@nxp.com wrote: > > From: Tang Yuantian <Yuantian.Tang@nxp.com> > > > > ls1012a has separate input root clocks for core PLLs versus the > > platform PLL, with the latter described as sysclk in the hw docs. > > Update the qoriq-clock binding to allow a second input clock, named > > "coreclk". If present, this clock will be used for the core PLLs. > > > > Signed-off-by: Scott Wood <oss@buserror.net> > > Signed-off-by: Tang Yuantian <yuantian.tang@nxp.com> > > --- > > Documentation/devicetree/bindings/clock/qoriq-clock.txt | 6 ++++++ > > 1 file changed, 6 insertions(+) > > The change looks fine, but sounds like Scott should remain the author (or > agree he shouldn't be). > Sure, please make Scott the author and apply this patch set. Regards, Yuantian > > > > diff --git a/Documentation/devicetree/bindings/clock/qoriq-clock.txt > > b/Documentation/devicetree/bindings/clock/qoriq-clock.txt > > index df9cb5a..97a9666 100644 > > --- a/Documentation/devicetree/bindings/clock/qoriq-clock.txt > > +++ b/Documentation/devicetree/bindings/clock/qoriq-clock.txt > > @@ -55,6 +55,11 @@ Optional properties: > > - clocks: If clock-frequency is not specified, sysclk may be provided > > as an input clock. Either clock-frequency or clocks must be > > provided. > > + A second input clock, called "coreclk", may be provided if > > + core PLLs are based on a different input clock from the > > + platform PLL. > > +- clock-names: Required if a coreclk is present. Valid names are > > + "sysclk" and "coreclk". > > > > 2. Clock Provider > > > > @@ -71,6 +76,7 @@ second cell is the clock index for the specified type. > > 2 hwaccel index (n in CLKCGnHWACSR) > > 3 fman 0 for fm1, 1 for fm2 > > 4 platform pll 0=pll, 1=pll/2, 2=pll/3, 3=pll/4 > > + 5 coreclk must be 0 > > > > 3. Example > > > > -- > > 2.1.0.27.g96db324 > >
On Tue, Feb 28, 2017 at 7:45 PM, Y.T. Tang <yuantian.tang@nxp.com> wrote: > Hi Rob, > >> -----Original Message----- >> From: Rob Herring [mailto:robh@kernel.org] >> Sent: Tuesday, February 28, 2017 1:19 AM >> To: Y.T. Tang <yuantian.tang@nxp.com> >> Cc: mturquette@baylibre.com; sboyd@codeaurora.org; >> mark.rutland@arm.com; linux-clk@vger.kernel.org; >> devicetree@vger.kernel.org; linux-kernel@vger.kernel.org; linux-arm- >> kernel@lists.infradead.org; Scott Wood <oss@buserror.net> >> Subject: Re: [PATCH 1/2] dt-bindings: qoriq-clock: Add coreclk >> >> On Wed, Feb 15, 2017 at 01:47:35PM +0800, yuantian.tang@nxp.com wrote: >> > From: Tang Yuantian <Yuantian.Tang@nxp.com> >> > >> > ls1012a has separate input root clocks for core PLLs versus the >> > platform PLL, with the latter described as sysclk in the hw docs. >> > Update the qoriq-clock binding to allow a second input clock, named >> > "coreclk". If present, this clock will be used for the core PLLs. >> > >> > Signed-off-by: Scott Wood <oss@buserror.net> >> > Signed-off-by: Tang Yuantian <yuantian.tang@nxp.com> >> > --- >> > Documentation/devicetree/bindings/clock/qoriq-clock.txt | 6 ++++++ >> > 1 file changed, 6 insertions(+) >> >> The change looks fine, but sounds like Scott should remain the author (or >> agree he shouldn't be). >> > Sure, please make Scott the author and apply this patch set. Fixing the author is your job. Plus you sent this TO Mike, so I'm assuming you only want my ack and Mike will apply. Rob
Hi Michael and Stephen, This patch set was acked by Rob Herring. Do you have any comments on them? BTW: Scott should stay in author, do I need to resend them with author changed or you can change it when applying? Regards, Yuantian > -----Original Message----- > From: Rob Herring [mailto:robh@kernel.org] > Sent: Tuesday, February 28, 2017 1:19 AM > To: Y.T. Tang > Cc: mturquette@baylibre.com; sboyd@codeaurora.org; > mark.rutland@arm.com; linux-clk@vger.kernel.org; > devicetree@vger.kernel.org; linux-kernel@vger.kernel.org; linux-arm- > kernel@lists.infradead.org; Scott Wood > Subject: Re: [PATCH 1/2] dt-bindings: qoriq-clock: Add coreclk > > On Wed, Feb 15, 2017 at 01:47:35PM +0800, yuantian.tang@nxp.com wrote: > > From: Tang Yuantian <Yuantian.Tang@nxp.com> > > > > ls1012a has separate input root clocks for core PLLs versus the > > platform PLL, with the latter described as sysclk in the hw docs. > > Update the qoriq-clock binding to allow a second input clock, named > > "coreclk". If present, this clock will be used for the core PLLs. > > > > Signed-off-by: Scott Wood <oss@buserror.net> > > Signed-off-by: Tang Yuantian <yuantian.tang@nxp.com> > > --- > > Documentation/devicetree/bindings/clock/qoriq-clock.txt | 6 ++++++ > > 1 file changed, 6 insertions(+) > > The change looks fine, but sounds like Scott should remain the author (or > agree he shouldn't be). > > > > > diff --git a/Documentation/devicetree/bindings/clock/qoriq-clock.txt > > b/Documentation/devicetree/bindings/clock/qoriq-clock.txt > > index df9cb5a..97a9666 100644 > > --- a/Documentation/devicetree/bindings/clock/qoriq-clock.txt > > +++ b/Documentation/devicetree/bindings/clock/qoriq-clock.txt > > @@ -55,6 +55,11 @@ Optional properties: > > - clocks: If clock-frequency is not specified, sysclk may be provided > > as an input clock. Either clock-frequency or clocks must be > > provided. > > + A second input clock, called "coreclk", may be provided if > > + core PLLs are based on a different input clock from the > > + platform PLL. > > +- clock-names: Required if a coreclk is present. Valid names are > > + "sysclk" and "coreclk". > > > > 2. Clock Provider > > > > @@ -71,6 +76,7 @@ second cell is the clock index for the specified type. > > 2 hwaccel index (n in CLKCGnHWACSR) > > 3 fman 0 for fm1, 1 for fm2 > > 4 platform pll 0=pll, 1=pll/2, 2=pll/3, 3=pll/4 > > + 5 coreclk must be 0 > > > > 3. Example > > > > -- > > 2.1.0.27.g96db324 > >
On 03/09, Y.T. Tang wrote: > Hi Michael and Stephen, > > This patch set was acked by Rob Herring. Do you have any comments on them? > > BTW: Scott should stay in author, do I need to resend them with author changed or you can change it when applying? > Please resend these two patches.
Hi, > -----Original Message----- > From: sboyd@codeaurora.org [mailto:sboyd@codeaurora.org] > Sent: Saturday, July 22, 2017 6:03 AM > To: Andy Tang <andy.tang@nxp.com> > Cc: mturquette@baylibre.com; mark.rutland@arm.com; linux- > clk@vger.kernel.org; devicetree@vger.kernel.org; linux- > kernel@vger.kernel.org; linux-arm-kernel@lists.infradead.org; Scott Wood > <oss@buserror.net>; Rob Herring <robh@kernel.org> > Subject: Re: [PATCH 1/2] dt-bindings: qoriq-clock: Add coreclk > > On 03/09, Y.T. Tang wrote: > > Hi Michael and Stephen, > > > > This patch set was acked by Rob Herring. Do you have any comments on > them? > > > > BTW: Scott should stay in author, do I need to resend them with author > changed or you can change it when applying? > > > > Please resend these two patches. Those two patches have been merged several months ago. No need to resend. Thanks. Regards, Andy > > -- > Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, a > Linux Foundation Collaborative Project
diff --git a/Documentation/devicetree/bindings/clock/qoriq-clock.txt b/Documentation/devicetree/bindings/clock/qoriq-clock.txt index df9cb5a..97a9666 100644 --- a/Documentation/devicetree/bindings/clock/qoriq-clock.txt +++ b/Documentation/devicetree/bindings/clock/qoriq-clock.txt @@ -55,6 +55,11 @@ Optional properties: - clocks: If clock-frequency is not specified, sysclk may be provided as an input clock. Either clock-frequency or clocks must be provided. + A second input clock, called "coreclk", may be provided if + core PLLs are based on a different input clock from the + platform PLL. +- clock-names: Required if a coreclk is present. Valid names are + "sysclk" and "coreclk". 2. Clock Provider @@ -71,6 +76,7 @@ second cell is the clock index for the specified type. 2 hwaccel index (n in CLKCGnHWACSR) 3 fman 0 for fm1, 1 for fm2 4 platform pll 0=pll, 1=pll/2, 2=pll/3, 3=pll/4 + 5 coreclk must be 0 3. Example