[PATCH v2] ARM64: dts: marvell: armada-cp110: Add registers clock for I2C nodes

[PATCH v2] ARM64: dts: marvell: armada-cp110: Add registers clock for I2C nodes

[Gregory CLEMENT]

This extra clock is needed to access the registers of the I2C controller
used on the Armada 7K/8K SoCs.

This follows the changes already made in the binding documentation (as
well as in the driver) in:
commit 1534156e999735fe0befad958e1447600c0c20e7 ("i2c: mv64xxx: Fix clock
resource by adding an optional bus clock")

Signed-off-by: Gregory CLEMENT <gregory.clement@free-electrons.com>
---
Changelog:

v1 -> v2
Add comment about the commit modifying the driver in the commit log


 arch/arm64/boot/dts/marvell/armada-cp110.dtsi | 8 ++++++--
 1 file changed, 6 insertions(+), 2 deletions(-)

diff --git a/arch/arm64/boot/dts/marvell/armada-cp110.dtsi b/arch/arm64/boot/dts/marvell/armada-cp110.dtsi
index 0ab921861a2f..c57bf6661622 100644
--- a/arch/arm64/boot/dts/marvell/armada-cp110.dtsi
+++ b/arch/arm64/boot/dts/marvell/armada-cp110.dtsi
@@ -280,7 +280,9 @@
 			#address-cells = <1>;
 			#size-cells = <0>;
 			interrupts = <ICU_GRP_NSR 120 IRQ_TYPE_LEVEL_HIGH>;
-			clocks = <&CP110_LABEL(clk) 1 21>;
+			clock-names = "core", "reg";
+			clocks = <&CP110_LABEL(clk) 1 21>,
+				 <&CP110_LABEL(clk) 1 17>;
 			status = "disabled";
 		};
 
@@ -290,7 +292,9 @@
 			#address-cells = <1>;
 			#size-cells = <0>;
 			interrupts = <ICU_GRP_NSR 121 IRQ_TYPE_LEVEL_HIGH>;
-			clocks = <&CP110_LABEL(clk) 1 21>;
+			clock-names = "core", "reg";
+			clocks = <&CP110_LABEL(clk) 1 21>,
+				 <&CP110_LABEL(clk) 1 17>;
 			status = "disabled";
 		};
 
-- 
2.15.1

[PATCH v2] ARM64: dts: marvell: armada-cp110: Add registers clock for I2C nodes

[Gregory CLEMENT]

Hi,
 
 On mar., janv. 30 2018, Gregory CLEMENT <gregory.clement@free-electrons.com> wrote:

> This extra clock is needed to access the registers of the I2C controller
> used on the Armada 7K/8K SoCs.
>
> This follows the changes already made in the binding documentation (as
> well as in the driver) in:
> commit 1534156e999735fe0befad958e1447600c0c20e7 ("i2c: mv64xxx: Fix clock
> resource by adding an optional bus clock")
>
> Signed-off-by: Gregory CLEMENT <gregory.clement@free-electrons.com>

Applied on mvebu/dt64 (with the email address modified to @bootlin.com)

Gregory

> ---
> Changelog:
>
> v1 -> v2
> Add comment about the commit modifying the driver in the commit log
>
>
>  arch/arm64/boot/dts/marvell/armada-cp110.dtsi | 8 ++++++--
>  1 file changed, 6 insertions(+), 2 deletions(-)
>
> diff --git a/arch/arm64/boot/dts/marvell/armada-cp110.dtsi b/arch/arm64/boot/dts/marvell/armada-cp110.dtsi
> index 0ab921861a2f..c57bf6661622 100644
> --- a/arch/arm64/boot/dts/marvell/armada-cp110.dtsi
> +++ b/arch/arm64/boot/dts/marvell/armada-cp110.dtsi
> @@ -280,7 +280,9 @@
>  			#address-cells = <1>;
>  			#size-cells = <0>;
>  			interrupts = <ICU_GRP_NSR 120 IRQ_TYPE_LEVEL_HIGH>;
> -			clocks = <&CP110_LABEL(clk) 1 21>;
> +			clock-names = "core", "reg";
> +			clocks = <&CP110_LABEL(clk) 1 21>,
> +				 <&CP110_LABEL(clk) 1 17>;
>  			status = "disabled";
>  		};
>  
> @@ -290,7 +292,9 @@
>  			#address-cells = <1>;
>  			#size-cells = <0>;
>  			interrupts = <ICU_GRP_NSR 121 IRQ_TYPE_LEVEL_HIGH>;
> -			clocks = <&CP110_LABEL(clk) 1 21>;
> +			clock-names = "core", "reg";
> +			clocks = <&CP110_LABEL(clk) 1 21>,
> +				 <&CP110_LABEL(clk) 1 17>;
>  			status = "disabled";
>  		};
>  
> -- 
> 2.15.1
>
>
> _______________________________________________
> linux-arm-kernel mailing list
> linux-arm-kernel at lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

-- 
Gregory Clement, Bootlin (formerly Free Electrons)
Embedded Linux and Kernel engineering
http://bootlin.com