* [PATCH] powerpc/dts: Update the core cluster PLL node(s) @ 2015-04-14 9:55 Igal.Liberman 2015-04-14 20:21 ` Scott Wood 0 siblings, 1 reply; 6+ messages in thread From: Igal.Liberman @ 2015-04-14 9:55 UTC (permalink / raw) To: devicetree, linuxppc-dev; +Cc: scottwood, Igal Liberman From: Igal Liberman <Igal.Liberman@freescale.com> This patch replaces the following: https://patchwork.ozlabs.org/patch/427664/ This patch is described by the following binding document update: https://patchwork.ozlabs.org/patch/461150/ Signed-off-by: Igal Liberman <Igal.Liberman@freescale.com> --- arch/powerpc/boot/dts/fsl/qoriq-clockgen2.dtsi | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/arch/powerpc/boot/dts/fsl/qoriq-clockgen2.dtsi b/arch/powerpc/boot/dts/fsl/qoriq-clockgen2.dtsi index 48e0b6e..7e1f074 100644 --- a/arch/powerpc/boot/dts/fsl/qoriq-clockgen2.dtsi +++ b/arch/powerpc/boot/dts/fsl/qoriq-clockgen2.dtsi @@ -49,14 +49,16 @@ global-utilities@e1000 { reg = <0x800 0x4>; compatible = "fsl,qoriq-core-pll-2.0"; clocks = <&sysclk>; - clock-output-names = "pll0", "pll0-div2", "pll0-div4"; + clock-output-names = "pll0", "pll0-div2", "pll0-div3", + "pll0-div4"; }; pll1: pll1@820 { #clock-cells = <1>; reg = <0x820 0x4>; compatible = "fsl,qoriq-core-pll-2.0"; clocks = <&sysclk>; - clock-output-names = "pll1", "pll1-div2", "pll1-div4"; + clock-output-names = "pll1", "pll1-div2", "pll1-div3", + "pll1-div4"; }; platform_pll: platform-pll@c00 { #clock-cells = <1>; -- 1.7.9.5 _______________________________________________ Linuxppc-dev mailing list Linuxppc-dev@lists.ozlabs.org https://lists.ozlabs.org/listinfo/linuxppc-dev ^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: [PATCH] powerpc/dts: Update the core cluster PLL node(s) 2015-04-14 9:55 [PATCH] powerpc/dts: Update the core cluster PLL node(s) Igal.Liberman @ 2015-04-14 20:21 ` Scott Wood 2015-04-14 20:22 ` Scott Wood 0 siblings, 1 reply; 6+ messages in thread From: Scott Wood @ 2015-04-14 20:21 UTC (permalink / raw) To: Igal.Liberman; +Cc: devicetree, linuxppc-dev On Tue, 2015-04-14 at 12:55 +0300, Igal.Liberman wrote: > From: Igal Liberman <Igal.Liberman@freescale.com> > > This patch replaces the following: > https://patchwork.ozlabs.org/patch/427664/ > > This patch is described by the following binding document update: > https://patchwork.ozlabs.org/patch/461150/ > > Signed-off-by: Igal Liberman <Igal.Liberman@freescale.com> > --- > arch/powerpc/boot/dts/fsl/qoriq-clockgen2.dtsi | 6 ++++-- > 1 file changed, 4 insertions(+), 2 deletions(-) > > diff --git a/arch/powerpc/boot/dts/fsl/qoriq-clockgen2.dtsi b/arch/powerpc/boot/dts/fsl/qoriq-clockgen2.dtsi > index 48e0b6e..7e1f074 100644 > --- a/arch/powerpc/boot/dts/fsl/qoriq-clockgen2.dtsi > +++ b/arch/powerpc/boot/dts/fsl/qoriq-clockgen2.dtsi > @@ -49,14 +49,16 @@ global-utilities@e1000 { > reg = <0x800 0x4>; > compatible = "fsl,qoriq-core-pll-2.0"; > clocks = <&sysclk>; > - clock-output-names = "pll0", "pll0-div2", "pll0-div4"; > + clock-output-names = "pll0", "pll0-div2", "pll0-div3", > + "pll0-div4"; > }; > pll1: pll1@820 { > #clock-cells = <1>; > reg = <0x820 0x4>; > compatible = "fsl,qoriq-core-pll-2.0"; > clocks = <&sysclk>; > - clock-output-names = "pll1", "pll1-div2", "pll1-div4"; > + clock-output-names = "pll1", "pll1-div2", "pll1-div3", > + "pll1-div4"; Wait, so if the driver implements the binding you submitted, you'll break compatibility with these older device trees... I think we need to just accept the ugly count-the-clock-names approach and document it. -Scott _______________________________________________ Linuxppc-dev mailing list Linuxppc-dev@lists.ozlabs.org https://lists.ozlabs.org/listinfo/linuxppc-dev ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] powerpc/dts: Update the core cluster PLL node(s) 2015-04-14 20:21 ` Scott Wood @ 2015-04-14 20:22 ` Scott Wood [not found] ` <1429042950.22867.690.camel-KZfg59tc24xl57MIdRCFDg@public.gmane.org> 0 siblings, 1 reply; 6+ messages in thread From: Scott Wood @ 2015-04-14 20:22 UTC (permalink / raw) To: Igal.Liberman; +Cc: devicetree, linuxppc-dev On Tue, 2015-04-14 at 15:21 -0500, Scott Wood wrote: > On Tue, 2015-04-14 at 12:55 +0300, Igal.Liberman wrote: > > From: Igal Liberman <Igal.Liberman@freescale.com> > > > > This patch replaces the following: > > https://patchwork.ozlabs.org/patch/427664/ > > > > This patch is described by the following binding document update: > > https://patchwork.ozlabs.org/patch/461150/ > > > > Signed-off-by: Igal Liberman <Igal.Liberman@freescale.com> > > --- > > arch/powerpc/boot/dts/fsl/qoriq-clockgen2.dtsi | 6 ++++-- > > 1 file changed, 4 insertions(+), 2 deletions(-) > > > > diff --git a/arch/powerpc/boot/dts/fsl/qoriq-clockgen2.dtsi b/arch/powerpc/boot/dts/fsl/qoriq-clockgen2.dtsi > > index 48e0b6e..7e1f074 100644 > > --- a/arch/powerpc/boot/dts/fsl/qoriq-clockgen2.dtsi > > +++ b/arch/powerpc/boot/dts/fsl/qoriq-clockgen2.dtsi > > @@ -49,14 +49,16 @@ global-utilities@e1000 { > > reg = <0x800 0x4>; > > compatible = "fsl,qoriq-core-pll-2.0"; > > clocks = <&sysclk>; > > - clock-output-names = "pll0", "pll0-div2", "pll0-div4"; > > + clock-output-names = "pll0", "pll0-div2", "pll0-div3", > > + "pll0-div4"; > > }; > > pll1: pll1@820 { > > #clock-cells = <1>; > > reg = <0x820 0x4>; > > compatible = "fsl,qoriq-core-pll-2.0"; > > clocks = <&sysclk>; > > - clock-output-names = "pll1", "pll1-div2", "pll1-div4"; > > + clock-output-names = "pll1", "pll1-div2", "pll1-div3", > > + "pll1-div4"; > > Wait, so if the driver implements the binding you submitted, you'll > break compatibility with these older device trees... > > I think we need to just accept the ugly count-the-clock-names approach > and document it. Is there any current 2.0 clock consumer that references pll-div4? -Scott _______________________________________________ Linuxppc-dev mailing list Linuxppc-dev@lists.ozlabs.org https://lists.ozlabs.org/listinfo/linuxppc-dev ^ permalink raw reply [flat|nested] 6+ messages in thread
[parent not found: <1429042950.22867.690.camel-KZfg59tc24xl57MIdRCFDg@public.gmane.org>]
* RE: [PATCH] powerpc/dts: Update the core cluster PLL node(s) [not found] ` <1429042950.22867.690.camel-KZfg59tc24xl57MIdRCFDg@public.gmane.org> @ 2015-04-15 11:07 ` Igal.Liberman-KZfg59tc24xl57MIdRCFDg [not found] ` <DM2PR03MB3831BEEDD2C17991818FFA9E6E50-ufbTtyGzTTRJonC5hhDUuuO6mTEJWrR4XA4E9RH9d+qIuWR1G4zioA@public.gmane.org> 0 siblings, 1 reply; 6+ messages in thread From: Igal.Liberman-KZfg59tc24xl57MIdRCFDg @ 2015-04-15 11:07 UTC (permalink / raw) To: Scott Wood Cc: devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linuxppc-dev-uLR06cmDAlY/bJ5BZ2RsiQ@public.gmane.org Regards, Igal Liberman. > -----Original Message----- > From: Wood Scott-B07421 > Sent: Tuesday, April 14, 2015 11:23 PM > To: Liberman Igal-B31950 > Cc: devicetree@vger.kernel.org; linuxppc-dev@lists.ozlabs.org > Subject: Re: [PATCH] powerpc/dts: Update the core cluster PLL node(s) > > On Tue, 2015-04-14 at 15:21 -0500, Scott Wood wrote: > > On Tue, 2015-04-14 at 12:55 +0300, Igal.Liberman wrote: > > > From: Igal Liberman <Igal.Liberman@freescale.com> > > > > > > This patch replaces the following: > > > https://patchwork.ozlabs.org/patch/427664/ > > > > > > This patch is described by the following binding document update: > > > https://patchwork.ozlabs.org/patch/461150/ > > > > > > Signed-off-by: Igal Liberman <Igal.Liberman@freescale.com> > > > --- > > > arch/powerpc/boot/dts/fsl/qoriq-clockgen2.dtsi | 6 ++++-- > > > 1 file changed, 4 insertions(+), 2 deletions(-) > > > > > > diff --git a/arch/powerpc/boot/dts/fsl/qoriq-clockgen2.dtsi > > > b/arch/powerpc/boot/dts/fsl/qoriq-clockgen2.dtsi > > > index 48e0b6e..7e1f074 100644 > > > --- a/arch/powerpc/boot/dts/fsl/qoriq-clockgen2.dtsi > > > +++ b/arch/powerpc/boot/dts/fsl/qoriq-clockgen2.dtsi > > > @@ -49,14 +49,16 @@ global-utilities@e1000 { > > > reg = <0x800 0x4>; > > > compatible = "fsl,qoriq-core-pll-2.0"; > > > clocks = <&sysclk>; > > > - clock-output-names = "pll0", "pll0-div2", "pll0-div4"; > > > + clock-output-names = "pll0", "pll0-div2", "pll0-div3", > > > + "pll0-div4"; > > > }; > > > pll1: pll1@820 { > > > #clock-cells = <1>; > > > reg = <0x820 0x4>; > > > compatible = "fsl,qoriq-core-pll-2.0"; > > > clocks = <&sysclk>; > > > - clock-output-names = "pll1", "pll1-div2", "pll1-div4"; > > > + clock-output-names = "pll1", "pll1-div2", "pll1-div3", > > > + "pll1-div4"; > > > > Wait, so if the driver implements the binding you submitted, you'll > > break compatibility with these older device trees... > > > > I think we need to just accept the ugly count-the-clock-names approach > > and document it. > > Is there any current 2.0 clock consumer that references pll-div4? > I looked at T4240 for example, there's a mux node which adds pll-div4 option: mux0: mux0@0 { #clock-cells = <0>; reg = <0x0 0x4>; compatible = "fsl,qoriq-core-mux-2.0"; clocks = <&pll0 0>, <&pll0 1>, <&pll0 2>, <&pll1 0>, <&pll1 1>, <&pll1 2>, <&pll2 0>, <&pll2 1>, <&pll2 2>; clock-names = "pll0", "pll0-div2", "pll0-div4", "pll1", "pll1-div2", "pll1-div4", "pll2", "pll2-div2", "pll2-div4"; clock-output-names = "cmux0"; }; After this change <&pll0 2> will represent "pll0-div3" and not "pll0-div4". > -Scott ^ permalink raw reply [flat|nested] 6+ messages in thread
[parent not found: <DM2PR03MB3831BEEDD2C17991818FFA9E6E50-ufbTtyGzTTRJonC5hhDUuuO6mTEJWrR4XA4E9RH9d+qIuWR1G4zioA@public.gmane.org>]
* Re: [PATCH] powerpc/dts: Update the core cluster PLL node(s) [not found] ` <DM2PR03MB3831BEEDD2C17991818FFA9E6E50-ufbTtyGzTTRJonC5hhDUuuO6mTEJWrR4XA4E9RH9d+qIuWR1G4zioA@public.gmane.org> @ 2015-04-15 17:14 ` Scott Wood 2015-04-16 6:13 ` Igal.Liberman 0 siblings, 1 reply; 6+ messages in thread From: Scott Wood @ 2015-04-15 17:14 UTC (permalink / raw) To: Liberman Igal-B31950 Cc: devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linuxppc-dev-uLR06cmDAlY/bJ5BZ2RsiQ@public.gmane.org On Wed, 2015-04-15 at 06:07 -0500, Liberman Igal-B31950 wrote: > > > Regards, > Igal Liberman. > > > -----Original Message----- > > From: Wood Scott-B07421 > > Sent: Tuesday, April 14, 2015 11:23 PM > > To: Liberman Igal-B31950 > > Cc: devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org; linuxppc-dev-uLR06cmDAlY/bJ5BZ2RsiQ@public.gmane.org > > Subject: Re: [PATCH] powerpc/dts: Update the core cluster PLL node(s) > > > > On Tue, 2015-04-14 at 15:21 -0500, Scott Wood wrote: > > > On Tue, 2015-04-14 at 12:55 +0300, Igal.Liberman wrote: > > > > From: Igal Liberman <Igal.Liberman-KZfg59tc24xl57MIdRCFDg@public.gmane.org> > > > > > > > > This patch replaces the following: > > > > https://patchwork.ozlabs.org/patch/427664/ > > > > > > > > This patch is described by the following binding document update: > > > > https://patchwork.ozlabs.org/patch/461150/ > > > > > > > > Signed-off-by: Igal Liberman <Igal.Liberman-KZfg59tc24xl57MIdRCFDg@public.gmane.org> > > > > --- > > > > arch/powerpc/boot/dts/fsl/qoriq-clockgen2.dtsi | 6 ++++-- > > > > 1 file changed, 4 insertions(+), 2 deletions(-) > > > > > > > > diff --git a/arch/powerpc/boot/dts/fsl/qoriq-clockgen2.dtsi > > > > b/arch/powerpc/boot/dts/fsl/qoriq-clockgen2.dtsi > > > > index 48e0b6e..7e1f074 100644 > > > > --- a/arch/powerpc/boot/dts/fsl/qoriq-clockgen2.dtsi > > > > +++ b/arch/powerpc/boot/dts/fsl/qoriq-clockgen2.dtsi > > > > @@ -49,14 +49,16 @@ global-utilities@e1000 { > > > > reg = <0x800 0x4>; > > > > compatible = "fsl,qoriq-core-pll-2.0"; > > > > clocks = <&sysclk>; > > > > - clock-output-names = "pll0", "pll0-div2", "pll0-div4"; > > > > + clock-output-names = "pll0", "pll0-div2", "pll0-div3", > > > > + "pll0-div4"; > > > > }; > > > > pll1: pll1@820 { > > > > #clock-cells = <1>; > > > > reg = <0x820 0x4>; > > > > compatible = "fsl,qoriq-core-pll-2.0"; > > > > clocks = <&sysclk>; > > > > - clock-output-names = "pll1", "pll1-div2", "pll1-div4"; > > > > + clock-output-names = "pll1", "pll1-div2", "pll1-div3", > > > > + "pll1-div4"; > > > > > > Wait, so if the driver implements the binding you submitted, you'll > > > break compatibility with these older device trees... > > > > > > I think we need to just accept the ugly count-the-clock-names approach > > > and document it. > > > > Is there any current 2.0 clock consumer that references pll-div4? > > > > I looked at T4240 for example, there's a mux node which adds pll-div4 option: > mux0: mux0@0 { > #clock-cells = <0>; > reg = <0x0 0x4>; > compatible = "fsl,qoriq-core-mux-2.0"; > clocks = <&pll0 0>, <&pll0 1>, <&pll0 2>, > <&pll1 0>, <&pll1 1>, <&pll1 2>, > <&pll2 0>, <&pll2 1>, <&pll2 2>; > clock-names = "pll0", "pll0-div2", "pll0-div4", > "pll1", "pll1-div2", "pll1-div4", > "pll2", "pll2-div2", "pll2-div4"; > clock-output-names = "cmux0"; > }; > > After this change <&pll0 2> will represent "pll0-div3" and not "pll0-div4". So this needs to be updated to match -- and it confirms that existing device trees will be broken if you base the interpretation on compatible rather than the number of clock-output-names. -Scott -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html ^ permalink raw reply [flat|nested] 6+ messages in thread
* RE: [PATCH] powerpc/dts: Update the core cluster PLL node(s) 2015-04-15 17:14 ` Scott Wood @ 2015-04-16 6:13 ` Igal.Liberman 0 siblings, 0 replies; 6+ messages in thread From: Igal.Liberman @ 2015-04-16 6:13 UTC (permalink / raw) To: Scott Wood; +Cc: devicetree@vger.kernel.org, linuxppc-dev@lists.ozlabs.org Regards, Igal Liberman. > -----Original Message----- > From: Wood Scott-B07421 > Sent: Wednesday, April 15, 2015 8:15 PM > To: Liberman Igal-B31950 > Cc: devicetree@vger.kernel.org; linuxppc-dev@lists.ozlabs.org > Subject: Re: [PATCH] powerpc/dts: Update the core cluster PLL node(s) > > On Wed, 2015-04-15 at 06:07 -0500, Liberman Igal-B31950 wrote: > > > > > > Regards, > > Igal Liberman. > > > > > -----Original Message----- > > > From: Wood Scott-B07421 > > > Sent: Tuesday, April 14, 2015 11:23 PM > > > To: Liberman Igal-B31950 > > > Cc: devicetree@vger.kernel.org; linuxppc-dev@lists.ozlabs.org > > > Subject: Re: [PATCH] powerpc/dts: Update the core cluster PLL > > > node(s) > > > > > > On Tue, 2015-04-14 at 15:21 -0500, Scott Wood wrote: > > > > On Tue, 2015-04-14 at 12:55 +0300, Igal.Liberman wrote: > > > > > From: Igal Liberman <Igal.Liberman@freescale.com> > > > > > > > > > > This patch replaces the following: > > > > > https://patchwork.ozlabs.org/patch/427664/ > > > > > > > > > > This patch is described by the following binding document update: > > > > > https://patchwork.ozlabs.org/patch/461150/ > > > > > > > > > > Signed-off-by: Igal Liberman <Igal.Liberman@freescale.com> > > > > > --- > > > > > arch/powerpc/boot/dts/fsl/qoriq-clockgen2.dtsi | 6 ++++-- > > > > > 1 file changed, 4 insertions(+), 2 deletions(-) > > > > > > > > > > diff --git a/arch/powerpc/boot/dts/fsl/qoriq-clockgen2.dtsi > > > > > b/arch/powerpc/boot/dts/fsl/qoriq-clockgen2.dtsi > > > > > index 48e0b6e..7e1f074 100644 > > > > > --- a/arch/powerpc/boot/dts/fsl/qoriq-clockgen2.dtsi > > > > > +++ b/arch/powerpc/boot/dts/fsl/qoriq-clockgen2.dtsi > > > > > @@ -49,14 +49,16 @@ global-utilities@e1000 { > > > > > reg = <0x800 0x4>; > > > > > compatible = "fsl,qoriq-core-pll-2.0"; > > > > > clocks = <&sysclk>; > > > > > - clock-output-names = "pll0", "pll0-div2", "pll0-div4"; > > > > > + clock-output-names = "pll0", "pll0-div2", "pll0-div3", > > > > > + "pll0-div4"; > > > > > }; > > > > > pll1: pll1@820 { > > > > > #clock-cells = <1>; > > > > > reg = <0x820 0x4>; > > > > > compatible = "fsl,qoriq-core-pll-2.0"; > > > > > clocks = <&sysclk>; > > > > > - clock-output-names = "pll1", "pll1-div2", "pll1-div4"; > > > > > + clock-output-names = "pll1", "pll1-div2", "pll1-div3", > > > > > + "pll1-div4"; > > > > > > > > Wait, so if the driver implements the binding you submitted, > > > > you'll break compatibility with these older device trees... > > > > > > > > I think we need to just accept the ugly count-the-clock-names > > > > approach and document it. > > > > > > Is there any current 2.0 clock consumer that references pll-div4? > > > > > > > I looked at T4240 for example, there's a mux node which adds pll-div4 > option: > > mux0: mux0@0 { > > #clock-cells = <0>; > > reg = <0x0 0x4>; > > compatible = "fsl,qoriq-core-mux-2.0"; > > clocks = <&pll0 0>, <&pll0 1>, <&pll0 2>, > > <&pll1 0>, <&pll1 1>, <&pll1 2>, > > <&pll2 0>, <&pll2 1>, <&pll2 2>; > > clock-names = "pll0", "pll0-div2", "pll0-div4", > > "pll1", "pll1-div2", "pll1-div4", > > "pll2", "pll2-div2", "pll2-div4"; > > clock-output-names = "cmux0"; > > }; > > > > After this change <&pll0 2> will represent "pll0-div3" and not "pll0-div4". > > So this needs to be updated to match -- and it confirms that existing device > trees will be broken if you base the interpretation on compatible rather than > the number of clock-output-names. > Yes. I'll submit a patch and mention this dependency. > -Scott > Igal. _______________________________________________ Linuxppc-dev mailing list Linuxppc-dev@lists.ozlabs.org https://lists.ozlabs.org/listinfo/linuxppc-dev ^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2015-04-16 6:13 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2015-04-14 9:55 [PATCH] powerpc/dts: Update the core cluster PLL node(s) Igal.Liberman
2015-04-14 20:21 ` Scott Wood
2015-04-14 20:22 ` Scott Wood
[not found] ` <1429042950.22867.690.camel-KZfg59tc24xl57MIdRCFDg@public.gmane.org>
2015-04-15 11:07 ` Igal.Liberman-KZfg59tc24xl57MIdRCFDg
[not found] ` <DM2PR03MB3831BEEDD2C17991818FFA9E6E50-ufbTtyGzTTRJonC5hhDUuuO6mTEJWrR4XA4E9RH9d+qIuWR1G4zioA@public.gmane.org>
2015-04-15 17:14 ` Scott Wood
2015-04-16 6:13 ` Igal.Liberman
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).