| Message ID | 20191127182127.1204-2-l.stach@pengutronix.de (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
| Series | [1/2] arm64: dts: imx8mq: add missing SAI nodes | expand |
Few comments inline about the naming: On Wed, 2019-11-27 at 19:21 +0100, Lucas Stach wrote: > This adds all the necessary device nodes to get audio support on both > the > RMB3 and Zest boards. > > Signed-off-by: Lucas Stach <l.stach@pengutronix.de> > --- > .../dts/freescale/imx8mq-zii-ultra-rmb3.dts | 87 ++++++++++++++++ > .../dts/freescale/imx8mq-zii-ultra-zest.dts | 30 ++++++ > .../boot/dts/freescale/imx8mq-zii-ultra.dtsi | 98 > +++++++++++++++++++ > 3 files changed, 215 insertions(+) > > diff --git a/arch/arm64/boot/dts/freescale/imx8mq-zii-ultra-rmb3.dts > b/arch/arm64/boot/dts/freescale/imx8mq-zii-ultra-rmb3.dts > index 6b3581366d67..d5e06885c683 100644 > --- a/arch/arm64/boot/dts/freescale/imx8mq-zii-ultra-rmb3.dts > +++ b/arch/arm64/boot/dts/freescale/imx8mq-zii-ultra-rmb3.dts > @@ -10,6 +10,56 @@ > / { > model = "ZII Ultra RMB3 Board"; > compatible = "zii,imx8mq-ultra-rmb3", "zii,imx8mq-ultra", > "fsl,imx8mq"; > + > + sound1 { I find sound1 / sound2 pretty generic. Perhaps we can go with something like sound-<codec_name> > + compatible = "simple-audio-card"; > + simple-audio-card,name = "RMB3 audio front"; > + simple-audio-card,format = "i2s"; > + simple-audio-card,bitclock-master = <&sound1_codec>; > + simple-audio-card,frame-master = <&sound1_codec>; > + simple-audio-card,widgets = > + "Headphone", "Headphone Jack Front"; > + simple-audio-card,routing = > + "Headphone Jack Front", "HPA1 HPLEFT", > + "Headphone Jack Front", "HPA1 HPRIGHT", > + "HPA1 LEFTIN", "HPL", > + "HPA1 RIGHTIN", "HPR"; > + simple-audio-card,aux-devs = <&hpa1>; > + > + sound1_cpu: simple-audio-card,cpu { > + sound-dai = <&sai2>; > + }; > + > + sound1_codec: simple-audio-card,codec { > + sound-dai = <&codec1>; Codec1 here is too generic. Perpahs we can add real code name. > + clocks = <&cs2000>; > + }; > + }; > + > + sound2 { > + compatible = "simple-audio-card"; > + simple-audio-card,name = "RMB3 audio back"; > + simple-audio-card,format = "i2s"; > + simple-audio-card,bitclock-master = <&sound2_codec>; > + simple-audio-card,frame-master = <&sound2_codec>; > + simple-audio-card,widgets = > + "Headphone", "Headphone Jack Back"; > + simple-audio-card,routing = > + "Headphone Jack Back", "HPA2 HPLEFT", > + "Headphone Jack Back", "HPA2 HPRIGHT", > + "HPA2 LEFTIN", "HPL", > + "HPA2 RIGHTIN", "HPR"; > + simple-audio-card,aux-devs = <&hpa2>; > + > + sound2_cpu: simple-audio-card,cpu { > + sound-dai = <&sai3>; > + }; > + > + sound2_codec: simple-audio-card,codec { > + sound-dai = <&codec2>; Ditto. > + clocks = <&cs2000>; > + }; > + }; > }; > > &ecspi1 { > @@ -27,6 +77,23 @@ > }; > }; > > +&i2c1 { > + codec2: codec@18 { Here we might go with something like: tlv320dac3100: codec@18 > + compatible = "ti,tlv320dac3100"; > + pinctrl-names = "default"; > + pinctrl-0 = <&pinctrl_codec2>; > + reg = <0x18>; > + #sound-dai-cells = <0>; > + HPVDD-supply = <®_3p3v>; > + SPRVDD-supply = <®_3p3v>; > + SPLVDD-supply = <®_3p3v>; > + AVDD-supply = <®_3p3v>; > + IOVDD-supply = <®_3p3v>; > + DVDD-supply = <&vgen4_reg>; > + gpio-reset = <&gpio3 4 GPIO_ACTIVE_HIGH>; > + }; > +}; > This is just a thought, naming is hard. But it should help with readability.
Hi Daniel, On Do, 2019-11-28 at 11:10 +0000, Daniel Baluta wrote: > Few comments inline about the naming: > > On Wed, 2019-11-27 at 19:21 +0100, Lucas Stach wrote: > > This adds all the necessary device nodes to get audio support on both > > the > > RMB3 and Zest boards. > > > > Signed-off-by: Lucas Stach <l.stach@pengutronix.de> > > --- > > .../dts/freescale/imx8mq-zii-ultra-rmb3.dts | 87 ++++++++++++++++ > > .../dts/freescale/imx8mq-zii-ultra-zest.dts | 30 ++++++ > > .../boot/dts/freescale/imx8mq-zii-ultra.dtsi | 98 > > +++++++++++++++++++ > > 3 files changed, 215 insertions(+) > > > > diff --git a/arch/arm64/boot/dts/freescale/imx8mq-zii-ultra-rmb3.dts > > b/arch/arm64/boot/dts/freescale/imx8mq-zii-ultra-rmb3.dts > > index 6b3581366d67..d5e06885c683 100644 > > --- a/arch/arm64/boot/dts/freescale/imx8mq-zii-ultra-rmb3.dts > > +++ b/arch/arm64/boot/dts/freescale/imx8mq-zii-ultra-rmb3.dts > > @@ -10,6 +10,56 @@ > > / { > > model = "ZII Ultra RMB3 Board"; > > compatible = "zii,imx8mq-ultra-rmb3", "zii,imx8mq-ultra", > > "fsl,imx8mq"; > > + > > + sound1 { > > I find sound1 / sound2 pretty generic. Perhaps we can go with something > like sound-<codec_name> > They are pretty generic, but I don't really see a reason to differentiate them more. Both sound paths are using a similar codec/hpa setup, the only difference is the SAI. If I name them after the SAI, will this really help much? > > > + compatible = "simple-audio-card"; > > + simple-audio-card,name = "RMB3 audio front"; > > + simple-audio-card,format = "i2s"; > > + simple-audio-card,bitclock-master = <&sound1_codec>; > > + simple-audio-card,frame-master = <&sound1_codec>; > > + simple-audio-card,widgets = > > + "Headphone", "Headphone Jack Front"; > > + simple-audio-card,routing = > > + "Headphone Jack Front", "HPA1 HPLEFT", > > + "Headphone Jack Front", "HPA1 HPRIGHT", > > + "HPA1 LEFTIN", "HPL", > > + "HPA1 RIGHTIN", "HPR"; > > + simple-audio-card,aux-devs = <&hpa1>; > > + > > + sound1_cpu: simple-audio-card,cpu { > > + sound-dai = <&sai2>; > > + }; > > + > > + sound1_codec: simple-audio-card,codec { > > + sound-dai = <&codec1>; > > Codec1 here is too generic. Perpahs we can add real code name. I agree that this might seem like thoughtless generic naming from the outside, but actually it lines up pretty well with the naming used in the schematics. I'm really reluctant to change the naming here, as any change will inevitably make it harder to match up the DTS with the schematics. Regards, Lucas > > + clocks = <&cs2000>; > > + }; > > + }; > > + > > + sound2 { > > + compatible = "simple-audio-card"; > > + simple-audio-card,name = "RMB3 audio back"; > > + simple-audio-card,format = "i2s"; > > + simple-audio-card,bitclock-master = <&sound2_codec>; > > + simple-audio-card,frame-master = <&sound2_codec>; > > + simple-audio-card,widgets = > > + "Headphone", "Headphone Jack Back"; > > + simple-audio-card,routing = > > + "Headphone Jack Back", "HPA2 HPLEFT", > > + "Headphone Jack Back", "HPA2 HPRIGHT", > > + "HPA2 LEFTIN", "HPL", > > + "HPA2 RIGHTIN", "HPR"; > > + simple-audio-card,aux-devs = <&hpa2>; > > + > > + sound2_cpu: simple-audio-card,cpu { > > + sound-dai = <&sai3>; > > + }; > > + > > + sound2_codec: simple-audio-card,codec { > > + sound-dai = <&codec2>; > Ditto. > > > + clocks = <&cs2000>; > > + }; > > + }; > > }; > > > > &ecspi1 { > > @@ -27,6 +77,23 @@ > > }; > > }; > > > > +&i2c1 { > > + codec2: codec@18 { > > Here we might go with something like: > > tlv320dac3100: codec@18 > > > > + compatible = "ti,tlv320dac3100"; > > + pinctrl-names = "default"; > > + pinctrl-0 = <&pinctrl_codec2>; > > + reg = <0x18>; > > + #sound-dai-cells = <0>; > > + HPVDD-supply = <®_3p3v>; > > + SPRVDD-supply = <®_3p3v>; > > + SPLVDD-supply = <®_3p3v>; > > + AVDD-supply = <®_3p3v>; > > + IOVDD-supply = <®_3p3v>; > > + DVDD-supply = <&vgen4_reg>; > > + gpio-reset = <&gpio3 4 GPIO_ACTIVE_HIGH>; > > + }; > > +}; > > > > This is just a thought, naming is hard. But it should help with > readability.
On Wed, Nov 27, 2019 at 07:21:27PM +0100, Lucas Stach wrote: > This adds all the necessary device nodes to get audio support on both the > RMB3 and Zest boards. > > Signed-off-by: Lucas Stach <l.stach@pengutronix.de> > --- > .../dts/freescale/imx8mq-zii-ultra-rmb3.dts | 87 ++++++++++++++++ > .../dts/freescale/imx8mq-zii-ultra-zest.dts | 30 ++++++ > .../boot/dts/freescale/imx8mq-zii-ultra.dtsi | 98 +++++++++++++++++++ > 3 files changed, 215 insertions(+) > > diff --git a/arch/arm64/boot/dts/freescale/imx8mq-zii-ultra-rmb3.dts b/arch/arm64/boot/dts/freescale/imx8mq-zii-ultra-rmb3.dts > index 6b3581366d67..d5e06885c683 100644 > --- a/arch/arm64/boot/dts/freescale/imx8mq-zii-ultra-rmb3.dts > +++ b/arch/arm64/boot/dts/freescale/imx8mq-zii-ultra-rmb3.dts > @@ -10,6 +10,56 @@ > / { > model = "ZII Ultra RMB3 Board"; > compatible = "zii,imx8mq-ultra-rmb3", "zii,imx8mq-ultra", "fsl,imx8mq"; > + > + sound1 { > + compatible = "simple-audio-card"; > + simple-audio-card,name = "RMB3 audio front"; > + simple-audio-card,format = "i2s"; > + simple-audio-card,bitclock-master = <&sound1_codec>; > + simple-audio-card,frame-master = <&sound1_codec>; > + simple-audio-card,widgets = > + "Headphone", "Headphone Jack Front"; > + simple-audio-card,routing = > + "Headphone Jack Front", "HPA1 HPLEFT", > + "Headphone Jack Front", "HPA1 HPRIGHT", > + "HPA1 LEFTIN", "HPL", > + "HPA1 RIGHTIN", "HPR"; > + simple-audio-card,aux-devs = <&hpa1>; > + > + sound1_cpu: simple-audio-card,cpu { > + sound-dai = <&sai2>; > + }; > + > + sound1_codec: simple-audio-card,codec { > + sound-dai = <&codec1>; > + clocks = <&cs2000>; > + }; > + }; > + > + sound2 { > + compatible = "simple-audio-card"; > + simple-audio-card,name = "RMB3 audio back"; > + simple-audio-card,format = "i2s"; > + simple-audio-card,bitclock-master = <&sound2_codec>; > + simple-audio-card,frame-master = <&sound2_codec>; > + simple-audio-card,widgets = > + "Headphone", "Headphone Jack Back"; > + simple-audio-card,routing = > + "Headphone Jack Back", "HPA2 HPLEFT", > + "Headphone Jack Back", "HPA2 HPRIGHT", > + "HPA2 LEFTIN", "HPL", > + "HPA2 RIGHTIN", "HPR"; > + simple-audio-card,aux-devs = <&hpa2>; > + > + sound2_cpu: simple-audio-card,cpu { > + sound-dai = <&sai3>; > + }; > + > + sound2_codec: simple-audio-card,codec { > + sound-dai = <&codec2>; > + clocks = <&cs2000>; > + }; > + }; > }; > > &ecspi1 { > @@ -27,6 +77,23 @@ > }; > }; > > +&i2c1 { > + codec2: codec@18 { > + compatible = "ti,tlv320dac3100"; > + pinctrl-names = "default"; > + pinctrl-0 = <&pinctrl_codec2>; > + reg = <0x18>; > + #sound-dai-cells = <0>; > + HPVDD-supply = <®_3p3v>; > + SPRVDD-supply = <®_3p3v>; > + SPLVDD-supply = <®_3p3v>; > + AVDD-supply = <®_3p3v>; > + IOVDD-supply = <®_3p3v>; > + DVDD-supply = <&vgen4_reg>; > + gpio-reset = <&gpio3 4 GPIO_ACTIVE_HIGH>; This property is deprecated. Shawn > + }; > +}; > + > &i2c2 { > temp-sense@48 { > compatible = "national,lm75"; > @@ -79,11 +146,23 @@ > }; > }; > > +&sai3 { > + pinctrl-names = "default"; > + pinctrl-0 = <&pinctrl_sai3>; > + status = "okay"; > +}; > + > &usbhub { > swap-dx-lanes = <0>; > }; > > &iomuxc { > + pinctrl_codec2: dac2grp { > + fsl,pins = < > + MX8MQ_IOMUXC_NAND_CE3_B_GPIO3_IO4 0x41 > + >; > + }; > + > pinctrl_ecspi1: ecspi1grp { > fsl,pins = < > MX8MQ_IOMUXC_ECSPI1_SS0_GPIO5_IO9 0x19 > @@ -92,4 +171,12 @@ > MX8MQ_IOMUXC_ECSPI1_MOSI_ECSPI1_MOSI 0x82 > >; > }; > + > + pinctrl_sai3: sai3grp { > + fsl,pins = < > + MX8MQ_IOMUXC_SAI3_TXFS_SAI3_TX_SYNC 0xd6 > + MX8MQ_IOMUXC_SAI3_TXC_SAI3_TX_BCLK 0xd6 > + MX8MQ_IOMUXC_SAI3_TXD_SAI3_TX_DATA0 0xd6 > + >; > + }; > }; > diff --git a/arch/arm64/boot/dts/freescale/imx8mq-zii-ultra-zest.dts b/arch/arm64/boot/dts/freescale/imx8mq-zii-ultra-zest.dts > index 173b9e9b2bbd..cf8cdc5935c6 100644 > --- a/arch/arm64/boot/dts/freescale/imx8mq-zii-ultra-zest.dts > +++ b/arch/arm64/boot/dts/freescale/imx8mq-zii-ultra-zest.dts > @@ -10,6 +10,36 @@ > / { > model = "ZII Ultra Zest Board"; > compatible = "zii,imx8mq-ultra-zest", "zii,imx8mq-ultra", "fsl,imx8mq"; > + > + sound { > + compatible = "simple-audio-card"; > + simple-audio-card,name = "Zest audio"; > + simple-audio-card,format = "i2s"; > + simple-audio-card,bitclock-master = <&sound_codec>; > + simple-audio-card,frame-master = <&sound_codec>; > + simple-audio-card,widgets = > + "Headphone", "Headphone Jack Front", > + "Headphone", "Headphone Jack Back"; > + simple-audio-card,routing = > + "Headphone Jack Front", "HPA1 HPLEFT", > + "Headphone Jack Front", "HPA1 HPRIGHT", > + "Headphone Jack Back", "HPA2 HPLEFT", > + "Headphone Jack Back", "HPA2 HPRIGHT", > + "HPA1 LEFTIN", "HPL", > + "HPA1 RIGHTIN", "HPR", > + "HPA2 LEFTIN", "HPL", > + "HPA2 RIGHTIN", "HPR"; > + simple-audio-card,aux-devs = <&hpa1>, <&hpa2>; > + > + sound_cpu: simple-audio-card,cpu { > + sound-dai = <&sai2>; > + }; > + > + sound_codec: simple-audio-card,codec { > + sound-dai = <&codec1>; > + clocks = <&cs2000>; > + }; > + }; > }; > > &i2c4 { > diff --git a/arch/arm64/boot/dts/freescale/imx8mq-zii-ultra.dtsi b/arch/arm64/boot/dts/freescale/imx8mq-zii-ultra.dtsi > index 32ce14936b01..c6a57dfe3ea9 100644 > --- a/arch/arm64/boot/dts/freescale/imx8mq-zii-ultra.dtsi > +++ b/arch/arm64/boot/dts/freescale/imx8mq-zii-ultra.dtsi > @@ -68,6 +68,15 @@ > regulator-always-on; > }; > > + reg_3p3v: regulator-3p3v { > + compatible = "regulator-fixed"; > + vin-supply = <®_3p3_main>; > + regulator-name = "GEN_3V3"; > + regulator-min-microvolt = <3300000>; > + regulator-max-microvolt = <3300000>; > + regulator-always-on; > + }; > + > reg_usdhc2_vmmc: regulator-vsd-3v3 { > pinctrl-names = "default"; > pinctrl-0 = <&pinctrl_reg_usdhc2>; > @@ -93,6 +102,18 @@ > 900000 0x0>; > regulator-always-on; > }; > + > + cs2000_ref: cs2000-ref { > + compatible = "fixed-clock"; > + #clock-cells = <0>; > + clock-frequency = <24576000>; > + }; > + > + cs2000_in_dummy: cs2000-in-dummy { > + compatible = "fixed-clock"; > + #clock-cells = <0>; > + clock-frequency = <0>; > + }; > }; > > &A53_0 { > @@ -263,6 +284,16 @@ > <18 IRQ_TYPE_EDGE_BOTH>; > interrupt-names = "a_det", "alert"; > }; > + > + hpa2: amp@60 { > + compatible = "ti,tpa6130a2"; > + pinctrl-names = "default"; > + pinctrl-0 = <&pinctrl_tpa2>; > + reg = <0x60>; > + power-gpio = <&gpio1 8 GPIO_ACTIVE_HIGH>; > + Vdd-supply = <®_5p0_main>; > + sound-name-prefix = "HPA2"; > + }; > }; > > &i2c2 { > @@ -355,11 +386,36 @@ > }; > }; > > + codec1: codec@18 { > + compatible = "ti,tlv320dac3100"; > + pinctrl-names = "default"; > + pinctrl-0 = <&pinctrl_codec1>; > + reg = <0x18>; > + #sound-dai-cells = <0>; > + HPVDD-supply = <®_3p3v>; > + SPRVDD-supply = <®_3p3v>; > + SPLVDD-supply = <®_3p3v>; > + AVDD-supply = <®_3p3v>; > + IOVDD-supply = <®_3p3v>; > + DVDD-supply = <&vgen4_reg>; > + reset-gpios = <&gpio3 3 GPIO_ACTIVE_LOW>; > + }; > + > eeprom@54 { > compatible = "atmel,24c128"; > reg = <0x54>; > }; > > + hpa1: amp@60 { > + compatible = "ti,tpa6130a2"; > + pinctrl-names = "default"; > + pinctrl-0 = <&pinctrl_tpa1>; > + reg = <0x60>; > + power-gpio = <&gpio4 10 GPIO_ACTIVE_HIGH>; > + Vdd-supply = <®_5p0_main>; > + sound-name-prefix = "HPA1"; > + }; > + > ds1341: rtc@68 { > compatible = "dallas,ds1341"; > reg = <0x68>; > @@ -379,6 +435,16 @@ > reg = <0x2c>; > reset-gpios = <&gpio3 25 GPIO_ACTIVE_LOW>; > }; > + > + cs2000: clkgen@4e { > + compatible = "cirrus,cs2000-cp"; > + reg = <0x4e>; > + #clock-cells = <0>; > + clock-names = "clk_in", "ref_clk"; > + clocks = <&cs2000_in_dummy>, <&cs2000_ref>; > + assigned-clocks = <&cs2000>; > + assigned-clock-rates = <24000000>; > + }; > }; > > &i2c4 { > @@ -388,6 +454,12 @@ > status = "okay"; > }; > > +&sai2 { > + pinctrl-names = "default"; > + pinctrl-0 = <&pinctrl_sai2>; > + status = "okay"; > +}; > + > &uart1 { > pinctrl-names = "default"; > pinctrl-0 = <&pinctrl_uart1>; > @@ -513,6 +585,12 @@ > }; > > &iomuxc { > + pinctrl_codec1: dac1grp { > + fsl,pins = < > + MX8MQ_IOMUXC_NAND_CE2_B_GPIO3_IO3 0x41 > + >; > + }; > + > pinctrl_fec1: fec1grp { > fsl,pins = < > MX8MQ_IOMUXC_ENET_MDC_ENET1_MDC 0x3 > @@ -604,12 +682,32 @@ > >; > }; > > + pinctrl_sai2: sai2grp { > + fsl,pins = < > + MX8MQ_IOMUXC_SAI2_TXFS_SAI2_TX_SYNC 0xd6 > + MX8MQ_IOMUXC_SAI2_TXC_SAI2_TX_BCLK 0xd6 > + MX8MQ_IOMUXC_SAI2_TXD0_SAI2_TX_DATA0 0xd6 > + >; > + }; > + > pinctrl_switch_irq: switchgrp { > fsl,pins = < > MX8MQ_IOMUXC_GPIO1_IO15_GPIO1_IO15 0x41 > >; > }; > > + pinctrl_tpa1: tpa6130-1grp { > + fsl,pins = < > + MX8MQ_IOMUXC_SAI1_TXFS_GPIO4_IO10 0x41 > + >; > + }; > + > + pinctrl_tpa2: tpa6130-2grp { > + fsl,pins = < > + MX8MQ_IOMUXC_GPIO1_IO08_GPIO1_IO8 0x41 > + >; > + }; > + > pinctrl_ts: tsgrp { > fsl,pins = < > MX8MQ_IOMUXC_GPIO1_IO11_GPIO1_IO11 0x96 > -- > 2.20.1 >
diff --git a/arch/arm64/boot/dts/freescale/imx8mq-zii-ultra-rmb3.dts b/arch/arm64/boot/dts/freescale/imx8mq-zii-ultra-rmb3.dts index 6b3581366d67..d5e06885c683 100644 --- a/arch/arm64/boot/dts/freescale/imx8mq-zii-ultra-rmb3.dts +++ b/arch/arm64/boot/dts/freescale/imx8mq-zii-ultra-rmb3.dts @@ -10,6 +10,56 @@ / { model = "ZII Ultra RMB3 Board"; compatible = "zii,imx8mq-ultra-rmb3", "zii,imx8mq-ultra", "fsl,imx8mq"; + + sound1 { + compatible = "simple-audio-card"; + simple-audio-card,name = "RMB3 audio front"; + simple-audio-card,format = "i2s"; + simple-audio-card,bitclock-master = <&sound1_codec>; + simple-audio-card,frame-master = <&sound1_codec>; + simple-audio-card,widgets = + "Headphone", "Headphone Jack Front"; + simple-audio-card,routing = + "Headphone Jack Front", "HPA1 HPLEFT", + "Headphone Jack Front", "HPA1 HPRIGHT", + "HPA1 LEFTIN", "HPL", + "HPA1 RIGHTIN", "HPR"; + simple-audio-card,aux-devs = <&hpa1>; + + sound1_cpu: simple-audio-card,cpu { + sound-dai = <&sai2>; + }; + + sound1_codec: simple-audio-card,codec { + sound-dai = <&codec1>; + clocks = <&cs2000>; + }; + }; + + sound2 { + compatible = "simple-audio-card"; + simple-audio-card,name = "RMB3 audio back"; + simple-audio-card,format = "i2s"; + simple-audio-card,bitclock-master = <&sound2_codec>; + simple-audio-card,frame-master = <&sound2_codec>; + simple-audio-card,widgets = + "Headphone", "Headphone Jack Back"; + simple-audio-card,routing = + "Headphone Jack Back", "HPA2 HPLEFT", + "Headphone Jack Back", "HPA2 HPRIGHT", + "HPA2 LEFTIN", "HPL", + "HPA2 RIGHTIN", "HPR"; + simple-audio-card,aux-devs = <&hpa2>; + + sound2_cpu: simple-audio-card,cpu { + sound-dai = <&sai3>; + }; + + sound2_codec: simple-audio-card,codec { + sound-dai = <&codec2>; + clocks = <&cs2000>; + }; + }; }; &ecspi1 { @@ -27,6 +77,23 @@ }; }; +&i2c1 { + codec2: codec@18 { + compatible = "ti,tlv320dac3100"; + pinctrl-names = "default"; + pinctrl-0 = <&pinctrl_codec2>; + reg = <0x18>; + #sound-dai-cells = <0>; + HPVDD-supply = <®_3p3v>; + SPRVDD-supply = <®_3p3v>; + SPLVDD-supply = <®_3p3v>; + AVDD-supply = <®_3p3v>; + IOVDD-supply = <®_3p3v>; + DVDD-supply = <&vgen4_reg>; + gpio-reset = <&gpio3 4 GPIO_ACTIVE_HIGH>; + }; +}; + &i2c2 { temp-sense@48 { compatible = "national,lm75"; @@ -79,11 +146,23 @@ }; }; +&sai3 { + pinctrl-names = "default"; + pinctrl-0 = <&pinctrl_sai3>; + status = "okay"; +}; + &usbhub { swap-dx-lanes = <0>; }; &iomuxc { + pinctrl_codec2: dac2grp { + fsl,pins = < + MX8MQ_IOMUXC_NAND_CE3_B_GPIO3_IO4 0x41 + >; + }; + pinctrl_ecspi1: ecspi1grp { fsl,pins = < MX8MQ_IOMUXC_ECSPI1_SS0_GPIO5_IO9 0x19 @@ -92,4 +171,12 @@ MX8MQ_IOMUXC_ECSPI1_MOSI_ECSPI1_MOSI 0x82 >; }; + + pinctrl_sai3: sai3grp { + fsl,pins = < + MX8MQ_IOMUXC_SAI3_TXFS_SAI3_TX_SYNC 0xd6 + MX8MQ_IOMUXC_SAI3_TXC_SAI3_TX_BCLK 0xd6 + MX8MQ_IOMUXC_SAI3_TXD_SAI3_TX_DATA0 0xd6 + >; + }; }; diff --git a/arch/arm64/boot/dts/freescale/imx8mq-zii-ultra-zest.dts b/arch/arm64/boot/dts/freescale/imx8mq-zii-ultra-zest.dts index 173b9e9b2bbd..cf8cdc5935c6 100644 --- a/arch/arm64/boot/dts/freescale/imx8mq-zii-ultra-zest.dts +++ b/arch/arm64/boot/dts/freescale/imx8mq-zii-ultra-zest.dts @@ -10,6 +10,36 @@ / { model = "ZII Ultra Zest Board"; compatible = "zii,imx8mq-ultra-zest", "zii,imx8mq-ultra", "fsl,imx8mq"; + + sound { + compatible = "simple-audio-card"; + simple-audio-card,name = "Zest audio"; + simple-audio-card,format = "i2s"; + simple-audio-card,bitclock-master = <&sound_codec>; + simple-audio-card,frame-master = <&sound_codec>; + simple-audio-card,widgets = + "Headphone", "Headphone Jack Front", + "Headphone", "Headphone Jack Back"; + simple-audio-card,routing = + "Headphone Jack Front", "HPA1 HPLEFT", + "Headphone Jack Front", "HPA1 HPRIGHT", + "Headphone Jack Back", "HPA2 HPLEFT", + "Headphone Jack Back", "HPA2 HPRIGHT", + "HPA1 LEFTIN", "HPL", + "HPA1 RIGHTIN", "HPR", + "HPA2 LEFTIN", "HPL", + "HPA2 RIGHTIN", "HPR"; + simple-audio-card,aux-devs = <&hpa1>, <&hpa2>; + + sound_cpu: simple-audio-card,cpu { + sound-dai = <&sai2>; + }; + + sound_codec: simple-audio-card,codec { + sound-dai = <&codec1>; + clocks = <&cs2000>; + }; + }; }; &i2c4 { diff --git a/arch/arm64/boot/dts/freescale/imx8mq-zii-ultra.dtsi b/arch/arm64/boot/dts/freescale/imx8mq-zii-ultra.dtsi index 32ce14936b01..c6a57dfe3ea9 100644 --- a/arch/arm64/boot/dts/freescale/imx8mq-zii-ultra.dtsi +++ b/arch/arm64/boot/dts/freescale/imx8mq-zii-ultra.dtsi @@ -68,6 +68,15 @@ regulator-always-on; }; + reg_3p3v: regulator-3p3v { + compatible = "regulator-fixed"; + vin-supply = <®_3p3_main>; + regulator-name = "GEN_3V3"; + regulator-min-microvolt = <3300000>; + regulator-max-microvolt = <3300000>; + regulator-always-on; + }; + reg_usdhc2_vmmc: regulator-vsd-3v3 { pinctrl-names = "default"; pinctrl-0 = <&pinctrl_reg_usdhc2>; @@ -93,6 +102,18 @@ 900000 0x0>; regulator-always-on; }; + + cs2000_ref: cs2000-ref { + compatible = "fixed-clock"; + #clock-cells = <0>; + clock-frequency = <24576000>; + }; + + cs2000_in_dummy: cs2000-in-dummy { + compatible = "fixed-clock"; + #clock-cells = <0>; + clock-frequency = <0>; + }; }; &A53_0 { @@ -263,6 +284,16 @@ <18 IRQ_TYPE_EDGE_BOTH>; interrupt-names = "a_det", "alert"; }; + + hpa2: amp@60 { + compatible = "ti,tpa6130a2"; + pinctrl-names = "default"; + pinctrl-0 = <&pinctrl_tpa2>; + reg = <0x60>; + power-gpio = <&gpio1 8 GPIO_ACTIVE_HIGH>; + Vdd-supply = <®_5p0_main>; + sound-name-prefix = "HPA2"; + }; }; &i2c2 { @@ -355,11 +386,36 @@ }; }; + codec1: codec@18 { + compatible = "ti,tlv320dac3100"; + pinctrl-names = "default"; + pinctrl-0 = <&pinctrl_codec1>; + reg = <0x18>; + #sound-dai-cells = <0>; + HPVDD-supply = <®_3p3v>; + SPRVDD-supply = <®_3p3v>; + SPLVDD-supply = <®_3p3v>; + AVDD-supply = <®_3p3v>; + IOVDD-supply = <®_3p3v>; + DVDD-supply = <&vgen4_reg>; + reset-gpios = <&gpio3 3 GPIO_ACTIVE_LOW>; + }; + eeprom@54 { compatible = "atmel,24c128"; reg = <0x54>; }; + hpa1: amp@60 { + compatible = "ti,tpa6130a2"; + pinctrl-names = "default"; + pinctrl-0 = <&pinctrl_tpa1>; + reg = <0x60>; + power-gpio = <&gpio4 10 GPIO_ACTIVE_HIGH>; + Vdd-supply = <®_5p0_main>; + sound-name-prefix = "HPA1"; + }; + ds1341: rtc@68 { compatible = "dallas,ds1341"; reg = <0x68>; @@ -379,6 +435,16 @@ reg = <0x2c>; reset-gpios = <&gpio3 25 GPIO_ACTIVE_LOW>; }; + + cs2000: clkgen@4e { + compatible = "cirrus,cs2000-cp"; + reg = <0x4e>; + #clock-cells = <0>; + clock-names = "clk_in", "ref_clk"; + clocks = <&cs2000_in_dummy>, <&cs2000_ref>; + assigned-clocks = <&cs2000>; + assigned-clock-rates = <24000000>; + }; }; &i2c4 { @@ -388,6 +454,12 @@ status = "okay"; }; +&sai2 { + pinctrl-names = "default"; + pinctrl-0 = <&pinctrl_sai2>; + status = "okay"; +}; + &uart1 { pinctrl-names = "default"; pinctrl-0 = <&pinctrl_uart1>; @@ -513,6 +585,12 @@ }; &iomuxc { + pinctrl_codec1: dac1grp { + fsl,pins = < + MX8MQ_IOMUXC_NAND_CE2_B_GPIO3_IO3 0x41 + >; + }; + pinctrl_fec1: fec1grp { fsl,pins = < MX8MQ_IOMUXC_ENET_MDC_ENET1_MDC 0x3 @@ -604,12 +682,32 @@ >; }; + pinctrl_sai2: sai2grp { + fsl,pins = < + MX8MQ_IOMUXC_SAI2_TXFS_SAI2_TX_SYNC 0xd6 + MX8MQ_IOMUXC_SAI2_TXC_SAI2_TX_BCLK 0xd6 + MX8MQ_IOMUXC_SAI2_TXD0_SAI2_TX_DATA0 0xd6 + >; + }; + pinctrl_switch_irq: switchgrp { fsl,pins = < MX8MQ_IOMUXC_GPIO1_IO15_GPIO1_IO15 0x41 >; }; + pinctrl_tpa1: tpa6130-1grp { + fsl,pins = < + MX8MQ_IOMUXC_SAI1_TXFS_GPIO4_IO10 0x41 + >; + }; + + pinctrl_tpa2: tpa6130-2grp { + fsl,pins = < + MX8MQ_IOMUXC_GPIO1_IO08_GPIO1_IO8 0x41 + >; + }; + pinctrl_ts: tsgrp { fsl,pins = < MX8MQ_IOMUXC_GPIO1_IO11_GPIO1_IO11 0x96
This adds all the necessary device nodes to get audio support on both the RMB3 and Zest boards. Signed-off-by: Lucas Stach <l.stach@pengutronix.de> --- .../dts/freescale/imx8mq-zii-ultra-rmb3.dts | 87 ++++++++++++++++ .../dts/freescale/imx8mq-zii-ultra-zest.dts | 30 ++++++ .../boot/dts/freescale/imx8mq-zii-ultra.dtsi | 98 +++++++++++++++++++ 3 files changed, 215 insertions(+)