From: "Igal.Liberman@freescale.com" <Igal.Liberman@freescale.com> To: Scott Wood <scottwood@freescale.com> Cc: "devicetree@vger.kernel.org" <devicetree@vger.kernel.org>, "linuxppc-dev@lists.ozlabs.org" <linuxppc-dev@lists.ozlabs.org> Subject: RE: [PATCH] powerpc/dts: Update the core cluster PLL node(s) Date: Thu, 16 Apr 2015 06:13:04 +0000 [thread overview] Message-ID: <DM2PR03MB383381357861862EC77024DE6E40@DM2PR03MB383.namprd03.prod.outlook.com> (raw) In-Reply-To: <1429118075.22867.719.camel@freescale.com> 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
WARNING: multiple messages have this Message-ID (diff)
From: "Igal.Liberman@freescale.com" <Igal.Liberman@freescale.com> To: Scott Wood <scottwood@freescale.com> Cc: "devicetree@vger.kernel.org" <devicetree@vger.kernel.org>, "linuxppc-dev@lists.ozlabs.org" <linuxppc-dev@lists.ozlabs.org> Subject: RE: [PATCH] powerpc/dts: Update the core cluster PLL node(s) Date: Thu, 16 Apr 2015 06:13:04 +0000 [thread overview] Message-ID: <DM2PR03MB383381357861862EC77024DE6E40@DM2PR03MB383.namprd03.prod.outlook.com> (raw) In-Reply-To: <1429118075.22867.719.camel@freescale.com> 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==
next prev parent reply other threads:[~2015-04-16 6:13 UTC|newest] Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top 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 [this message] 2015-04-16 6:13 ` Igal.Liberman
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=DM2PR03MB383381357861862EC77024DE6E40@DM2PR03MB383.namprd03.prod.outlook.com \ --to=igal.liberman@freescale.com \ --cc=devicetree@vger.kernel.org \ --cc=linuxppc-dev@lists.ozlabs.org \ --cc=scottwood@freescale.com \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
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.