[PATCH v7] ARM: dts: mvebu: Introduce a new compatible string for mv64xxx-i2c

[PATCH v7] ARM: dts: mvebu: Introduce a new compatible string for mv64xxx-i2c

[Gregory CLEMENT]

The mv64xxx-i2c embedded in the Armada XP have a new feature to
offload i2c transaction. This new version of the IP come also with
some errata. This lead to the introduction to a another compatible
string.

This commit updates the Device Tree binding documentation accordingly.

Signed-off-by: Gregory CLEMENT <gregory.clement-wi1+55ScJUtKEb57/3fJTNBPR1lH4CV8@public.gmane.org>
---
Changelog v6 -> v7:

- only keep "marvell,mv7230-i2c" for compatible name

- replace "For a controller which support the I2C Transaction
  Generator" by "For the Armada XP" in the examples

 Documentation/devicetree/bindings/i2c/i2c-mv64xxx.txt | 10 ++++++++++
 1 file changed, 10 insertions(+)

diff --git a/Documentation/devicetree/bindings/i2c/i2c-mv64xxx.txt b/Documentation/devicetree/bindings/i2c/i2c-mv64xxx.txt
index 6113f92..73cdc03 100644
--- a/Documentation/devicetree/bindings/i2c/i2c-mv64xxx.txt
+++ b/Documentation/devicetree/bindings/i2c/i2c-mv64xxx.txt
@@ -5,6 +5,7 @@ Required properties :
 
  - reg             : Offset and length of the register set for the device
  - compatible      : Should be "marvell,mv64xxx-i2c" or "allwinner,sun4i-i2c"
+                     or "marvell,mv7230-i2c"
  - interrupts      : The interrupt number
 
 Optional properties :
@@ -20,3 +21,12 @@ Examples:
 		interrupts = <29>;
 		clock-frequency = <100000>;
 	};
+
+For the Armada XP:
+
+	i2c@11000 {
+		compatible = "marvell,mv78230-i2c", "marvell,mv64xxx-i2c";
+		reg = <0x11000 0x100>;
+		interrupts = <29>;
+		clock-frequency = <100000>;
+	};
-- 
1.8.1.2

Re: [PATCH v7] ARM: dts: mvebu: Introduce a new compatible string for mv64xxx-i2c

[Wolfram Sang]

[-- Attachment #1: Type: text/plain, Size: 502 bytes --]

On Fri, Aug 23, 2013 at 09:32:54AM +0200, Gregory CLEMENT wrote:
> The mv64xxx-i2c embedded in the Armada XP have a new feature to
> offload i2c transaction. This new version of the IP come also with
> some errata. This lead to the introduction to a another compatible
> string.
> 
> This commit updates the Device Tree binding documentation accordingly.
> 
> Signed-off-by: Gregory CLEMENT <gregory.clement-wi1+55ScJUtKEb57/3fJTNBPR1lH4CV8@public.gmane.org>

Applied to for-next, thanks!


[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 836 bytes --]

Re: [PATCH v7] ARM: dts: mvebu: Introduce a new compatible string for mv64xxx-i2c

[Mark Rutland]

On Fri, Aug 23, 2013 at 08:32:54AM +0100, Gregory CLEMENT wrote:
> The mv64xxx-i2c embedded in the Armada XP have a new feature to
> offload i2c transaction. This new version of the IP come also with
> some errata. This lead to the introduction to a another compatible
> string.
> 
> This commit updates the Device Tree binding documentation accordingly.
> 
> Signed-off-by: Gregory CLEMENT <gregory.clement-wi1+55ScJUtKEb57/3fJTNBPR1lH4CV8@public.gmane.org>
> ---
> Changelog v6 -> v7:
> 
> - only keep "marvell,mv7230-i2c" for compatible name
> 
> - replace "For a controller which support the I2C Transaction
>   Generator" by "For the Armada XP" in the examples
> 
>  Documentation/devicetree/bindings/i2c/i2c-mv64xxx.txt | 10 ++++++++++
>  1 file changed, 10 insertions(+)
> 
> diff --git a/Documentation/devicetree/bindings/i2c/i2c-mv64xxx.txt b/Documentation/devicetree/bindings/i2c/i2c-mv64xxx.txt
> index 6113f92..73cdc03 100644
> --- a/Documentation/devicetree/bindings/i2c/i2c-mv64xxx.txt
> +++ b/Documentation/devicetree/bindings/i2c/i2c-mv64xxx.txt
> @@ -5,6 +5,7 @@ Required properties :
>  
>   - reg             : Offset and length of the register set for the device
>   - compatible      : Should be "marvell,mv64xxx-i2c" or "allwinner,sun4i-i2c"
> +                     or "marvell,mv7230-i2c"

Ok, the new string looks sane, so I'm not opposed to that.

>   - interrupts      : The interrupt number
>  
>  Optional properties :
> @@ -20,3 +21,12 @@ Examples:
>  		interrupts = <29>;
>  		clock-frequency = <100000>;
>  	};
> +
> +For the Armada XP:
> +
> +	i2c@11000 {
> +		compatible = "marvell,mv78230-i2c", "marvell,mv64xxx-i2c";
                                         ^ That '8' shouldn't be there.

> +		reg = <0x11000 0x100>;
> +		interrupts = <29>;
> +		clock-frequency = <100000>;
> +	};

Thanks,
Mark.

Re: [PATCH v7] ARM: dts: mvebu: Introduce a new compatible string for mv64xxx-i2c

[Wolfram Sang]

[-- Attachment #1.1: Type: text/plain, Size: 227 bytes --]


> > +	i2c@11000 {
> > +		compatible = "marvell,mv78230-i2c", "marvell,mv64xxx-i2c";
>                                          ^ That '8' shouldn't be there.

Oops, well, precisely the '8' is missing above. Fixed it, thanks!


[-- Attachment #1.2: Digital signature --]
[-- Type: application/pgp-signature, Size: 836 bytes --]

[-- Attachment #2: Type: text/plain, Size: 176 bytes --]

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel