Re: [PATCH v5 3/6] i2c: at91: update documentation for DT bindings

[Nicolas Ferre <nicolas.ferre@atmel.com>]

Le 03/06/2015 18:25, Cyrille Pitchen a écrit :
> add a new value "atmel,at91sama5d2-i2c" for the "compatible" property.

Well, here I would like "atmel,sama5d2-i2c" like any other "SAMA5"
product compatibility string out there...

Aka NAK for me ;-)


> add a new optional property "atmel,fifo-size" to enable FIFO support when
> available.
> add missing optional properties "dmas" and "dma-names".
> 
> Signed-off-by: Cyrille Pitchen <cyrille.pitchen@atmel.com>
> ---
>  Documentation/devicetree/bindings/i2c/i2c-at91.txt | 29 ++++++++++++++++++++--
>  1 file changed, 27 insertions(+), 2 deletions(-)
> 
> diff --git a/Documentation/devicetree/bindings/i2c/i2c-at91.txt b/Documentation/devicetree/bindings/i2c/i2c-at91.txt
> index 388f0a2..7c04fd9 100644
> --- a/Documentation/devicetree/bindings/i2c/i2c-at91.txt
> +++ b/Documentation/devicetree/bindings/i2c/i2c-at91.txt
> @@ -2,8 +2,8 @@ I2C for Atmel platforms
>  
>  Required properties :
>  - compatible : Must be "atmel,at91rm9200-i2c", "atmel,at91sam9261-i2c",
> -     "atmel,at91sam9260-i2c", "atmel,at91sam9g20-i2c", "atmel,at91sam9g10-i2c"
> -     or "atmel,at91sam9x5-i2c"
> +     "atmel,at91sam9260-i2c", "atmel,at91sam9g20-i2c", "atmel,at91sam9g10-i2c",
> +     "atmel,at91sam9x5-i2c" or "atmel,at91sama5d2-i2c"

Ditto

>  - reg: physical base address of the controller and length of memory mapped
>       region.
>  - interrupts: interrupt number to the cpu.
> @@ -13,6 +13,9 @@ Required properties :
>  
>  Optional properties:
>  - clock-frequency: Desired I2C bus frequency in Hz, otherwise defaults to 100000
> +- dmas: A list of two dma specifiers, one for each entry in dma-names.
> +- dma-names: should contain "tx" and "rx".
> +- atmel,fifo-size: size of the RX and TX FIFOs, if available.
>  - Child nodes conforming to i2c bus binding
>  
>  Examples :
> @@ -32,3 +35,25 @@ i2c0: i2c@fff84000 {
>  		pagesize = <128>;
>  	}
>  }
> +
> +i2c0: i2c@f8034600 {
> +	compatible = "atmel,at91sama5d2-i2c";

Ditto

> +	reg = <0xf8034600 0x100>;
> +	interrupts = <19 IRQ_TYPE_LEVEL_HIGH 7>;
> +	dmas = <&dma0
> +		(AT91_XDMAC_DT_MEM_IF(0) | AT91_XDMAC_DT_PER_IF(1))
> +		AT91_XDMAC_DT_PERID(11)>,
> +	       <&dma0
> +		(AT91_XDMAC_DT_MEM_IF(0) | AT91_XDMAC_DT_PER_IF(1))
> +		AT91_XDMAC_DT_PERID(12)>;
> +	dma-names = "tx", "rx";
> +	#address-cells = <1>;
> +	#size-cells = <0>;
> +	clocks = <&flx0>;
> +	atmel,fifo-size = <32>;
> +
> +	wm8731: wm8731@1a {
> +		compatible = "wm8731";
> +		reg = <0x1a>;
> +	};
> +};
> 


-- 
Nicolas Ferre