[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <51DC0E26.7080508@atmel.com>
Date: Tue, 9 Jul 2013 15:20:38 +0200
From: Nicolas Ferre <nicolas.ferre@...el.com>
To: Richard Genoud <richard.genoud@...il.com>
CC: Mark Brown <broonie@...nel.org>,
Liam Girdwood <lgirdwood@...il.com>,
"Bo Shen" <voice.shen@...el.com>,
Lars-Peter Clausen <lars@...afoo.de>,
<linux-arm-kernel@...ts.infradead.org>,
<linux-kernel@...r.kernel.org>, <alsa-devel@...a-project.org>,
<devicetree-discuss@...ts.ozlabs.org>
Subject: Re: [PATCH v3 3/7] Documentation: DT: update atmel SSC with DMA binding
On 09/07/2013 15:19, Richard Genoud :
> 2013/7/9 Richard Genoud <richard.genoud@...il.com>:
>> As atmel-ssc can be used with DMA, the documentation should be updated.
>> Also, a configuration DMA example is given.
>>
>> Signed-off-by: Richard Genoud <richard.genoud@...il.com>
>> ---
>> .../devicetree/bindings/misc/atmel-ssc.txt | 24 ++++++++++++++++++--
>> 1 file changed, 22 insertions(+), 2 deletions(-)
>>
>> diff --git a/Documentation/devicetree/bindings/misc/atmel-ssc.txt b/Documentation/devicetree/bindings/misc/atmel-ssc.txt
>> index 38e51ad..96958ca 100644
>> --- a/Documentation/devicetree/bindings/misc/atmel-ssc.txt
>> +++ b/Documentation/devicetree/bindings/misc/atmel-ssc.txt
>> @@ -6,10 +6,30 @@ Required properties:
>> - atmel,at91sam9g45-ssc: support dma transfer
>> - reg: Should contain SSC registers location and length
>> - interrupts: Should contain SSC interrupt
>> +For dma transfer:
>> +- dmas: DMA specifier, consisting of a phandle to DMA controller node,
>> + the memory interface and SSC DMA channel ID (for tx and rx).
>> + See Documentation/devicetree/bindings/dma/atmel-dma.txt for details.
>> +- dma-names: Must be "tx", "rx".
>>
>> -Example:
>> +
>> +Examples:
>> ssc0: ssc@...bc000 {
>> compatible = "atmel,at91rm9200-ssc";
>> reg = <0xfffbc000 0x4000>;
>> - interrupts = <14 4 5>;
>> + interrupts = <14 IRQ_TYPE_LEVEL_HIGH 5>;
> maybe I should not use macro in the documentation (not sure if I should or not).
absolutely.
>> };
>> +
>> +
>> +ssc0: ssc@...10000 {
>> + compatible = "atmel,at91sam9g45-ssc";
>> + reg = <0xf0010000 0x4000>;
>> + interrupts = <28 IRQ_TYPE_LEVEL_HIGH 5>;
>> + dmas = <&dma0 1 AT91_DMA_CFG_PER_ID(13)>,
>> + <&dma0 1 AT91_DMA_CFG_PER_ID(14)>;
> same here
>> + dma-names = "tx", "rx";
>> + pinctrl-names = "default";
>> + pinctrl-0 = <&pinctrl_ssc0_tx &pinctrl_ssc0_rx>;
>> + status = "disabled";
>> +};
>> +
>> --
>> 1.7.10.4
>>
>
>
>
--
Nicolas Ferre
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists