[PATCH v5 4/4] i2c: mv64xxx: Document the newly introduced Armada XP A0 compatible
Jason Cooper
jason at lakedaemon.net
Fri Jan 10 16:55:26 EST 2014
Gregory,
On Wed, Jan 08, 2014 at 04:06:29PM +0100, Gregory CLEMENT wrote:
> The first variants of Armada XP SoCs (A0 stepping) have issues related
> to the i2c controller which prevent to use the offload mechanism and
> ead to a kernel hang during boot.
I'll fixup s/ead/lead/ here.
>
> The commit introduces a new the compatible string
> marvell,mv78230-a0-i2c for the i2c controller.
>
> Signed-off-by: Gregory CLEMENT <gregory.clement at free-electrons.com>
> cc: devicetree at vger.kernel.org
> ---
> Documentation/devicetree/bindings/i2c/i2c-mv64xxx.txt | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/Documentation/devicetree/bindings/i2c/i2c-mv64xxx.txt b/Documentation/devicetree/bindings/i2c/i2c-mv64xxx.txt
> index 82e8f6f17179..9410ed72ec45 100644
> --- a/Documentation/devicetree/bindings/i2c/i2c-mv64xxx.txt
> +++ b/Documentation/devicetree/bindings/i2c/i2c-mv64xxx.txt
> @@ -5,7 +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,mv78230-i2c"
> + or "marvell,mv78230-i2c" or "marvell,mv78230-a0-i2c"
If it's ok with you Gregory, I'll amend this hunk as follows:
or "marvell,mv78230-i2c" or "marvell,mv78230-a0-i2c"
Only use "marvell,mv78230-a0-i2c" for a very
rare, initial version of the SoC which had
broken offload support. Linux auto-detects this
and sets it appropriately.
thx,
Jason.
> - interrupts : The interrupt number
>
> Optional properties :
> --
> 1.8.1.2
>
More information about the linux-arm-kernel
mailing list