From: "Igal.Liberman-KZfg59tc24xl57MIdRCFDg@public.gmane.org" <Igal.Liberman-KZfg59tc24xl57MIdRCFDg@public.gmane.org> To: Scott Wood <scottwood-KZfg59tc24xl57MIdRCFDg@public.gmane.org> Cc: "devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org" <devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org>, "linuxppc-dev-uLR06cmDAlY/bJ5BZ2RsiQ@public.gmane.org" <linuxppc-dev-uLR06cmDAlY/bJ5BZ2RsiQ@public.gmane.org> Subject: RE: [v3] dt/bindings: qoriq-clock: Add binding for FMan clock mux Date: Mon, 20 Apr 2015 11:07:17 +0000 [thread overview] Message-ID: <DM2PR03MB38396D79588E66194EC7608E6E00@DM2PR03MB383.namprd03.prod.outlook.com> (raw) In-Reply-To: <1429249281.32545.52.camel-KZfg59tc24xl57MIdRCFDg@public.gmane.org> Regards, Igal Liberman. > -----Original Message----- > From: Wood Scott-B07421 > Sent: Friday, April 17, 2015 8:41 AM > To: Liberman Igal-B31950 > Cc: devicetree@vger.kernel.org; linuxppc-dev@lists.ozlabs.org > Subject: Re: [v3] dt/bindings: qoriq-clock: Add binding for FMan clock mux > > On Thu, 2015-04-16 at 01:11 -0500, Liberman Igal-B31950 wrote: > > > > > > Regards, > > Igal Liberman. > > > > > -----Original Message----- > > > From: Wood Scott-B07421 > > > Sent: Wednesday, April 15, 2015 8:36 PM > > > To: Liberman Igal-B31950 > > > Cc: devicetree@vger.kernel.org; linuxppc-dev@lists.ozlabs.org > > > Subject: Re: [v3] dt/bindings: qoriq-clock: Add binding for FMan > > > clock mux > > > > > > On Tue, 2015-04-14 at 13:56 +0300, Igal.Liberman wrote: > > > > From: Igal Liberman <Igal.Liberman@freescale.com> > > > > > > > > v3: Addressed feedback from Scott: > > > > - Removed clock specifier description. > > > > > > > > v2: Addressed feedback from Scott: > > > > - Moved the "fman-clk-mux" clock provider details > > > > under "clocks" property. > > > > > > > > Signed-off-by: Igal Liberman <Igal.Liberman@freescale.com> > > > > --- > > > > .../devicetree/bindings/clock/qoriq-clock.txt | 17 > +++++++++++++++-- > > > > 1 file changed, 15 insertions(+), 2 deletions(-) > > > > > > > > diff --git > > > > a/Documentation/devicetree/bindings/clock/qoriq-clock.txt > > > > b/Documentation/devicetree/bindings/clock/qoriq-clock.txt > > > > index b0d7b73..2bb3b38 100644 > > > > --- a/Documentation/devicetree/bindings/clock/qoriq-clock.txt > > > > +++ b/Documentation/devicetree/bindings/clock/qoriq-clock.txt > > > > @@ -65,9 +65,10 @@ Required properties: > > > > It takes parent's clock-frequency as its clock. > > > > * "fsl,qoriq-platform-pll-1.0" for the platform PLL clock (v1.0) > > > > * "fsl,qoriq-platform-pll-2.0" for the platform PLL clock (v2.0) > > > > + * "fsl,fman-clk-mux" for the Frame Manager clock. > > > > - #clock-cells: From common clock binding. The number of cells in a > > > > - clock-specifier. Should be <0> for "fsl,qoriq-sysclk-[1,2].0" > > > > - clocks, or <1> for "fsl,qoriq-core-pll-[1,2].0" clocks. > > > > + clock-specifier. Should be <0> for "fsl,qoriq-sysclk-[1,2].0" and > > > > + "fsl,fman-clk-mux" clocks or <1> for "fsl,qoriq-core-pll-[1,2].0". > > > > For "fsl,qoriq-core-pll-1.0" clocks, the single > > > > clock-specifier cell may take the following values: > > > > * 0 - equal to the PLL frequency @@ -145,6 +146,18 @@ Example > > > > for clock block and clock provider: > > > > clocks = <&sysclk>; > > > > clock-output-names = "platform-pll", "platform-pll- > > > div2"; > > > > }; > > > > + > > > > + fm0clk: fm0-clk-mux { > > > > + #clock-cells = <0>; > > > > + reg = <0x10 4> > > > > + compatible = "fsl,fman-clk-mux"; > > > > + clocks = <&pll0 0>, <&pll0 1>, <&pll0 2>, <&pll0 3>, > > > > + <&platform_pll 0>, <&pll1 1>, <&pll1 2>; > > > > + clock-names = "pll0", "pll0-div2", "pll0-div3", > > > > + "pll0-div4", "platform-pll", "pll1-div2", > > > > + "pll1-div3"; > > > > + clock-output-names = "fm0-clk"; > > > > + }; > > > > }; > > > > }; > > > > > > > > > > I don't see this register in the manuals for older DPAA chips, such > > > as > > > p4080 or p3041. Is it present but undocumented? Should I be > > > looking somewhere other than "Clocking Memory Map"? > > > > > > > It's available only in part of the new chips (T4, T2, B4). > > In T1024/T1040 there's only one option for FMan clock so this register is not > available. > > So it's part of the 2.0 chassis? I'd stick a 2.0 in there, then. Who knows what > we may see in the future. > OK, We can go with "fsl,fman-clk-mux-1/2-0.". In that case, we need to update FMan nodes and the clock driver: https://patchwork.ozlabs.org/patch/443973/ https://patchwork.ozlabs.org/patch/461813/ I will update those patches separately. > -Scott > Igal
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: [v3] dt/bindings: qoriq-clock: Add binding for FMan clock mux Date: Mon, 20 Apr 2015 11:07:17 +0000 [thread overview] Message-ID: <DM2PR03MB38396D79588E66194EC7608E6E00@DM2PR03MB383.namprd03.prod.outlook.com> (raw) In-Reply-To: <1429249281.32545.52.camel@freescale.com> DQoNClJlZ2FyZHMsDQpJZ2FsIExpYmVybWFuLg0KDQo+IC0tLS0tT3JpZ2luYWwgTWVzc2FnZS0t LS0tDQo+IEZyb206IFdvb2QgU2NvdHQtQjA3NDIxDQo+IFNlbnQ6IEZyaWRheSwgQXByaWwgMTcs IDIwMTUgODo0MSBBTQ0KPiBUbzogTGliZXJtYW4gSWdhbC1CMzE5NTANCj4gQ2M6IGRldmljZXRy ZWVAdmdlci5rZXJuZWwub3JnOyBsaW51eHBwYy1kZXZAbGlzdHMub3psYWJzLm9yZw0KPiBTdWJq ZWN0OiBSZTogW3YzXSBkdC9iaW5kaW5nczogcW9yaXEtY2xvY2s6IEFkZCBiaW5kaW5nIGZvciBG TWFuIGNsb2NrIG11eA0KPiANCj4gT24gVGh1LCAyMDE1LTA0LTE2IGF0IDAxOjExIC0wNTAwLCBM aWJlcm1hbiBJZ2FsLUIzMTk1MCB3cm90ZToNCj4gPg0KPiA+DQo+ID4gUmVnYXJkcywNCj4gPiBJ Z2FsIExpYmVybWFuLg0KPiA+DQo+ID4gPiAtLS0tLU9yaWdpbmFsIE1lc3NhZ2UtLS0tLQ0KPiA+ ID4gRnJvbTogV29vZCBTY290dC1CMDc0MjENCj4gPiA+IFNlbnQ6IFdlZG5lc2RheSwgQXByaWwg MTUsIDIwMTUgODozNiBQTQ0KPiA+ID4gVG86IExpYmVybWFuIElnYWwtQjMxOTUwDQo+ID4gPiBD YzogZGV2aWNldHJlZUB2Z2VyLmtlcm5lbC5vcmc7IGxpbnV4cHBjLWRldkBsaXN0cy5vemxhYnMu b3JnDQo+ID4gPiBTdWJqZWN0OiBSZTogW3YzXSBkdC9iaW5kaW5nczogcW9yaXEtY2xvY2s6IEFk ZCBiaW5kaW5nIGZvciBGTWFuDQo+ID4gPiBjbG9jayBtdXgNCj4gPiA+DQo+ID4gPiBPbiBUdWUs IDIwMTUtMDQtMTQgYXQgMTM6NTYgKzAzMDAsIElnYWwuTGliZXJtYW4gd3JvdGU6DQo+ID4gPiA+ IEZyb206IElnYWwgTGliZXJtYW4gPElnYWwuTGliZXJtYW5AZnJlZXNjYWxlLmNvbT4NCj4gPiA+ ID4NCj4gPiA+ID4gdjM6IEFkZHJlc3NlZCBmZWVkYmFjayBmcm9tIFNjb3R0Og0KPiA+ID4gPiAJ LSBSZW1vdmVkIGNsb2NrIHNwZWNpZmllciBkZXNjcmlwdGlvbi4NCj4gPiA+ID4NCj4gPiA+ID4g djI6IEFkZHJlc3NlZCBmZWVkYmFjayBmcm9tIFNjb3R0Og0KPiA+ID4gPiAJLSBNb3ZlZCB0aGUg ImZtYW4tY2xrLW11eCIgY2xvY2sgcHJvdmlkZXIgZGV0YWlscw0KPiA+ID4gPiAJICB1bmRlciAi Y2xvY2tzIiBwcm9wZXJ0eS4NCj4gPiA+ID4NCj4gPiA+ID4gU2lnbmVkLW9mZi1ieTogSWdhbCBM aWJlcm1hbiA8SWdhbC5MaWJlcm1hbkBmcmVlc2NhbGUuY29tPg0KPiA+ID4gPiAtLS0NCj4gPiA+ ID4gIC4uLi9kZXZpY2V0cmVlL2JpbmRpbmdzL2Nsb2NrL3FvcmlxLWNsb2NrLnR4dCAgICAgIHwg ICAxNw0KPiArKysrKysrKysrKysrKystLQ0KPiA+ID4gPiAgMSBmaWxlIGNoYW5nZWQsIDE1IGlu c2VydGlvbnMoKyksIDIgZGVsZXRpb25zKC0pDQo+ID4gPiA+DQo+ID4gPiA+IGRpZmYgLS1naXQN Cj4gPiA+ID4gYS9Eb2N1bWVudGF0aW9uL2RldmljZXRyZWUvYmluZGluZ3MvY2xvY2svcW9yaXEt Y2xvY2sudHh0DQo+ID4gPiA+IGIvRG9jdW1lbnRhdGlvbi9kZXZpY2V0cmVlL2JpbmRpbmdzL2Ns b2NrL3FvcmlxLWNsb2NrLnR4dA0KPiA+ID4gPiBpbmRleCBiMGQ3YjczLi4yYmIzYjM4IDEwMDY0 NA0KPiA+ID4gPiAtLS0gYS9Eb2N1bWVudGF0aW9uL2RldmljZXRyZWUvYmluZGluZ3MvY2xvY2sv cW9yaXEtY2xvY2sudHh0DQo+ID4gPiA+ICsrKyBiL0RvY3VtZW50YXRpb24vZGV2aWNldHJlZS9i aW5kaW5ncy9jbG9jay9xb3JpcS1jbG9jay50eHQNCj4gPiA+ID4gQEAgLTY1LDkgKzY1LDEwIEBA IFJlcXVpcmVkIHByb3BlcnRpZXM6DQo+ID4gPiA+ICAJCUl0IHRha2VzIHBhcmVudCdzIGNsb2Nr LWZyZXF1ZW5jeSBhcyBpdHMgY2xvY2suDQo+ID4gPiA+ICAJKiAiZnNsLHFvcmlxLXBsYXRmb3Jt LXBsbC0xLjAiIGZvciB0aGUgcGxhdGZvcm0gUExMIGNsb2NrICh2MS4wKQ0KPiA+ID4gPiAgCSog ImZzbCxxb3JpcS1wbGF0Zm9ybS1wbGwtMi4wIiBmb3IgdGhlIHBsYXRmb3JtIFBMTCBjbG9jayAo djIuMCkNCj4gPiA+ID4gKwkqICJmc2wsZm1hbi1jbGstbXV4IiBmb3IgdGhlIEZyYW1lIE1hbmFn ZXIgY2xvY2suDQo+ID4gPiA+ICAtICNjbG9jay1jZWxsczogRnJvbSBjb21tb24gY2xvY2sgYmlu ZGluZy4gVGhlIG51bWJlciBvZiBjZWxscyBpbiBhDQo+ID4gPiA+IC0JY2xvY2stc3BlY2lmaWVy LiBTaG91bGQgYmUgPDA+IGZvciAiZnNsLHFvcmlxLXN5c2Nsay1bMSwyXS4wIg0KPiA+ID4gPiAt CWNsb2Nrcywgb3IgPDE+IGZvciAiZnNsLHFvcmlxLWNvcmUtcGxsLVsxLDJdLjAiIGNsb2Nrcy4N Cj4gPiA+ID4gKwljbG9jay1zcGVjaWZpZXIuIFNob3VsZCBiZSA8MD4gZm9yICJmc2wscW9yaXEt c3lzY2xrLVsxLDJdLjAiIGFuZA0KPiA+ID4gPiArCSJmc2wsZm1hbi1jbGstbXV4IiBjbG9ja3Mg b3IgPDE+IGZvciAiZnNsLHFvcmlxLWNvcmUtcGxsLVsxLDJdLjAiLg0KPiA+ID4gPiAgCUZvciAi ZnNsLHFvcmlxLWNvcmUtcGxsLTEuMCIgY2xvY2tzLCB0aGUgc2luZ2xlDQo+ID4gPiA+ICAJY2xv Y2stc3BlY2lmaWVyIGNlbGwgbWF5IHRha2UgdGhlIGZvbGxvd2luZyB2YWx1ZXM6DQo+ID4gPiA+ ICAJKiAwIC0gZXF1YWwgdG8gdGhlIFBMTCBmcmVxdWVuY3kgQEAgLTE0NSw2ICsxNDYsMTggQEAg RXhhbXBsZQ0KPiA+ID4gPiBmb3IgY2xvY2sgYmxvY2sgYW5kIGNsb2NrIHByb3ZpZGVyOg0KPiA+ ID4gPiAgCQkJY2xvY2tzID0gPCZzeXNjbGs+Ow0KPiA+ID4gPiAgCQkJY2xvY2stb3V0cHV0LW5h bWVzID0gInBsYXRmb3JtLXBsbCIsICJwbGF0Zm9ybS1wbGwtDQo+ID4gPiBkaXYyIjsNCj4gPiA+ ID4gIAkJfTsNCj4gPiA+ID4gKw0KPiA+ID4gPiArCQlmbTBjbGs6IGZtMC1jbGstbXV4IHsNCj4g PiA+ID4gKwkJCSNjbG9jay1jZWxscyA9IDwwPjsNCj4gPiA+ID4gKwkJCXJlZyA9IDwweDEwIDQ+ DQo+ID4gPiA+ICsJCQljb21wYXRpYmxlID0gImZzbCxmbWFuLWNsay1tdXgiOw0KPiA+ID4gPiAr CQkJY2xvY2tzID0gPCZwbGwwIDA+LCA8JnBsbDAgMT4sIDwmcGxsMCAyPiwgPCZwbGwwIDM+LA0K PiA+ID4gPiArCQkJCSA8JnBsYXRmb3JtX3BsbCAwPiwgPCZwbGwxIDE+LCA8JnBsbDEgMj47DQo+ ID4gPiA+ICsJCQljbG9jay1uYW1lcyA9ICJwbGwwIiwgInBsbDAtZGl2MiIsICJwbGwwLWRpdjMi LA0KPiA+ID4gPiArCQkJCSAgICAgICJwbGwwLWRpdjQiLCAicGxhdGZvcm0tcGxsIiwgInBsbDEt ZGl2MiIsDQo+ID4gPiA+ICsJCQkJICAgICAgInBsbDEtZGl2MyI7DQo+ID4gPiA+ICsJCQljbG9j ay1vdXRwdXQtbmFtZXMgPSAiZm0wLWNsayI7DQo+ID4gPiA+ICsJCX07DQo+ID4gPiA+ICAJfTsN Cj4gPiA+ID4gIH07DQo+ID4gPiA+DQo+ID4gPg0KPiA+ID4gSSBkb24ndCBzZWUgdGhpcyByZWdp c3RlciBpbiB0aGUgbWFudWFscyBmb3Igb2xkZXIgRFBBQSBjaGlwcywgc3VjaA0KPiA+ID4gYXMN Cj4gPiA+IHA0MDgwIG9yIHAzMDQxLiAgSXMgaXQgcHJlc2VudCBidXQgdW5kb2N1bWVudGVkPyAg U2hvdWxkIEkgYmUNCj4gPiA+IGxvb2tpbmcgc29tZXdoZXJlIG90aGVyIHRoYW4gIkNsb2NraW5n IE1lbW9yeSBNYXAiPw0KPiA+ID4NCj4gPg0KPiA+IEl0J3MgYXZhaWxhYmxlIG9ubHkgaW4gcGFy dCBvZiB0aGUgbmV3IGNoaXBzIChUNCwgVDIsIEI0KS4NCj4gPiBJbiBUMTAyNC9UMTA0MCB0aGVy ZSdzIG9ubHkgb25lIG9wdGlvbiBmb3IgRk1hbiBjbG9jayBzbyB0aGlzIHJlZ2lzdGVyIGlzIG5v dA0KPiBhdmFpbGFibGUuDQo+IA0KPiBTbyBpdCdzIHBhcnQgb2YgdGhlIDIuMCBjaGFzc2lzPyAg SSdkIHN0aWNrIGEgMi4wIGluIHRoZXJlLCB0aGVuLiAgV2hvIGtub3dzIHdoYXQNCj4gd2UgbWF5 IHNlZSBpbiB0aGUgZnV0dXJlLg0KPiANCg0KT0ssDQpXZSBjYW4gZ28gd2l0aCAiZnNsLGZtYW4t Y2xrLW11eC0xLzItMC4iLg0KSW4gdGhhdCBjYXNlLCB3ZSBuZWVkIHRvIHVwZGF0ZSBGTWFuIG5v ZGVzIGFuZCB0aGUgY2xvY2sgZHJpdmVyOg0KaHR0cHM6Ly9wYXRjaHdvcmsub3psYWJzLm9yZy9w YXRjaC80NDM5NzMvDQpodHRwczovL3BhdGNod29yay5vemxhYnMub3JnL3BhdGNoLzQ2MTgxMy8N Ckkgd2lsbCB1cGRhdGUgdGhvc2UgcGF0Y2hlcyBzZXBhcmF0ZWx5Lg0KDQo+IC1TY290dA0KPiAN Cg0KSWdhbA0KDQo=
next prev parent reply other threads:[~2015-04-20 11:07 UTC|newest] Thread overview: 26+ messages / expand[flat|nested] mbox.gz Atom feed top 2015-04-14 10:56 [v3] dt/bindings: qoriq-clock: Add binding for FMan clock mux Igal.Liberman 2015-04-14 10:56 ` Igal.Liberman 2015-04-15 17:35 ` Scott Wood 2015-04-15 17:35 ` Scott Wood [not found] ` <1429119357.22867.724.camel-KZfg59tc24xl57MIdRCFDg@public.gmane.org> 2015-04-16 6:11 ` Igal.Liberman-KZfg59tc24xl57MIdRCFDg 2015-04-16 6:11 ` Igal.Liberman [not found] ` <DM2PR03MB38330DBBFFD1BC45D58F447E6E40-ufbTtyGzTTRJonC5hhDUuuO6mTEJWrR4XA4E9RH9d+qIuWR1G4zioA@public.gmane.org> 2015-04-17 5:41 ` Scott Wood 2015-04-17 5:41 ` Scott Wood [not found] ` <1429249281.32545.52.camel-KZfg59tc24xl57MIdRCFDg@public.gmane.org> 2015-04-20 11:07 ` Igal.Liberman-KZfg59tc24xl57MIdRCFDg [this message] 2015-04-20 11:07 ` Igal.Liberman 2015-04-20 11:40 ` Igal.Liberman-KZfg59tc24xl57MIdRCFDg 2015-04-20 11:40 ` Igal.Liberman [not found] ` <DM2PR03MB383E5F67C358CCE2E38EA71E6E00-ufbTtyGzTTRJonC5hhDUuuO6mTEJWrR4XA4E9RH9d+qIuWR1G4zioA@public.gmane.org> 2015-04-21 0:51 ` Scott Wood 2015-04-21 0:51 ` Scott Wood [not found] ` <1429577504.4352.67.camel-KZfg59tc24xl57MIdRCFDg@public.gmane.org> 2015-04-22 10:47 ` Igal.Liberman-KZfg59tc24xl57MIdRCFDg 2015-04-22 10:47 ` Igal.Liberman [not found] ` <DM2PR03MB383C35FA5959662DD48D00BE6EE0-ufbTtyGzTTRJonC5hhDUuuO6mTEJWrR4XA4E9RH9d+qIuWR1G4zioA@public.gmane.org> 2015-04-30 0:30 ` Scott Wood 2015-04-30 0:30 ` Scott Wood [not found] ` <1430353832.16357.138.camel-KZfg59tc24xl57MIdRCFDg@public.gmane.org> 2015-04-30 14:28 ` Igal.Liberman-KZfg59tc24xl57MIdRCFDg 2015-04-30 14:28 ` Igal.Liberman [not found] ` <DM2PR03MB3839C9D3FDFD7978C1A3497E6D60-ufbTtyGzTTRJonC5hhDUuuO6mTEJWrR4XA4E9RH9d+qIuWR1G4zioA@public.gmane.org> 2015-05-01 23:42 ` Scott Wood 2015-05-01 23:42 ` Scott Wood [not found] ` <1430523775.16357.177.camel-KZfg59tc24xl57MIdRCFDg@public.gmane.org> 2015-05-05 21:02 ` Igal.Liberman-KZfg59tc24xl57MIdRCFDg 2015-05-05 21:02 ` Igal.Liberman [not found] ` <BY2PR03MB3794413FB3FF795DF520D23E6D10-+7O3WWA3DPtn35zppGJRk+O6mTEJWrR4XA4E9RH9d+qIuWR1G4zioA@public.gmane.org> 2015-05-05 21:16 ` Scott Wood 2015-05-05 21:16 ` Scott Wood
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=DM2PR03MB38396D79588E66194EC7608E6E00@DM2PR03MB383.namprd03.prod.outlook.com \ --to=igal.liberman-kzfg59tc24xl57midrcfdg@public.gmane.org \ --cc=devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \ --cc=linuxppc-dev-uLR06cmDAlY/bJ5BZ2RsiQ@public.gmane.org \ --cc=scottwood-KZfg59tc24xl57MIdRCFDg@public.gmane.org \ /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.