[PATCH] ARM: dts: keystone: fix netcp's clocks definitions

[PATCH] ARM: dts: keystone: fix netcp's clocks definitions

[Grygorii Strashko]

The clocks tree for Keystone 2 NTCP devices should be
defined as following:
[refclk] - board dependent
 |- <papllclk> - PLL clock
    |- <paclk13> - fixed factor clock div=3 mul=1
       |- <clkpa> - gated clock
       |- <clkcpgmac> - gated clock
       |- <clksa> - gated clock

Hence, update Keystone 2 DT to follow HW specification.

Signed-off-by: WingMan Kwok <w-kwok2-l0cyMroinI0@public.gmane.org>
Signed-off-by: Grygorii Strashko <grygorii.strashko-l0cyMroinI0@public.gmane.org>
---
 arch/arm/boot/dts/k2e-clocks.dtsi      |    2 +-
 arch/arm/boot/dts/k2hk-clocks.dtsi     |    2 +-
 arch/arm/boot/dts/k2l-clocks.dtsi      |    2 +-
 arch/arm/boot/dts/keystone-clocks.dtsi |    2 +-
 4 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/arch/arm/boot/dts/k2e-clocks.dtsi b/arch/arm/boot/dts/k2e-clocks.dtsi
index 90774d6..598afe9 100644
--- a/arch/arm/boot/dts/k2e-clocks.dtsi
+++ b/arch/arm/boot/dts/k2e-clocks.dtsi
@@ -22,7 +22,7 @@ clocks {
 		#clock-cells = <0>;
 		compatible = "ti,keystone,pll-clock";
 		clocks = <&refclkpass>;
-		clock-output-names = "pa-pll-clk";
+		clock-output-names = "papllclk";
 		reg = <0x02620358 4>;
 		reg-names = "control";
 	};
diff --git a/arch/arm/boot/dts/k2hk-clocks.dtsi b/arch/arm/boot/dts/k2hk-clocks.dtsi
index 96e6536..d5adee3 100644
--- a/arch/arm/boot/dts/k2hk-clocks.dtsi
+++ b/arch/arm/boot/dts/k2hk-clocks.dtsi
@@ -31,7 +31,7 @@ clocks {
 		#clock-cells = <0>;
 		compatible = "ti,keystone,pll-clock";
 		clocks = <&refclkpass>;
-		clock-output-names = "pa-pll-clk";
+		clock-output-names = "papllclk";
 		reg = <0x02620358 4>;
 		reg-names = "control";
 	};
diff --git a/arch/arm/boot/dts/k2l-clocks.dtsi b/arch/arm/boot/dts/k2l-clocks.dtsi
index f584b80..eb1e3e2 100644
--- a/arch/arm/boot/dts/k2l-clocks.dtsi
+++ b/arch/arm/boot/dts/k2l-clocks.dtsi
@@ -31,7 +31,7 @@ clocks {
 		#clock-cells = <0>;
 		compatible = "ti,keystone,pll-clock";
 		clocks = <&refclksys>;
-		clock-output-names = "pa-pll-clk";
+		clock-output-names = "papllclk";
 		reg = <0x02620358 4>;
 		reg-names = "control";
 	};
diff --git a/arch/arm/boot/dts/keystone-clocks.dtsi b/arch/arm/boot/dts/keystone-clocks.dtsi
index 93f82c7..0c334b2 100644
--- a/arch/arm/boot/dts/keystone-clocks.dtsi
+++ b/arch/arm/boot/dts/keystone-clocks.dtsi
@@ -215,7 +215,7 @@ clocks {
 	clkpa: clkpa {
 		#clock-cells = <0>;
 		compatible = "ti,keystone,psc-clock";
-		clocks = <&chipclk16>;
+		clocks = <&paclk13>;
 		clock-output-names = "pa";
 		reg = <0x0235001c 0xb00>, <0x02350008 0x400>;
 		reg-names = "control", "domain";
-- 
1.7.9.5

--
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] ARM: dts: keystone: fix netcp's clocks definitions

[Santosh Shilimkar]

On Tuesday 08 July 2014 11:43 AM, Grygorii Strashko wrote:
> The clocks tree for Keystone 2 NTCP devices should be
> defined as following:
> [refclk] - board dependent
>  |- <papllclk> - PLL clock
>     |- <paclk13> - fixed factor clock div=3 mul=1
>        |- <clkpa> - gated clock
>        |- <clkcpgmac> - gated clock
>        |- <clksa> - gated clock
> 
> Hence, update Keystone 2 DT to follow HW specification.
> 
> Signed-off-by: WingMan Kwok <w-kwok2-l0cyMroinI0@public.gmane.org>
> Signed-off-by: Grygorii Strashko <grygorii.strashko-l0cyMroinI0@public.gmane.org>
> ---
Thanks for the patch. Queuing it up for 3.17

>  arch/arm/boot/dts/k2e-clocks.dtsi      |    2 +-
>  arch/arm/boot/dts/k2hk-clocks.dtsi     |    2 +-
>  arch/arm/boot/dts/k2l-clocks.dtsi      |    2 +-
>  arch/arm/boot/dts/keystone-clocks.dtsi |    2 +-
>  4 files changed, 4 insertions(+), 4 deletions(-)
> 
> diff --git a/arch/arm/boot/dts/k2e-clocks.dtsi b/arch/arm/boot/dts/k2e-clocks.dtsi
> index 90774d6..598afe9 100644
> --- a/arch/arm/boot/dts/k2e-clocks.dtsi
> +++ b/arch/arm/boot/dts/k2e-clocks.dtsi
> @@ -22,7 +22,7 @@ clocks {
>  		#clock-cells = <0>;
>  		compatible = "ti,keystone,pll-clock";
>  		clocks = <&refclkpass>;
> -		clock-output-names = "pa-pll-clk";
> +		clock-output-names = "papllclk";
>  		reg = <0x02620358 4>;
>  		reg-names = "control";
>  	};
> diff --git a/arch/arm/boot/dts/k2hk-clocks.dtsi b/arch/arm/boot/dts/k2hk-clocks.dtsi
> index 96e6536..d5adee3 100644
> --- a/arch/arm/boot/dts/k2hk-clocks.dtsi
> +++ b/arch/arm/boot/dts/k2hk-clocks.dtsi
> @@ -31,7 +31,7 @@ clocks {
>  		#clock-cells = <0>;
>  		compatible = "ti,keystone,pll-clock";
>  		clocks = <&refclkpass>;
> -		clock-output-names = "pa-pll-clk";
> +		clock-output-names = "papllclk";
>  		reg = <0x02620358 4>;
>  		reg-names = "control";
>  	};
> diff --git a/arch/arm/boot/dts/k2l-clocks.dtsi b/arch/arm/boot/dts/k2l-clocks.dtsi
> index f584b80..eb1e3e2 100644
> --- a/arch/arm/boot/dts/k2l-clocks.dtsi
> +++ b/arch/arm/boot/dts/k2l-clocks.dtsi
> @@ -31,7 +31,7 @@ clocks {
>  		#clock-cells = <0>;
>  		compatible = "ti,keystone,pll-clock";
>  		clocks = <&refclksys>;
> -		clock-output-names = "pa-pll-clk";
> +		clock-output-names = "papllclk";
>  		reg = <0x02620358 4>;
>  		reg-names = "control";
>  	};
> diff --git a/arch/arm/boot/dts/keystone-clocks.dtsi b/arch/arm/boot/dts/keystone-clocks.dtsi
> index 93f82c7..0c334b2 100644
> --- a/arch/arm/boot/dts/keystone-clocks.dtsi
> +++ b/arch/arm/boot/dts/keystone-clocks.dtsi
> @@ -215,7 +215,7 @@ clocks {
>  	clkpa: clkpa {
>  		#clock-cells = <0>;
>  		compatible = "ti,keystone,psc-clock";
> -		clocks = <&chipclk16>;
> +		clocks = <&paclk13>;
>  		clock-output-names = "pa";
>  		reg = <0x0235001c 0xb00>, <0x02350008 0x400>;
>  		reg-names = "control", "domain";
> 

--
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