Re: [PATCH V2] ASoC: SAMSUNG: Add DT support for i2s

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Thu, Sep 27, 2012 at 8:58 PM, Stephen Warren <swarren@xxxxxxxxxxxxx> wrote:
> On 09/27/2012 04:25 AM, Padmavathi Venna wrote:
>> Add support for device based discovery.
>
>> +++ b/Documentation/devicetree/bindings/sound/samsung-i2s.txt
>
>> +i2s@03830000 {
>> +     compatible = "samsung,samsung-i2s";
>> +     reg = <0x03830000 0x100>;
>
>> +     tx-dma-channel-secondary = <&pdma0 8>;
>> +     tx-dma-channel = <&pdma0 10>;
>> +     rx-dma-channel = <&pdma0 9>;
>> +     supports-6ch;
>> +     supports-rstclr;
>> +     supports-secdai;
>
> Those 6 properties are custom to this binding so should have a vendor
> prefix, e.g. "samsung,supports-6ch".
Okay. I will change this.
>
> Also, I wonder if this shouldn't wait for the forthcoming standardized
> DT DMA bindings (or perhaps document custom bindings that are identical
> to the new standardized bindings)? Otherwise, the DMA stuff will all
> have to be redefined again. Apparently it's hoped the bindings will be
> in 3.7, or if not there, in 3.8.
Okay. Once the generic DT DMA bindings are available I will use that.
But It will be good if atleast other things get reviewed.

Thanks
Padma
> --
> To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in
> the body of a message to majordomo@xxxxxxxxxxxxxxx
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
--
To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Index of Archives]     [Linux SoC Development]     [Linux Rockchip Development]     [Linux USB Development]     [Video for Linux]     [Linux Audio Users]     [Linux SCSI]     [Yosemite News]

  Powered by Linux