[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <54C62739.6090902@metafoo.de>
Date: Mon, 26 Jan 2015 12:38:33 +0100
From: Lars-Peter Clausen <lars@...afoo.de>
To: Zubair Lutfullah Kakakhel <Zubair.Kakakhel@...tec.com>,
tiwai@...e.de, perex@...ex.cz
CC: devicetree@...r.kernel.org, alsa-devel@...a-project.org,
broonie@...nel.org, lgirdwood@...il.com,
linux-kernel@...r.kernel.org
Subject: Re: [alsa-devel] [PATCH 2/4] dt: sound: jz4740: Add binding documentation
for jz4740-i2s
On 01/26/2015 12:32 PM, Zubair Lutfullah Kakakhel wrote:
>
> On 26/01/15 10:37, Lars-Peter Clausen wrote:
>> On 01/26/2015 11:18 AM, Zubair Lutfullah Kakakhel wrote:
>> [...]
>>> +Required properties:
>>> +- compatible : "ingenic,jz4740-i2s"
>>> +- reg : I2S registers location and length
>>> +- clocks : AIC and I2S PLL clock specifiers.
>>> +- clock-names: "aic" and "i2s"
>>
>> We also need a handle to the DMA channels. Currently the request ids are hardcoded in the driver, but that needs to go away once the driver starts to support multiple SoCs with different request lines.
>
> I know. I've left DMA until we push the jz4780-dma driver upstream and see how things fit..
It doesn't hurt to put it in the spec. It will be two DMA channels one for
rx, one for tx. See for example
Documentation/devicetree/bindings/sound/bcm2835-i2s.txt
- Lars
--
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