diff mbox

ARM: imx27.dtsi: fix CSPI PER clock id

Message ID 1376635535-10325-1-git-send-email-gwenhael.goavec-merou@armadeus.com (mailing list archive)
State New, archived
Headers show

Commit Message

Gwenhael Goavec-Merou Aug. 16, 2013, 6:45 a.m. UTC
CSPI PER clock is per2clk (per2_gate id 60) instead of cspiX_ipg_gate.

Signed-off-by: Gwenhael Goavec-Merou <gwenhael.goavec-merou@armadeus.com>
---
 arch/arm/boot/dts/imx27.dtsi | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

Comments

Shawn Guo Aug. 19, 2013, 2:24 p.m. UTC | #1
On Fri, Aug 16, 2013 at 08:45:35AM +0200, Gwenhael Goavec-Merou wrote:
> CSPI PER clock is per2clk (per2_gate id 60) instead of cspiX_ipg_gate.
> 
> Signed-off-by: Gwenhael Goavec-Merou <gwenhael.goavec-merou@armadeus.com>

Sascha,

Does it look right to you?

Shawn

> ---
>  arch/arm/boot/dts/imx27.dtsi | 6 +++---
>  1 file changed, 3 insertions(+), 3 deletions(-)
> 
> diff --git a/arch/arm/boot/dts/imx27.dtsi b/arch/arm/boot/dts/imx27.dtsi
> index 0695264..f71d233 100644
> --- a/arch/arm/boot/dts/imx27.dtsi
> +++ b/arch/arm/boot/dts/imx27.dtsi
> @@ -152,7 +152,7 @@
>  				compatible = "fsl,imx27-cspi";
>  				reg = <0x1000e000 0x1000>;
>  				interrupts = <16>;
> -				clocks = <&clks 53>, <&clks 53>;
> +				clocks = <&clks 53>, <&clks 60>;
>  				clock-names = "ipg", "per";
>  				status = "disabled";
>  			};
> @@ -163,7 +163,7 @@
>  				compatible = "fsl,imx27-cspi";
>  				reg = <0x1000f000 0x1000>;
>  				interrupts = <15>;
> -				clocks = <&clks 52>, <&clks 52>;
> +				clocks = <&clks 52>, <&clks 60>;
>  				clock-names = "ipg", "per";
>  				status = "disabled";
>  			};
> @@ -266,7 +266,7 @@
>  				compatible = "fsl,imx27-cspi";
>  				reg = <0x10017000 0x1000>;
>  				interrupts = <6>;
> -				clocks = <&clks 51>, <&clks 51>;
> +				clocks = <&clks 51>, <&clks 60>;
>  				clock-names = "ipg", "per";
>  				status = "disabled";
>  			};
> -- 
> 1.8.1.5
>
Shawn Guo Aug. 19, 2013, 2:43 p.m. UTC | #2
Correct Sascha's address.

On Fri, Aug 16, 2013 at 08:45:35AM +0200, Gwenhael Goavec-Merou wrote:
> CSPI PER clock is per2clk (per2_gate id 60) instead of cspiX_ipg_gate.
> 
> Signed-off-by: Gwenhael Goavec-Merou <gwenhael.goavec-merou@armadeus.com>
> ---
>  arch/arm/boot/dts/imx27.dtsi | 6 +++---
>  1 file changed, 3 insertions(+), 3 deletions(-)
> 
> diff --git a/arch/arm/boot/dts/imx27.dtsi b/arch/arm/boot/dts/imx27.dtsi
> index 0695264..f71d233 100644
> --- a/arch/arm/boot/dts/imx27.dtsi
> +++ b/arch/arm/boot/dts/imx27.dtsi
> @@ -152,7 +152,7 @@
>  				compatible = "fsl,imx27-cspi";
>  				reg = <0x1000e000 0x1000>;
>  				interrupts = <16>;
> -				clocks = <&clks 53>, <&clks 53>;
> +				clocks = <&clks 53>, <&clks 60>;
>  				clock-names = "ipg", "per";
>  				status = "disabled";
>  			};
> @@ -163,7 +163,7 @@
>  				compatible = "fsl,imx27-cspi";
>  				reg = <0x1000f000 0x1000>;
>  				interrupts = <15>;
> -				clocks = <&clks 52>, <&clks 52>;
> +				clocks = <&clks 52>, <&clks 60>;
>  				clock-names = "ipg", "per";
>  				status = "disabled";
>  			};
> @@ -266,7 +266,7 @@
>  				compatible = "fsl,imx27-cspi";
>  				reg = <0x10017000 0x1000>;
>  				interrupts = <6>;
> -				clocks = <&clks 51>, <&clks 51>;
> +				clocks = <&clks 51>, <&clks 60>;
>  				clock-names = "ipg", "per";
>  				status = "disabled";
>  			};
> -- 
> 1.8.1.5
>
Sascha Hauer Aug. 20, 2013, 7:34 a.m. UTC | #3
On Fri, Aug 16, 2013 at 08:45:35AM +0200, Gwenhael Goavec-Merou wrote:
> CSPI PER clock is per2clk (per2_gate id 60) instead of cspiX_ipg_gate.
> 
> Signed-off-by: Gwenhael Goavec-Merou <gwenhael.goavec-merou@armadeus.com>

Acked-by: Sascha Hauer <s.hauer@pengutronix.de>

Sascha

> ---
>  arch/arm/boot/dts/imx27.dtsi | 6 +++---
>  1 file changed, 3 insertions(+), 3 deletions(-)
> 
> diff --git a/arch/arm/boot/dts/imx27.dtsi b/arch/arm/boot/dts/imx27.dtsi
> index 0695264..f71d233 100644
> --- a/arch/arm/boot/dts/imx27.dtsi
> +++ b/arch/arm/boot/dts/imx27.dtsi
> @@ -152,7 +152,7 @@
>  				compatible = "fsl,imx27-cspi";
>  				reg = <0x1000e000 0x1000>;
>  				interrupts = <16>;
> -				clocks = <&clks 53>, <&clks 53>;
> +				clocks = <&clks 53>, <&clks 60>;
>  				clock-names = "ipg", "per";
>  				status = "disabled";
>  			};
> @@ -163,7 +163,7 @@
>  				compatible = "fsl,imx27-cspi";
>  				reg = <0x1000f000 0x1000>;
>  				interrupts = <15>;
> -				clocks = <&clks 52>, <&clks 52>;
> +				clocks = <&clks 52>, <&clks 60>;
>  				clock-names = "ipg", "per";
>  				status = "disabled";
>  			};
> @@ -266,7 +266,7 @@
>  				compatible = "fsl,imx27-cspi";
>  				reg = <0x10017000 0x1000>;
>  				interrupts = <6>;
> -				clocks = <&clks 51>, <&clks 51>;
> +				clocks = <&clks 51>, <&clks 60>;
>  				clock-names = "ipg", "per";
>  				status = "disabled";
>  			};
> -- 
> 1.8.1.5
> 
> 
> _______________________________________________
> linux-arm-kernel mailing list
> linux-arm-kernel@lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
>
Gwenhael Goavec-Merou Aug. 27, 2013, 4:41 p.m. UTC | #4
ping?

Gwen

On Fri, 16 Aug 2013 08:45:35 +0200
Gwenhael Goavec-Merou <gwenhael.goavec-merou@armadeus.com> wrote:

> CSPI PER clock is per2clk (per2_gate id 60) instead of cspiX_ipg_gate.
> 
> Signed-off-by: Gwenhael Goavec-Merou <gwenhael.goavec-merou@armadeus.com>
> ---
>  arch/arm/boot/dts/imx27.dtsi | 6 +++---
>  1 file changed, 3 insertions(+), 3 deletions(-)
> 
> diff --git a/arch/arm/boot/dts/imx27.dtsi b/arch/arm/boot/dts/imx27.dtsi
> index 0695264..f71d233 100644
> --- a/arch/arm/boot/dts/imx27.dtsi
> +++ b/arch/arm/boot/dts/imx27.dtsi
> @@ -152,7 +152,7 @@
>  				compatible = "fsl,imx27-cspi";
>  				reg = <0x1000e000 0x1000>;
>  				interrupts = <16>;
> -				clocks = <&clks 53>, <&clks 53>;
> +				clocks = <&clks 53>, <&clks 60>;
>  				clock-names = "ipg", "per";
>  				status = "disabled";
>  			};
> @@ -163,7 +163,7 @@
>  				compatible = "fsl,imx27-cspi";
>  				reg = <0x1000f000 0x1000>;
>  				interrupts = <15>;
> -				clocks = <&clks 52>, <&clks 52>;
> +				clocks = <&clks 52>, <&clks 60>;
>  				clock-names = "ipg", "per";
>  				status = "disabled";
>  			};
> @@ -266,7 +266,7 @@
>  				compatible = "fsl,imx27-cspi";
>  				reg = <0x10017000 0x1000>;
>  				interrupts = <6>;
> -				clocks = <&clks 51>, <&clks 51>;
> +				clocks = <&clks 51>, <&clks 60>;
>  				clock-names = "ipg", "per";
>  				status = "disabled";
>  			};
> -- 
> 1.8.1.5
> 
> 
> _______________________________________________
> linux-arm-kernel mailing list
> linux-arm-kernel@lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
>
Shawn Guo Aug. 29, 2013, 5:12 a.m. UTC | #5
On Tue, Aug 27, 2013 at 06:41:24PM +0200, gwenhael.goavec wrote:
> ping?
> 

Applied.  Sorry for the delay.

Shawn
diff mbox

Patch

diff --git a/arch/arm/boot/dts/imx27.dtsi b/arch/arm/boot/dts/imx27.dtsi
index 0695264..f71d233 100644
--- a/arch/arm/boot/dts/imx27.dtsi
+++ b/arch/arm/boot/dts/imx27.dtsi
@@ -152,7 +152,7 @@ 
 				compatible = "fsl,imx27-cspi";
 				reg = <0x1000e000 0x1000>;
 				interrupts = <16>;
-				clocks = <&clks 53>, <&clks 53>;
+				clocks = <&clks 53>, <&clks 60>;
 				clock-names = "ipg", "per";
 				status = "disabled";
 			};
@@ -163,7 +163,7 @@ 
 				compatible = "fsl,imx27-cspi";
 				reg = <0x1000f000 0x1000>;
 				interrupts = <15>;
-				clocks = <&clks 52>, <&clks 52>;
+				clocks = <&clks 52>, <&clks 60>;
 				clock-names = "ipg", "per";
 				status = "disabled";
 			};
@@ -266,7 +266,7 @@ 
 				compatible = "fsl,imx27-cspi";
 				reg = <0x10017000 0x1000>;
 				interrupts = <6>;
-				clocks = <&clks 51>, <&clks 51>;
+				clocks = <&clks 51>, <&clks 60>;
 				clock-names = "ipg", "per";
 				status = "disabled";
 			};