[PATCH] dt/bindings: qoriq-clock: Add binding for FMan clock mux

[PATCH] dt/bindings: qoriq-clock: Add binding for FMan clock mux

[Igal.Liberman]

From: Igal Liberman <Igal.Liberman@freescale.com>

Signed-off-by: Igal Liberman <Igal.Liberman@freescale.com>
---
 .../devicetree/bindings/clock/qoriq-clock.txt      |   59 ++++++++++++++++++++
 1 file changed, 59 insertions(+)

diff --git a/Documentation/devicetree/bindings/clock/qoriq-clock.txt b/Documentation/devicetree/bindings/clock/qoriq-clock.txt
index df4a259..a7e84ce 100644
--- a/Documentation/devicetree/bindings/clock/qoriq-clock.txt
+++ b/Documentation/devicetree/bindings/clock/qoriq-clock.txt
@@ -65,6 +65,7 @@ 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.
@@ -73,6 +74,52 @@ Required properties:
 	* 0 - equal to the PLL frequency
 	* 1 - equal to the PLL frequency divided by 2
 	* 2 - equal to the PLL frequency divided by 4
+	For "fsl,fman-clk-mux" clocks, the single clock-specifier cell may
+	take values according the Reset Configuration Word of the specific
+	device:
+		* P2041, P3041:
+			* 0 - equal to platform PLL divided by 2
+			* 1 - equal to PLL2 divided by 2
+		* P4080 (Both FMans):
+			* 0 - equal to platform PLL divided by 2
+			* 1 - equal to PLL3 divided by 2
+		* P5020:
+			* 0 - equal to platform PLL divided by 2
+			* 1 - equal to PLL2 divided by 2
+			* 2 - equal to PLL2 divided by 4
+		* P5040 (Both FMans):
+			* 0 - equal to platform PLL divided by 2
+			* 1 - equal to PLL3 divided by 2
+			* 2 - equal to PLL3 divided by 4
+		* T1024:
+			* 0 - equal to PLL1 divided by 2
+		* T1040:
+			* 0 - equal to platform PLL
+		* T2080, B4860, B4420:
+			* 0 - equal to PLL1
+			* 1 - equal to PLL1 divided by 2
+			* 2 - equal to PLL1 divided by 3
+			* 3 - equal to PLL1 divided by 4
+			* 4 - equal to platform PLL
+			* 5 - equal to PLL2 divided by 2
+			* 6 - equal to PLL2 divided by 3
+		* T4240:
+			* FM1:
+			* 0 equal to PLL1 divided by 2
+			* 1 equal to PLL1 divided by 3
+			* 2 equal to PLL1 divided by 4
+			* 3 equal to platform PLL
+			* 4 equal to PLL2 divided by 2
+			* FM2:
+			* 0 equal to PLL2 divided by 2
+			* 1 equal to PLL2 divided by 3
+			* 2 equal to PLL2 divided by 4
+			* 3 equal to platform PLL
+			* 4 equal to PLL1 divided by 2
+			* 5 equal to PLL1 divided by 3
+		In Part of FMan V3 devices (B4, T2, T4) the single
+		clock-specifier cell may be determined by the CLKCGxHWACSR
+		register in addtion to RCW.
 
 Recommended properties:
 - clocks: Should be the phandle of input parent clock
@@ -139,6 +186,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";
+		};
 	};
 };
 
-- 
1.7.9.5

_______________________________________________
Linuxppc-dev mailing list
Linuxppc-dev@lists.ozlabs.org
https://lists.ozlabs.org/listinfo/linuxppc-dev

[PATCH] dt/bindings: qoriq-clock: Add binding for FMan clock mux

[Igal.Liberman]

From: Igal Liberman <Igal.Liberman@freescale.com>

Signed-off-by: Igal Liberman <Igal.Liberman@freescale.com>
---
 .../devicetree/bindings/clock/qoriq-clock.txt      |   59 ++++++++++++++++++++
 1 file changed, 59 insertions(+)

diff --git a/Documentation/devicetree/bindings/clock/qoriq-clock.txt b/Documentation/devicetree/bindings/clock/qoriq-clock.txt
index df4a259..a7e84ce 100644
--- a/Documentation/devicetree/bindings/clock/qoriq-clock.txt
+++ b/Documentation/devicetree/bindings/clock/qoriq-clock.txt
@@ -65,6 +65,7 @@ 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.
@@ -73,6 +74,52 @@ Required properties:
 	* 0 - equal to the PLL frequency
 	* 1 - equal to the PLL frequency divided by 2
 	* 2 - equal to the PLL frequency divided by 4
+	For "fsl,fman-clk-mux" clocks, the single clock-specifier cell may
+	take values according the Reset Configuration Word of the specific
+	device:
+		* P2041, P3041:
+			* 0 - equal to platform PLL divided by 2
+			* 1 - equal to PLL2 divided by 2
+		* P4080 (Both FMans):
+			* 0 - equal to platform PLL divided by 2
+			* 1 - equal to PLL3 divided by 2
+		* P5020:
+			* 0 - equal to platform PLL divided by 2
+			* 1 - equal to PLL2 divided by 2
+			* 2 - equal to PLL2 divided by 4
+		* P5040 (Both FMans):
+			* 0 - equal to platform PLL divided by 2
+			* 1 - equal to PLL3 divided by 2
+			* 2 - equal to PLL3 divided by 4
+		* T1024:
+			* 0 - equal to PLL1 divided by 2
+		* T1040:
+			* 0 - equal to platform PLL
+		* T2080, B4860, B4420:
+			* 0 - equal to PLL1
+			* 1 - equal to PLL1 divided by 2
+			* 2 - equal to PLL1 divided by 3
+			* 3 - equal to PLL1 divided by 4
+			* 4 - equal to platform PLL
+			* 5 - equal to PLL2 divided by 2
+			* 6 - equal to PLL2 divided by 3
+		* T4240:
+			* FM1:
+			* 0 equal to PLL1 divided by 2
+			* 1 equal to PLL1 divided by 3
+			* 2 equal to PLL1 divided by 4
+			* 3 equal to platform PLL
+			* 4 equal to PLL2 divided by 2
+			* FM2:
+			* 0 equal to PLL2 divided by 2
+			* 1 equal to PLL2 divided by 3
+			* 2 equal to PLL2 divided by 4
+			* 3 equal to platform PLL
+			* 4 equal to PLL1 divided by 2
+			* 5 equal to PLL1 divided by 3
+		In Part of FMan V3 devices (B4, T2, T4) the single
+		clock-specifier cell may be determined by the CLKCGxHWACSR
+		register in addtion to RCW.
 
 Recommended properties:
 - clocks: Should be the phandle of input parent clock
@@ -139,6 +186,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";
+		};
 	};
 };
 
-- 
1.7.9.5

Re: [PATCH] dt/bindings: qoriq-clock: Add binding for FMan clock mux

[Scott Wood]

On Tue, 2015-04-07 at 07:08 +0300, Igal.Liberman wrote:
> From: Igal Liberman <Igal.Liberman@freescale.com>
> 
> Signed-off-by: Igal Liberman <Igal.Liberman@freescale.com>
> ---
>  .../devicetree/bindings/clock/qoriq-clock.txt      |   59 ++++++++++++++++++++
>  1 file changed, 59 insertions(+)
> 
> diff --git a/Documentation/devicetree/bindings/clock/qoriq-clock.txt b/Documentation/devicetree/bindings/clock/qoriq-clock.txt
> index df4a259..a7e84ce 100644
> --- a/Documentation/devicetree/bindings/clock/qoriq-clock.txt
> +++ b/Documentation/devicetree/bindings/clock/qoriq-clock.txt
> @@ -65,6 +65,7 @@ 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.

Is there any versioning required on this?

>  - #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.
> @@ -73,6 +74,52 @@ Required properties:
>  	* 0 - equal to the PLL frequency
>  	* 1 - equal to the PLL frequency divided by 2
>  	* 2 - equal to the PLL frequency divided by 4
> +	For "fsl,fman-clk-mux" clocks, the single clock-specifier cell may
> +	take values according the Reset Configuration Word of the specific
> +	device:

fsl,fman-clk-mux nodes have #clock-cells of zero, so I don't understand
this section.

If this is meant to refer to the clock specifier that fsl,fman-clk-mux
nodes use in their input clocks, that is determined by the clock
provider and not the clock consumer.  

-Scott


_______________________________________________
Linuxppc-dev mailing list
Linuxppc-dev@lists.ozlabs.org
https://lists.ozlabs.org/listinfo/linuxppc-dev

Re: [PATCH] dt/bindings: qoriq-clock: Add binding for FMan clock mux

[Scott Wood]

On Tue, 2015-04-07 at 07:08 +0300, Igal.Liberman wrote:
> From: Igal Liberman <Igal.Liberman@freescale.com>
> 
> Signed-off-by: Igal Liberman <Igal.Liberman@freescale.com>
> ---
>  .../devicetree/bindings/clock/qoriq-clock.txt      |   59 ++++++++++++++++++++
>  1 file changed, 59 insertions(+)
> 
> diff --git a/Documentation/devicetree/bindings/clock/qoriq-clock.txt b/Documentation/devicetree/bindings/clock/qoriq-clock.txt
> index df4a259..a7e84ce 100644
> --- a/Documentation/devicetree/bindings/clock/qoriq-clock.txt
> +++ b/Documentation/devicetree/bindings/clock/qoriq-clock.txt
> @@ -65,6 +65,7 @@ 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.

Is there any versioning required on this?

>  - #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.
> @@ -73,6 +74,52 @@ Required properties:
>  	* 0 - equal to the PLL frequency
>  	* 1 - equal to the PLL frequency divided by 2
>  	* 2 - equal to the PLL frequency divided by 4
> +	For "fsl,fman-clk-mux" clocks, the single clock-specifier cell may
> +	take values according the Reset Configuration Word of the specific
> +	device:

fsl,fman-clk-mux nodes have #clock-cells of zero, so I don't understand
this section.

If this is meant to refer to the clock specifier that fsl,fman-clk-mux
nodes use in their input clocks, that is determined by the clock
provider and not the clock consumer.  

-Scott

RE: [PATCH] dt/bindings: qoriq-clock: Add binding for FMan clock mux

[Igal.Liberman-KZfg59tc24xl57MIdRCFDg]

Hi Scott

Regards,
Igal Liberman.

> -----Original Message-----
> From: Wood Scott-B07421
> Sent: Wednesday, April 08, 2015 1:40 AM
> To: Liberman Igal-B31950
> Cc: devicetree@vger.kernel.org; linuxppc-dev@lists.ozlabs.org
> Subject: Re: [PATCH] dt/bindings: qoriq-clock: Add binding for FMan clock
> mux
> 
> On Tue, 2015-04-07 at 07:08 +0300, Igal.Liberman wrote:
> > From: Igal Liberman <Igal.Liberman@freescale.com>
> >
> > Signed-off-by: Igal Liberman <Igal.Liberman@freescale.com>
> > ---
> >  .../devicetree/bindings/clock/qoriq-clock.txt      |   59
> ++++++++++++++++++++
> >  1 file changed, 59 insertions(+)
> >
> > diff --git a/Documentation/devicetree/bindings/clock/qoriq-clock.txt
> > b/Documentation/devicetree/bindings/clock/qoriq-clock.txt
> > index df4a259..a7e84ce 100644
> > --- a/Documentation/devicetree/bindings/clock/qoriq-clock.txt
> > +++ b/Documentation/devicetree/bindings/clock/qoriq-clock.txt
> > @@ -65,6 +65,7 @@ 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.
> 
> Is there any versioning required on this?
> 

I don't think we need versioning here.

> >  - #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.
> > @@ -73,6 +74,52 @@ Required properties:
> >  	* 0 - equal to the PLL frequency
> >  	* 1 - equal to the PLL frequency divided by 2
> >  	* 2 - equal to the PLL frequency divided by 4
> > +	For "fsl,fman-clk-mux" clocks, the single clock-specifier cell may
> > +	take values according the Reset Configuration Word of the specific
> > +	device:
> 
> fsl,fman-clk-mux nodes have #clock-cells of zero, so I don't understand this
> section.
> 
> If this is meant to refer to the clock specifier that fsl,fman-clk-mux nodes use
> in their input clocks, that is determined by the clock provider and not the
> clock consumer.
> 

This section refers to the clock provider.
I need to move the text bellow to more appropriate place, not under #clock-cells. 
Any comments on the content itself? 

> -Scott
> 

Thanks, Igal

RE: [PATCH] dt/bindings: qoriq-clock: Add binding for FMan clock mux

[Igal.Liberman]

SGkgU2NvdHQNCg0KUmVnYXJkcywNCklnYWwgTGliZXJtYW4uDQoNCj4gLS0tLS1PcmlnaW5hbCBN
ZXNzYWdlLS0tLS0NCj4gRnJvbTogV29vZCBTY290dC1CMDc0MjENCj4gU2VudDogV2VkbmVzZGF5
LCBBcHJpbCAwOCwgMjAxNSAxOjQwIEFNDQo+IFRvOiBMaWJlcm1hbiBJZ2FsLUIzMTk1MA0KPiBD
YzogZGV2aWNldHJlZUB2Z2VyLmtlcm5lbC5vcmc7IGxpbnV4cHBjLWRldkBsaXN0cy5vemxhYnMu
b3JnDQo+IFN1YmplY3Q6IFJlOiBbUEFUQ0hdIGR0L2JpbmRpbmdzOiBxb3JpcS1jbG9jazogQWRk
IGJpbmRpbmcgZm9yIEZNYW4gY2xvY2sNCj4gbXV4DQo+IA0KPiBPbiBUdWUsIDIwMTUtMDQtMDcg
YXQgMDc6MDggKzAzMDAsIElnYWwuTGliZXJtYW4gd3JvdGU6DQo+ID4gRnJvbTogSWdhbCBMaWJl
cm1hbiA8SWdhbC5MaWJlcm1hbkBmcmVlc2NhbGUuY29tPg0KPiA+DQo+ID4gU2lnbmVkLW9mZi1i
eTogSWdhbCBMaWJlcm1hbiA8SWdhbC5MaWJlcm1hbkBmcmVlc2NhbGUuY29tPg0KPiA+IC0tLQ0K
PiA+ICAuLi4vZGV2aWNldHJlZS9iaW5kaW5ncy9jbG9jay9xb3JpcS1jbG9jay50eHQgICAgICB8
ICAgNTkNCj4gKysrKysrKysrKysrKysrKysrKysNCj4gPiAgMSBmaWxlIGNoYW5nZWQsIDU5IGlu
c2VydGlvbnMoKykNCj4gPg0KPiA+IGRpZmYgLS1naXQgYS9Eb2N1bWVudGF0aW9uL2RldmljZXRy
ZWUvYmluZGluZ3MvY2xvY2svcW9yaXEtY2xvY2sudHh0DQo+ID4gYi9Eb2N1bWVudGF0aW9uL2Rl
dmljZXRyZWUvYmluZGluZ3MvY2xvY2svcW9yaXEtY2xvY2sudHh0DQo+ID4gaW5kZXggZGY0YTI1
OS4uYTdlODRjZSAxMDA2NDQNCj4gPiAtLS0gYS9Eb2N1bWVudGF0aW9uL2RldmljZXRyZWUvYmlu
ZGluZ3MvY2xvY2svcW9yaXEtY2xvY2sudHh0DQo+ID4gKysrIGIvRG9jdW1lbnRhdGlvbi9kZXZp
Y2V0cmVlL2JpbmRpbmdzL2Nsb2NrL3FvcmlxLWNsb2NrLnR4dA0KPiA+IEBAIC02NSw2ICs2NSw3
IEBAIFJlcXVpcmVkIHByb3BlcnRpZXM6DQo+ID4gIAkJSXQgdGFrZXMgcGFyZW50J3MgY2xvY2st
ZnJlcXVlbmN5IGFzIGl0cyBjbG9jay4NCj4gPiAgCSogImZzbCxxb3JpcS1wbGF0Zm9ybS1wbGwt
MS4wIiBmb3IgdGhlIHBsYXRmb3JtIFBMTCBjbG9jayAodjEuMCkNCj4gPiAgCSogImZzbCxxb3Jp
cS1wbGF0Zm9ybS1wbGwtMi4wIiBmb3IgdGhlIHBsYXRmb3JtIFBMTCBjbG9jayAodjIuMCkNCj4g
PiArCSogImZzbCxmbWFuLWNsay1tdXgiIGZvciB0aGUgRnJhbWUgTWFuYWdlciBjbG9jay4NCj4g
DQo+IElzIHRoZXJlIGFueSB2ZXJzaW9uaW5nIHJlcXVpcmVkIG9uIHRoaXM/DQo+IA0KDQpJIGRv
bid0IHRoaW5rIHdlIG5lZWQgdmVyc2lvbmluZyBoZXJlLg0KDQo+ID4gIC0gI2Nsb2NrLWNlbGxz
OiBGcm9tIGNvbW1vbiBjbG9jayBiaW5kaW5nLiBUaGUgbnVtYmVyIG9mIGNlbGxzIGluIGENCj4g
PiAgCWNsb2NrLXNwZWNpZmllci4gU2hvdWxkIGJlIDwwPiBmb3IgImZzbCxxb3JpcS1zeXNjbGst
WzEsMl0uMCINCj4gPiAgCWNsb2Nrcywgb3IgPDE+IGZvciAiZnNsLHFvcmlxLWNvcmUtcGxsLVsx
LDJdLjAiIGNsb2Nrcy4NCj4gPiBAQCAtNzMsNiArNzQsNTIgQEAgUmVxdWlyZWQgcHJvcGVydGll
czoNCj4gPiAgCSogMCAtIGVxdWFsIHRvIHRoZSBQTEwgZnJlcXVlbmN5DQo+ID4gIAkqIDEgLSBl
cXVhbCB0byB0aGUgUExMIGZyZXF1ZW5jeSBkaXZpZGVkIGJ5IDINCj4gPiAgCSogMiAtIGVxdWFs
IHRvIHRoZSBQTEwgZnJlcXVlbmN5IGRpdmlkZWQgYnkgNA0KPiA+ICsJRm9yICJmc2wsZm1hbi1j
bGstbXV4IiBjbG9ja3MsIHRoZSBzaW5nbGUgY2xvY2stc3BlY2lmaWVyIGNlbGwgbWF5DQo+ID4g
Kwl0YWtlIHZhbHVlcyBhY2NvcmRpbmcgdGhlIFJlc2V0IENvbmZpZ3VyYXRpb24gV29yZCBvZiB0
aGUgc3BlY2lmaWMNCj4gPiArCWRldmljZToNCj4gDQo+IGZzbCxmbWFuLWNsay1tdXggbm9kZXMg
aGF2ZSAjY2xvY2stY2VsbHMgb2YgemVybywgc28gSSBkb24ndCB1bmRlcnN0YW5kIHRoaXMNCj4g
c2VjdGlvbi4NCj4gDQo+IElmIHRoaXMgaXMgbWVhbnQgdG8gcmVmZXIgdG8gdGhlIGNsb2NrIHNw
ZWNpZmllciB0aGF0IGZzbCxmbWFuLWNsay1tdXggbm9kZXMgdXNlDQo+IGluIHRoZWlyIGlucHV0
IGNsb2NrcywgdGhhdCBpcyBkZXRlcm1pbmVkIGJ5IHRoZSBjbG9jayBwcm92aWRlciBhbmQgbm90
IHRoZQ0KPiBjbG9jayBjb25zdW1lci4NCj4gDQoNClRoaXMgc2VjdGlvbiByZWZlcnMgdG8gdGhl
IGNsb2NrIHByb3ZpZGVyLg0KSSBuZWVkIHRvIG1vdmUgdGhlIHRleHQgYmVsbG93IHRvIG1vcmUg
YXBwcm9wcmlhdGUgcGxhY2UsIG5vdCB1bmRlciAjY2xvY2stY2VsbHMuIA0KQW55IGNvbW1lbnRz
IG9uIHRoZSBjb250ZW50IGl0c2VsZj8gDQoNCj4gLVNjb3R0DQo+IA0KDQpUaGFua3MsIElnYWwN
Cg==

Re: [PATCH] dt/bindings: qoriq-clock: Add binding for FMan clock mux

[Scott Wood]

On Wed, 2015-04-08 at 03:54 -0500, Liberman Igal-B31950 wrote:
> Hi Scott
> 
> Regards,
> Igal Liberman.
> 
> > -----Original Message-----
> > From: Wood Scott-B07421
> > Sent: Wednesday, April 08, 2015 1:40 AM
> > To: Liberman Igal-B31950
> > Cc: devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org; linuxppc-dev-uLR06cmDAlY/bJ5BZ2RsiQ@public.gmane.org
> > Subject: Re: [PATCH] dt/bindings: qoriq-clock: Add binding for FMan clock
> > mux
> > 
> > On Tue, 2015-04-07 at 07:08 +0300, Igal.Liberman wrote:
> > > From: Igal Liberman <Igal.Liberman-KZfg59tc24xl57MIdRCFDg@public.gmane.org>
> > >
> > > Signed-off-by: Igal Liberman <Igal.Liberman-KZfg59tc24xl57MIdRCFDg@public.gmane.org>
> > > ---
> > >  .../devicetree/bindings/clock/qoriq-clock.txt      |   59
> > ++++++++++++++++++++
> > >  1 file changed, 59 insertions(+)
> > >
> > > diff --git a/Documentation/devicetree/bindings/clock/qoriq-clock.txt
> > > b/Documentation/devicetree/bindings/clock/qoriq-clock.txt
> > > index df4a259..a7e84ce 100644
> > > --- a/Documentation/devicetree/bindings/clock/qoriq-clock.txt
> > > +++ b/Documentation/devicetree/bindings/clock/qoriq-clock.txt
> > > @@ -65,6 +65,7 @@ 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.
> > 
> > Is there any versioning required on this?
> > 
> 
> I don't think we need versioning here.

Even though we needed it on the other nodes?

-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

Re: [PATCH] dt/bindings: qoriq-clock: Add binding for FMan clock mux

[Scott Wood]

On Wed, 2015-04-08 at 03:54 -0500, Liberman Igal-B31950 wrote:
> Hi Scott
> 
> Regards,
> Igal Liberman.
> 
> > -----Original Message-----
> > From: Wood Scott-B07421
> > Sent: Wednesday, April 08, 2015 1:40 AM
> > To: Liberman Igal-B31950
> > Cc: devicetree@vger.kernel.org; linuxppc-dev@lists.ozlabs.org
> > Subject: Re: [PATCH] dt/bindings: qoriq-clock: Add binding for FMan clock
> > mux
> > 
> > On Tue, 2015-04-07 at 07:08 +0300, Igal.Liberman wrote:
> > > From: Igal Liberman <Igal.Liberman@freescale.com>
> > >
> > > Signed-off-by: Igal Liberman <Igal.Liberman@freescale.com>
> > > ---
> > >  .../devicetree/bindings/clock/qoriq-clock.txt      |   59
> > ++++++++++++++++++++
> > >  1 file changed, 59 insertions(+)
> > >
> > > diff --git a/Documentation/devicetree/bindings/clock/qoriq-clock.txt
> > > b/Documentation/devicetree/bindings/clock/qoriq-clock.txt
> > > index df4a259..a7e84ce 100644
> > > --- a/Documentation/devicetree/bindings/clock/qoriq-clock.txt
> > > +++ b/Documentation/devicetree/bindings/clock/qoriq-clock.txt
> > > @@ -65,6 +65,7 @@ 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.
> > 
> > Is there any versioning required on this?
> > 
> 
> I don't think we need versioning here.

Even though we needed it on the other nodes?

-Scott