[PATCH 2/4] ARM: mvebu: add audio I2S controller to Armada 38x Device Tree
From: sebastian.hesselbarth@gmail.com (Sebastian Hesselbarth)
Date: 2015-02-25 22:21:10
Also in:
alsa-devel
On 25.02.2015 22:58, Marcin Wojtas wrote:
quoted hunk ↗ jump to hunk
This commit adds the description of the I2S controller to the Marvell Armada 38x SoC's Device Tree, as well as its pin configuration. Signed-off-by: Marcin Wojtas <redacted> --- arch/arm/boot/dts/armada-38x.dtsi | 17 +++++++++++++++++ 1 file changed, 17 insertions(+)diff --git a/arch/arm/boot/dts/armada-38x.dtsi b/arch/arm/boot/dts/armada-38x.dtsi index 1dff30a..fa21cb5 100644 --- a/arch/arm/boot/dts/armada-38x.dtsi +++ b/arch/arm/boot/dts/armada-38x.dtsi
[...]
quoted hunk ↗ jump to hunk
@@ -555,6 +561,17 @@ status = "disabled"; }; + audio_controller: audio-controller at e8000 { + #sound-dai-cells = <1>; + compatible = "marvell,armada-380-audio"; + reg = <0xe8000 0x4000>, <0x18410 0xc>, <0x18204 0x4>; + reg-names = "i2s_regs", "pll_regs", "soc_ctrl";
Marcin, sorry but NACK. The PLL and SoC ctrl are not even close to the audio controller registers.
+ interrupts = <GIC_SPI 75 IRQ_TYPE_LEVEL_HIGH>; + clocks = <&gateclk 0>; + clock-names = "internal";
How about providing access to audio PLL by a clock driver and amend the binding to allow for a second more precise PLL clock, e.g. clocks = <&gateclk CLK_AUDIO>, <&pll PLL_AUDIO>; clock-names = "internal", "pll"; we already check for an "extclk" on Dove for the same reason but the name might be misleading here. Also, i2c/spdif muxing option could be handled by 38x's pinctrl driver, we have the same for Dove's internal i2c mux. If you want to use i2s you just add the option to the default pinctrl hog: pinctrl-0 = <&i2s_pins &audio_mux_i2s>; pinctrl-names = "default"; Sebastian
+ status = "disabled";
+ };
+
usb3 at f0000 {
compatible = "marvell,armada-380-xhci";
reg = <0xf0000 0x4000>,<0xf4000 0x4000>;