All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] powerpc/dts: Update the core cluster PLL node(s)
@ 2015-04-14  9:55 ` Igal.Liberman
  0 siblings, 0 replies; 12+ 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] 12+ messages in thread
* [PATCH] powerpc/dts: Update the core cluster PLL node(s)
@ 2015-04-14  9:55 ` Igal.Liberman
  0 siblings, 0 replies; 12+ 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

^ permalink raw reply related	[flat|nested] 12+ messages in thread
* Re: [PATCH] powerpc/dts: Update the core cluster PLL node(s)
  2015-04-14  9:55 ` Igal.Liberman
@ 2015-04-14 20:21   ` Scott Wood
  -1 siblings, 0 replies; 12+ 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] 12+ messages in thread
* Re: [PATCH] powerpc/dts: Update the core cluster PLL node(s)
@ 2015-04-14 20:21   ` Scott Wood
  0 siblings, 0 replies; 12+ 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

^ permalink raw reply	[flat|nested] 12+ 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
  -1 siblings, 0 replies; 12+ 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] 12+ messages in thread
* Re: [PATCH] powerpc/dts: Update the core cluster PLL node(s)
@ 2015-04-14 20:22     ` Scott Wood
  0 siblings, 0 replies; 12+ 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

^ permalink raw reply	[flat|nested] 12+ messages in thread
* RE: [PATCH] powerpc/dts: Update the core cluster PLL node(s)
  2015-04-14 20:22     ` Scott Wood
@ 2015-04-15 11:07         ` Igal.Liberman
  -1 siblings, 0 replies; 12+ messages in thread
From: Igal.Liberman-KZfg59tc24xl57MIdRCFDg @ 2015-04-15 11:07 UTC (permalink / raw)
  To: Scott Wood
  Cc: devicetree-u79uwXL29TY76Z2rM5mHXA, linuxppc-dev-uLR06cmDAlY/bJ5BZ2RsiQ



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] 12+ messages in thread
* RE: [PATCH] powerpc/dts: Update the core cluster PLL node(s)
@ 2015-04-15 11:07         ` Igal.Liberman
  0 siblings, 0 replies; 12+ messages in thread
From: Igal.Liberman @ 2015-04-15 11:07 UTC (permalink / raw)
  To: Scott Wood; +Cc: devicetree, linuxppc-dev

DQoNClJlZ2FyZHMsDQpJZ2FsIExpYmVybWFuLg0KDQo+IC0tLS0tT3JpZ2luYWwgTWVzc2FnZS0t
LS0tDQo+IEZyb206IFdvb2QgU2NvdHQtQjA3NDIxDQo+IFNlbnQ6IFR1ZXNkYXksIEFwcmlsIDE0
LCAyMDE1IDExOjIzIFBNDQo+IFRvOiBMaWJlcm1hbiBJZ2FsLUIzMTk1MA0KPiBDYzogZGV2aWNl
dHJlZUB2Z2VyLmtlcm5lbC5vcmc7IGxpbnV4cHBjLWRldkBsaXN0cy5vemxhYnMub3JnDQo+IFN1
YmplY3Q6IFJlOiBbUEFUQ0hdIHBvd2VycGMvZHRzOiBVcGRhdGUgdGhlIGNvcmUgY2x1c3RlciBQ
TEwgbm9kZShzKQ0KPiANCj4gT24gVHVlLCAyMDE1LTA0LTE0IGF0IDE1OjIxIC0wNTAwLCBTY290
dCBXb29kIHdyb3RlOg0KPiA+IE9uIFR1ZSwgMjAxNS0wNC0xNCBhdCAxMjo1NSArMDMwMCwgSWdh
bC5MaWJlcm1hbiB3cm90ZToNCj4gPiA+IEZyb206IElnYWwgTGliZXJtYW4gPElnYWwuTGliZXJt
YW5AZnJlZXNjYWxlLmNvbT4NCj4gPiA+DQo+ID4gPiBUaGlzIHBhdGNoIHJlcGxhY2VzIHRoZSBm
b2xsb3dpbmc6DQo+ID4gPiAJaHR0cHM6Ly9wYXRjaHdvcmsub3psYWJzLm9yZy9wYXRjaC80Mjc2
NjQvDQo+ID4gPg0KPiA+ID4gVGhpcyBwYXRjaCBpcyBkZXNjcmliZWQgYnkgdGhlIGZvbGxvd2lu
ZyBiaW5kaW5nIGRvY3VtZW50IHVwZGF0ZToNCj4gPiA+IAlodHRwczovL3BhdGNod29yay5vemxh
YnMub3JnL3BhdGNoLzQ2MTE1MC8NCj4gPiA+DQo+ID4gPiBTaWduZWQtb2ZmLWJ5OiBJZ2FsIExp
YmVybWFuIDxJZ2FsLkxpYmVybWFuQGZyZWVzY2FsZS5jb20+DQo+ID4gPiAtLS0NCj4gPiA+ICBh
cmNoL3Bvd2VycGMvYm9vdC9kdHMvZnNsL3FvcmlxLWNsb2NrZ2VuMi5kdHNpIHwgICAgNiArKysr
LS0NCj4gPiA+ICAxIGZpbGUgY2hhbmdlZCwgNCBpbnNlcnRpb25zKCspLCAyIGRlbGV0aW9ucygt
KQ0KPiA+ID4NCj4gPiA+IGRpZmYgLS1naXQgYS9hcmNoL3Bvd2VycGMvYm9vdC9kdHMvZnNsL3Fv
cmlxLWNsb2NrZ2VuMi5kdHNpDQo+ID4gPiBiL2FyY2gvcG93ZXJwYy9ib290L2R0cy9mc2wvcW9y
aXEtY2xvY2tnZW4yLmR0c2kNCj4gPiA+IGluZGV4IDQ4ZTBiNmUuLjdlMWYwNzQgMTAwNjQ0DQo+
ID4gPiAtLS0gYS9hcmNoL3Bvd2VycGMvYm9vdC9kdHMvZnNsL3FvcmlxLWNsb2NrZ2VuMi5kdHNp
DQo+ID4gPiArKysgYi9hcmNoL3Bvd2VycGMvYm9vdC9kdHMvZnNsL3FvcmlxLWNsb2NrZ2VuMi5k
dHNpDQo+ID4gPiBAQCAtNDksMTQgKzQ5LDE2IEBAIGdsb2JhbC11dGlsaXRpZXNAZTEwMDAgew0K
PiA+ID4gIAkJcmVnID0gPDB4ODAwIDB4ND47DQo+ID4gPiAgCQljb21wYXRpYmxlID0gImZzbCxx
b3JpcS1jb3JlLXBsbC0yLjAiOw0KPiA+ID4gIAkJY2xvY2tzID0gPCZzeXNjbGs+Ow0KPiA+ID4g
LQkJY2xvY2stb3V0cHV0LW5hbWVzID0gInBsbDAiLCAicGxsMC1kaXYyIiwgInBsbDAtZGl2NCI7
DQo+ID4gPiArCQljbG9jay1vdXRwdXQtbmFtZXMgPSAicGxsMCIsICJwbGwwLWRpdjIiLCAicGxs
MC1kaXYzIiwNCj4gPiA+ICsJCQkJICAgICAgInBsbDAtZGl2NCI7DQo+ID4gPiAgCX07DQo+ID4g
PiAgCXBsbDE6IHBsbDFAODIwIHsNCj4gPiA+ICAJCSNjbG9jay1jZWxscyA9IDwxPjsNCj4gPiA+
ICAJCXJlZyA9IDwweDgyMCAweDQ+Ow0KPiA+ID4gIAkJY29tcGF0aWJsZSA9ICJmc2wscW9yaXEt
Y29yZS1wbGwtMi4wIjsNCj4gPiA+ICAJCWNsb2NrcyA9IDwmc3lzY2xrPjsNCj4gPiA+IC0JCWNs
b2NrLW91dHB1dC1uYW1lcyA9ICJwbGwxIiwgInBsbDEtZGl2MiIsICJwbGwxLWRpdjQiOw0KPiA+
ID4gKwkJY2xvY2stb3V0cHV0LW5hbWVzID0gInBsbDEiLCAicGxsMS1kaXYyIiwgInBsbDEtZGl2
MyIsDQo+ID4gPiArCQkJCSAgICAgICJwbGwxLWRpdjQiOw0KPiA+DQo+ID4gV2FpdCwgc28gaWYg
dGhlIGRyaXZlciBpbXBsZW1lbnRzIHRoZSBiaW5kaW5nIHlvdSBzdWJtaXR0ZWQsIHlvdSdsbA0K
PiA+IGJyZWFrIGNvbXBhdGliaWxpdHkgd2l0aCB0aGVzZSBvbGRlciBkZXZpY2UgdHJlZXMuLi4N
Cj4gPg0KPiA+IEkgdGhpbmsgd2UgbmVlZCB0byBqdXN0IGFjY2VwdCB0aGUgdWdseSBjb3VudC10
aGUtY2xvY2stbmFtZXMgYXBwcm9hY2gNCj4gPiBhbmQgZG9jdW1lbnQgaXQuDQo+IA0KPiBJcyB0
aGVyZSBhbnkgY3VycmVudCAyLjAgY2xvY2sgY29uc3VtZXIgdGhhdCByZWZlcmVuY2VzIHBsbC1k
aXY0Pw0KPiANCg0KSSBsb29rZWQgYXQgVDQyNDAgZm9yIGV4YW1wbGUsIHRoZXJlJ3MgYSBtdXgg
bm9kZSB3aGljaCBhZGRzIHBsbC1kaXY0IG9wdGlvbjoNCm11eDA6IG11eDBAMCB7DQoJI2Nsb2Nr
LWNlbGxzID0gPDA+Ow0KCXJlZyA9IDwweDAgMHg0PjsNCgljb21wYXRpYmxlID0gImZzbCxxb3Jp
cS1jb3JlLW11eC0yLjAiOw0KCWNsb2NrcyA9IDwmcGxsMCAwPiwgPCZwbGwwIDE+LCA8JnBsbDAg
Mj4sDQoJCTwmcGxsMSAwPiwgPCZwbGwxIDE+LCA8JnBsbDEgMj4sDQoJCTwmcGxsMiAwPiwgPCZw
bGwyIDE+LCA8JnBsbDIgMj47DQoJY2xvY2stbmFtZXMgPSAicGxsMCIsICJwbGwwLWRpdjIiLCAi
cGxsMC1kaXY0IiwNCgkJInBsbDEiLCAicGxsMS1kaXYyIiwgInBsbDEtZGl2NCIsDQoJCSJwbGwy
IiwgInBsbDItZGl2MiIsICJwbGwyLWRpdjQiOw0KCWNsb2NrLW91dHB1dC1uYW1lcyA9ICJjbXV4
MCI7DQp9Ow0KDQpBZnRlciB0aGlzIGNoYW5nZSA8JnBsbDAgMj4gd2lsbCByZXByZXNlbnQgInBs
bDAtZGl2MyIgYW5kIG5vdCAicGxsMC1kaXY0Ii4NCiANCj4gLVNjb3R0DQoNCg==

^ permalink raw reply	[flat|nested] 12+ messages in thread
* Re: [PATCH] powerpc/dts: Update the core cluster PLL node(s)
  2015-04-15 11:07         ` Igal.Liberman
@ 2015-04-15 17:14             ` Scott Wood
  -1 siblings, 0 replies; 12+ messages in thread
From: Scott Wood @ 2015-04-15 17:14 UTC (permalink / raw)
  To: Liberman Igal-B31950
  Cc: devicetree-u79uwXL29TY76Z2rM5mHXA, linuxppc-dev-uLR06cmDAlY/bJ5BZ2RsiQ

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] 12+ messages in thread
* Re: [PATCH] powerpc/dts: Update the core cluster PLL node(s)
@ 2015-04-15 17:14             ` Scott Wood
  0 siblings, 0 replies; 12+ messages in thread
From: Scott Wood @ 2015-04-15 17:14 UTC (permalink / raw)
  To: Liberman Igal-B31950; +Cc: devicetree, linuxppc-dev

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.

-Scott

^ permalink raw reply	[flat|nested] 12+ 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
  -1 siblings, 0 replies; 12+ messages in thread
From: Igal.Liberman @ 2015-04-16  6:13 UTC (permalink / raw)
  To: Scott Wood; +Cc: devicetree, linuxppc-dev



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] 12+ messages in thread
* RE: [PATCH] powerpc/dts: Update the core cluster PLL node(s)
@ 2015-04-16  6:13               ` Igal.Liberman
  0 siblings, 0 replies; 12+ messages in thread
From: Igal.Liberman @ 2015-04-16  6:13 UTC (permalink / raw)
  To: Scott Wood; +Cc: devicetree, linuxppc-dev

DQoNClJlZ2FyZHMsDQpJZ2FsIExpYmVybWFuLg0KDQo+IC0tLS0tT3JpZ2luYWwgTWVzc2FnZS0t
LS0tDQo+IEZyb206IFdvb2QgU2NvdHQtQjA3NDIxDQo+IFNlbnQ6IFdlZG5lc2RheSwgQXByaWwg
MTUsIDIwMTUgODoxNSBQTQ0KPiBUbzogTGliZXJtYW4gSWdhbC1CMzE5NTANCj4gQ2M6IGRldmlj
ZXRyZWVAdmdlci5rZXJuZWwub3JnOyBsaW51eHBwYy1kZXZAbGlzdHMub3psYWJzLm9yZw0KPiBT
dWJqZWN0OiBSZTogW1BBVENIXSBwb3dlcnBjL2R0czogVXBkYXRlIHRoZSBjb3JlIGNsdXN0ZXIg
UExMIG5vZGUocykNCj4gDQo+IE9uIFdlZCwgMjAxNS0wNC0xNSBhdCAwNjowNyAtMDUwMCwgTGli
ZXJtYW4gSWdhbC1CMzE5NTAgd3JvdGU6DQo+ID4NCj4gPg0KPiA+IFJlZ2FyZHMsDQo+ID4gSWdh
bCBMaWJlcm1hbi4NCj4gPg0KPiA+ID4gLS0tLS1PcmlnaW5hbCBNZXNzYWdlLS0tLS0NCj4gPiA+
IEZyb206IFdvb2QgU2NvdHQtQjA3NDIxDQo+ID4gPiBTZW50OiBUdWVzZGF5LCBBcHJpbCAxNCwg
MjAxNSAxMToyMyBQTQ0KPiA+ID4gVG86IExpYmVybWFuIElnYWwtQjMxOTUwDQo+ID4gPiBDYzog
ZGV2aWNldHJlZUB2Z2VyLmtlcm5lbC5vcmc7IGxpbnV4cHBjLWRldkBsaXN0cy5vemxhYnMub3Jn
DQo+ID4gPiBTdWJqZWN0OiBSZTogW1BBVENIXSBwb3dlcnBjL2R0czogVXBkYXRlIHRoZSBjb3Jl
IGNsdXN0ZXIgUExMDQo+ID4gPiBub2RlKHMpDQo+ID4gPg0KPiA+ID4gT24gVHVlLCAyMDE1LTA0
LTE0IGF0IDE1OjIxIC0wNTAwLCBTY290dCBXb29kIHdyb3RlOg0KPiA+ID4gPiBPbiBUdWUsIDIw
MTUtMDQtMTQgYXQgMTI6NTUgKzAzMDAsIElnYWwuTGliZXJtYW4gd3JvdGU6DQo+ID4gPiA+ID4g
RnJvbTogSWdhbCBMaWJlcm1hbiA8SWdhbC5MaWJlcm1hbkBmcmVlc2NhbGUuY29tPg0KPiA+ID4g
PiA+DQo+ID4gPiA+ID4gVGhpcyBwYXRjaCByZXBsYWNlcyB0aGUgZm9sbG93aW5nOg0KPiA+ID4g
PiA+IAlodHRwczovL3BhdGNod29yay5vemxhYnMub3JnL3BhdGNoLzQyNzY2NC8NCj4gPiA+ID4g
Pg0KPiA+ID4gPiA+IFRoaXMgcGF0Y2ggaXMgZGVzY3JpYmVkIGJ5IHRoZSBmb2xsb3dpbmcgYmlu
ZGluZyBkb2N1bWVudCB1cGRhdGU6DQo+ID4gPiA+ID4gCWh0dHBzOi8vcGF0Y2h3b3JrLm96bGFi
cy5vcmcvcGF0Y2gvNDYxMTUwLw0KPiA+ID4gPiA+DQo+ID4gPiA+ID4gU2lnbmVkLW9mZi1ieTog
SWdhbCBMaWJlcm1hbiA8SWdhbC5MaWJlcm1hbkBmcmVlc2NhbGUuY29tPg0KPiA+ID4gPiA+IC0t
LQ0KPiA+ID4gPiA+ICBhcmNoL3Bvd2VycGMvYm9vdC9kdHMvZnNsL3FvcmlxLWNsb2NrZ2VuMi5k
dHNpIHwgICAgNiArKysrLS0NCj4gPiA+ID4gPiAgMSBmaWxlIGNoYW5nZWQsIDQgaW5zZXJ0aW9u
cygrKSwgMiBkZWxldGlvbnMoLSkNCj4gPiA+ID4gPg0KPiA+ID4gPiA+IGRpZmYgLS1naXQgYS9h
cmNoL3Bvd2VycGMvYm9vdC9kdHMvZnNsL3FvcmlxLWNsb2NrZ2VuMi5kdHNpDQo+ID4gPiA+ID4g
Yi9hcmNoL3Bvd2VycGMvYm9vdC9kdHMvZnNsL3FvcmlxLWNsb2NrZ2VuMi5kdHNpDQo+ID4gPiA+
ID4gaW5kZXggNDhlMGI2ZS4uN2UxZjA3NCAxMDA2NDQNCj4gPiA+ID4gPiAtLS0gYS9hcmNoL3Bv
d2VycGMvYm9vdC9kdHMvZnNsL3FvcmlxLWNsb2NrZ2VuMi5kdHNpDQo+ID4gPiA+ID4gKysrIGIv
YXJjaC9wb3dlcnBjL2Jvb3QvZHRzL2ZzbC9xb3JpcS1jbG9ja2dlbjIuZHRzaQ0KPiA+ID4gPiA+
IEBAIC00OSwxNCArNDksMTYgQEAgZ2xvYmFsLXV0aWxpdGllc0BlMTAwMCB7DQo+ID4gPiA+ID4g
IAkJcmVnID0gPDB4ODAwIDB4ND47DQo+ID4gPiA+ID4gIAkJY29tcGF0aWJsZSA9ICJmc2wscW9y
aXEtY29yZS1wbGwtMi4wIjsNCj4gPiA+ID4gPiAgCQljbG9ja3MgPSA8JnN5c2Nsaz47DQo+ID4g
PiA+ID4gLQkJY2xvY2stb3V0cHV0LW5hbWVzID0gInBsbDAiLCAicGxsMC1kaXYyIiwgInBsbDAt
ZGl2NCI7DQo+ID4gPiA+ID4gKwkJY2xvY2stb3V0cHV0LW5hbWVzID0gInBsbDAiLCAicGxsMC1k
aXYyIiwgInBsbDAtZGl2MyIsDQo+ID4gPiA+ID4gKwkJCQkgICAgICAicGxsMC1kaXY0IjsNCj4g
PiA+ID4gPiAgCX07DQo+ID4gPiA+ID4gIAlwbGwxOiBwbGwxQDgyMCB7DQo+ID4gPiA+ID4gIAkJ
I2Nsb2NrLWNlbGxzID0gPDE+Ow0KPiA+ID4gPiA+ICAJCXJlZyA9IDwweDgyMCAweDQ+Ow0KPiA+
ID4gPiA+ICAJCWNvbXBhdGlibGUgPSAiZnNsLHFvcmlxLWNvcmUtcGxsLTIuMCI7DQo+ID4gPiA+
ID4gIAkJY2xvY2tzID0gPCZzeXNjbGs+Ow0KPiA+ID4gPiA+IC0JCWNsb2NrLW91dHB1dC1uYW1l
cyA9ICJwbGwxIiwgInBsbDEtZGl2MiIsICJwbGwxLWRpdjQiOw0KPiA+ID4gPiA+ICsJCWNsb2Nr
LW91dHB1dC1uYW1lcyA9ICJwbGwxIiwgInBsbDEtZGl2MiIsICJwbGwxLWRpdjMiLA0KPiA+ID4g
PiA+ICsJCQkJICAgICAgInBsbDEtZGl2NCI7DQo+ID4gPiA+DQo+ID4gPiA+IFdhaXQsIHNvIGlm
IHRoZSBkcml2ZXIgaW1wbGVtZW50cyB0aGUgYmluZGluZyB5b3Ugc3VibWl0dGVkLA0KPiA+ID4g
PiB5b3UnbGwgYnJlYWsgY29tcGF0aWJpbGl0eSB3aXRoIHRoZXNlIG9sZGVyIGRldmljZSB0cmVl
cy4uLg0KPiA+ID4gPg0KPiA+ID4gPiBJIHRoaW5rIHdlIG5lZWQgdG8ganVzdCBhY2NlcHQgdGhl
IHVnbHkgY291bnQtdGhlLWNsb2NrLW5hbWVzDQo+ID4gPiA+IGFwcHJvYWNoIGFuZCBkb2N1bWVu
dCBpdC4NCj4gPiA+DQo+ID4gPiBJcyB0aGVyZSBhbnkgY3VycmVudCAyLjAgY2xvY2sgY29uc3Vt
ZXIgdGhhdCByZWZlcmVuY2VzIHBsbC1kaXY0Pw0KPiA+ID4NCj4gPg0KPiA+IEkgbG9va2VkIGF0
IFQ0MjQwIGZvciBleGFtcGxlLCB0aGVyZSdzIGEgbXV4IG5vZGUgd2hpY2ggYWRkcyBwbGwtZGl2
NA0KPiBvcHRpb246DQo+ID4gbXV4MDogbXV4MEAwIHsNCj4gPiAJI2Nsb2NrLWNlbGxzID0gPDA+
Ow0KPiA+IAlyZWcgPSA8MHgwIDB4ND47DQo+ID4gCWNvbXBhdGlibGUgPSAiZnNsLHFvcmlxLWNv
cmUtbXV4LTIuMCI7DQo+ID4gCWNsb2NrcyA9IDwmcGxsMCAwPiwgPCZwbGwwIDE+LCA8JnBsbDAg
Mj4sDQo+ID4gCQk8JnBsbDEgMD4sIDwmcGxsMSAxPiwgPCZwbGwxIDI+LA0KPiA+IAkJPCZwbGwy
IDA+LCA8JnBsbDIgMT4sIDwmcGxsMiAyPjsNCj4gPiAJY2xvY2stbmFtZXMgPSAicGxsMCIsICJw
bGwwLWRpdjIiLCAicGxsMC1kaXY0IiwNCj4gPiAJCSJwbGwxIiwgInBsbDEtZGl2MiIsICJwbGwx
LWRpdjQiLA0KPiA+IAkJInBsbDIiLCAicGxsMi1kaXYyIiwgInBsbDItZGl2NCI7DQo+ID4gCWNs
b2NrLW91dHB1dC1uYW1lcyA9ICJjbXV4MCI7DQo+ID4gfTsNCj4gPg0KPiA+IEFmdGVyIHRoaXMg
Y2hhbmdlIDwmcGxsMCAyPiB3aWxsIHJlcHJlc2VudCAicGxsMC1kaXYzIiBhbmQgbm90ICJwbGww
LWRpdjQiLg0KPiANCj4gU28gdGhpcyBuZWVkcyB0byBiZSB1cGRhdGVkIHRvIG1hdGNoIC0tIGFu
ZCBpdCBjb25maXJtcyB0aGF0IGV4aXN0aW5nIGRldmljZQ0KPiB0cmVlcyB3aWxsIGJlIGJyb2tl
biBpZiB5b3UgYmFzZSB0aGUgaW50ZXJwcmV0YXRpb24gb24gY29tcGF0aWJsZSByYXRoZXIgdGhh
bg0KPiB0aGUgbnVtYmVyIG9mIGNsb2NrLW91dHB1dC1uYW1lcy4NCj4gDQoNClllcy4NCkknbGwg
c3VibWl0IGEgcGF0Y2ggYW5kIG1lbnRpb24gdGhpcyBkZXBlbmRlbmN5LiANCg0KPiAtU2NvdHQN
Cj4gDQoNCklnYWwuDQoNCg==

^ permalink raw reply	[flat|nested] 12+ messages in thread
end of thread, other threads:[~2015-04-16  6:13 UTC | newest]

Thread overview: 12+ 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  9:55 ` Igal.Liberman
2015-04-14 20:21 ` Scott Wood
2015-04-14 20:21   ` Scott Wood
2015-04-14 20:22   ` 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
2015-04-15 11:07         ` Igal.Liberman
     [not found]         ` <DM2PR03MB3831BEEDD2C17991818FFA9E6E50-ufbTtyGzTTRJonC5hhDUuuO6mTEJWrR4XA4E9RH9d+qIuWR1G4zioA@public.gmane.org>
2015-04-15 17:14           ` Scott Wood
2015-04-15 17:14             ` Scott Wood
2015-04-16  6:13             ` Igal.Liberman
2015-04-16  6:13               ` Igal.Liberman

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.