Hello. On 29-08-2013 13:38, Jean-Francois Moine wrote:
diff --git a/arch/arm/boot/dts/dove.dtsi b/arch/arm/boot/dts/dove.dtsi index 499abad..78227e2 100644 --- a/arch/arm/boot/dts/dove.dtsi +++ b/arch/arm/boot/dts/dove.dtsi @@ -573,6 +573,24 @@ phy-handle = <ðphy>; }; }; + + i2s0: audio-controller@b0000 {
According to ePAPR [1] the node should be called "sound", not "audio-controller".
+ compatible = "marvell,mvebu-audio"; + reg = <0xb0000 0x2210>; + interrupts = <19>, <20>; + clocks = <&gate_clk 12>; + clock-names = "internal"; + status = "disabled"; + };
AFAIK, "sound" describes the global audio subsystem. For the Cubox, this will be done by something like:
sound { compatible = "simple-audio"; audio-controller = <&i2s1>; audio-codec = <&spdif>; codec-dai-name = "dit-hifi"; };
Well, then "sound-controller" sounds somewhat more appropriate. WBR, Sergei -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html