Re: [PATCH v4 6/6] arm: dts: mediatek: modify audio related nodes for both MT2701 and MT7623

From: Matthias Brugger
Date: Tue Apr 17 2018 - 10:26:32 EST




On 03/06/2018 10:09 AM, Ryder Lee wrote:
> Modify audio related nodes to reflect the actual usage in binding documents.
>
> Signed-off-by: Ryder Lee <ryder.lee@xxxxxxxxxxxx>
> ---

applied to v4.17-next/dts32

Thanks!

> arch/arm/boot/dts/mt2701.dtsi | 188 ++++++++++++++++++++---------------------
> arch/arm/boot/dts/mt7623.dtsi | 190 ++++++++++++++++++++----------------------
> 2 files changed, 182 insertions(+), 196 deletions(-)
>
> diff --git a/arch/arm/boot/dts/mt2701.dtsi b/arch/arm/boot/dts/mt2701.dtsi
> index 05557fc..05cf65c 100644
> --- a/arch/arm/boot/dts/mt2701.dtsi
> +++ b/arch/arm/boot/dts/mt2701.dtsi
> @@ -426,104 +426,96 @@
> status = "disabled";
> };
>
> - afe: audio-controller@11220000 {
> - compatible = "mediatek,mt2701-audio";
> - reg = <0 0x11220000 0 0x2000>,
> - <0 0x112a0000 0 0x20000>;
> - interrupts = <GIC_SPI 104 IRQ_TYPE_LEVEL_LOW>,
> - <GIC_SPI 132 IRQ_TYPE_LEVEL_LOW>;
> - interrupt-names = "afe", "asys";
> - power-domains = <&scpsys MT2701_POWER_DOMAIN_IFR_MSC>;
> -
> - clocks = <&infracfg CLK_INFRA_AUDIO>,
> - <&topckgen CLK_TOP_AUD_MUX1_SEL>,
> - <&topckgen CLK_TOP_AUD_MUX2_SEL>,
> - <&topckgen CLK_TOP_AUD_MUX1_DIV>,
> - <&topckgen CLK_TOP_AUD_MUX2_DIV>,
> - <&topckgen CLK_TOP_AUD_48K_TIMING>,
> - <&topckgen CLK_TOP_AUD_44K_TIMING>,
> - <&topckgen CLK_TOP_AUDPLL_MUX_SEL>,
> - <&topckgen CLK_TOP_APLL_SEL>,
> - <&topckgen CLK_TOP_AUD1PLL_98M>,
> - <&topckgen CLK_TOP_AUD2PLL_90M>,
> - <&topckgen CLK_TOP_HADDS2PLL_98M>,
> - <&topckgen CLK_TOP_HADDS2PLL_294M>,
> - <&topckgen CLK_TOP_AUDPLL>,
> - <&topckgen CLK_TOP_AUDPLL_D4>,
> - <&topckgen CLK_TOP_AUDPLL_D8>,
> - <&topckgen CLK_TOP_AUDPLL_D16>,
> - <&topckgen CLK_TOP_AUDPLL_D24>,
> - <&topckgen CLK_TOP_AUDINTBUS_SEL>,
> - <&clk26m>,
> - <&topckgen CLK_TOP_SYSPLL1_D4>,
> - <&topckgen CLK_TOP_AUD_K1_SRC_SEL>,
> - <&topckgen CLK_TOP_AUD_K2_SRC_SEL>,
> - <&topckgen CLK_TOP_AUD_K3_SRC_SEL>,
> - <&topckgen CLK_TOP_AUD_K4_SRC_SEL>,
> - <&topckgen CLK_TOP_AUD_K5_SRC_SEL>,
> - <&topckgen CLK_TOP_AUD_K6_SRC_SEL>,
> - <&topckgen CLK_TOP_AUD_K1_SRC_DIV>,
> - <&topckgen CLK_TOP_AUD_K2_SRC_DIV>,
> - <&topckgen CLK_TOP_AUD_K3_SRC_DIV>,
> - <&topckgen CLK_TOP_AUD_K4_SRC_DIV>,
> - <&topckgen CLK_TOP_AUD_K5_SRC_DIV>,
> - <&topckgen CLK_TOP_AUD_K6_SRC_DIV>,
> - <&topckgen CLK_TOP_AUD_I2S1_MCLK>,
> - <&topckgen CLK_TOP_AUD_I2S2_MCLK>,
> - <&topckgen CLK_TOP_AUD_I2S3_MCLK>,
> - <&topckgen CLK_TOP_AUD_I2S4_MCLK>,
> - <&topckgen CLK_TOP_AUD_I2S5_MCLK>,
> - <&topckgen CLK_TOP_AUD_I2S6_MCLK>,
> - <&topckgen CLK_TOP_ASM_M_SEL>,
> - <&topckgen CLK_TOP_ASM_H_SEL>,
> - <&topckgen CLK_TOP_UNIVPLL2_D4>,
> - <&topckgen CLK_TOP_UNIVPLL2_D2>,
> - <&topckgen CLK_TOP_SYSPLL_D5>;
> -
> - clock-names = "infra_sys_audio_clk",
> - "top_audio_mux1_sel",
> - "top_audio_mux2_sel",
> - "top_audio_mux1_div",
> - "top_audio_mux2_div",
> - "top_audio_48k_timing",
> - "top_audio_44k_timing",
> - "top_audpll_mux_sel",
> - "top_apll_sel",
> - "top_aud1_pll_98M",
> - "top_aud2_pll_90M",
> - "top_hadds2_pll_98M",
> - "top_hadds2_pll_294M",
> - "top_audpll",
> - "top_audpll_d4",
> - "top_audpll_d8",
> - "top_audpll_d16",
> - "top_audpll_d24",
> - "top_audintbus_sel",
> - "clk_26m",
> - "top_syspll1_d4",
> - "top_aud_k1_src_sel",
> - "top_aud_k2_src_sel",
> - "top_aud_k3_src_sel",
> - "top_aud_k4_src_sel",
> - "top_aud_k5_src_sel",
> - "top_aud_k6_src_sel",
> - "top_aud_k1_src_div",
> - "top_aud_k2_src_div",
> - "top_aud_k3_src_div",
> - "top_aud_k4_src_div",
> - "top_aud_k5_src_div",
> - "top_aud_k6_src_div",
> - "top_aud_i2s1_mclk",
> - "top_aud_i2s2_mclk",
> - "top_aud_i2s3_mclk",
> - "top_aud_i2s4_mclk",
> - "top_aud_i2s5_mclk",
> - "top_aud_i2s6_mclk",
> - "top_asm_m_sel",
> - "top_asm_h_sel",
> - "top_univpll2_d4",
> - "top_univpll2_d2",
> - "top_syspll_d5";
> + audsys: clock-controller@11220000 {
> + compatible = "mediatek,mt2701-audsys", "syscon";
> + reg = <0 0x11220000 0 0x2000>;
> + #clock-cells = <1>;
> +
> + afe: audio-controller {
> + compatible = "mediatek,mt2701-audio";
> + interrupts = <GIC_SPI 104 IRQ_TYPE_LEVEL_LOW>,
> + <GIC_SPI 132 IRQ_TYPE_LEVEL_LOW>;
> + interrupt-names = "afe", "asys";
> + power-domains = <&scpsys MT2701_POWER_DOMAIN_IFR_MSC>;
> +
> + clocks = <&infracfg CLK_INFRA_AUDIO>,
> + <&topckgen CLK_TOP_AUD_MUX1_SEL>,
> + <&topckgen CLK_TOP_AUD_MUX2_SEL>,
> + <&topckgen CLK_TOP_AUD_48K_TIMING>,
> + <&topckgen CLK_TOP_AUD_44K_TIMING>,
> + <&topckgen CLK_TOP_AUD_K1_SRC_SEL>,
> + <&topckgen CLK_TOP_AUD_K2_SRC_SEL>,
> + <&topckgen CLK_TOP_AUD_K3_SRC_SEL>,
> + <&topckgen CLK_TOP_AUD_K4_SRC_SEL>,
> + <&topckgen CLK_TOP_AUD_K1_SRC_DIV>,
> + <&topckgen CLK_TOP_AUD_K2_SRC_DIV>,
> + <&topckgen CLK_TOP_AUD_K3_SRC_DIV>,
> + <&topckgen CLK_TOP_AUD_K4_SRC_DIV>,
> + <&topckgen CLK_TOP_AUD_I2S1_MCLK>,
> + <&topckgen CLK_TOP_AUD_I2S2_MCLK>,
> + <&topckgen CLK_TOP_AUD_I2S3_MCLK>,
> + <&topckgen CLK_TOP_AUD_I2S4_MCLK>,
> + <&audsys CLK_AUD_I2SO1>,
> + <&audsys CLK_AUD_I2SO2>,
> + <&audsys CLK_AUD_I2SO3>,
> + <&audsys CLK_AUD_I2SO4>,
> + <&audsys CLK_AUD_I2SIN1>,
> + <&audsys CLK_AUD_I2SIN2>,
> + <&audsys CLK_AUD_I2SIN3>,
> + <&audsys CLK_AUD_I2SIN4>,
> + <&audsys CLK_AUD_ASRCO1>,
> + <&audsys CLK_AUD_ASRCO2>,
> + <&audsys CLK_AUD_ASRCO3>,
> + <&audsys CLK_AUD_ASRCO4>,
> + <&audsys CLK_AUD_AFE>,
> + <&audsys CLK_AUD_AFE_CONN>,
> + <&audsys CLK_AUD_A1SYS>,
> + <&audsys CLK_AUD_A2SYS>,
> + <&audsys CLK_AUD_AFE_MRGIF>;
> +
> + clock-names = "infra_sys_audio_clk",
> + "top_audio_mux1_sel",
> + "top_audio_mux2_sel",
> + "top_audio_a1sys_hp",
> + "top_audio_a2sys_hp",
> + "i2s0_src_sel",
> + "i2s1_src_sel",
> + "i2s2_src_sel",
> + "i2s3_src_sel",
> + "i2s0_src_div",
> + "i2s1_src_div",
> + "i2s2_src_div",
> + "i2s3_src_div",
> + "i2s0_mclk_en",
> + "i2s1_mclk_en",
> + "i2s2_mclk_en",
> + "i2s3_mclk_en",
> + "i2so0_hop_ck",
> + "i2so1_hop_ck",
> + "i2so2_hop_ck",
> + "i2so3_hop_ck",
> + "i2si0_hop_ck",
> + "i2si1_hop_ck",
> + "i2si2_hop_ck",
> + "i2si3_hop_ck",
> + "asrc0_out_ck",
> + "asrc1_out_ck",
> + "asrc2_out_ck",
> + "asrc3_out_ck",
> + "audio_afe_pd",
> + "audio_afe_conn_pd",
> + "audio_a1sys_pd",
> + "audio_a2sys_pd",
> + "audio_mrgif_pd";
> +
> + assigned-clocks = <&topckgen CLK_TOP_AUD_MUX1_SEL>,
> + <&topckgen CLK_TOP_AUD_MUX2_SEL>,
> + <&topckgen CLK_TOP_AUD_MUX1_DIV>,
> + <&topckgen CLK_TOP_AUD_MUX2_DIV>;
> + assigned-clock-parents = <&topckgen CLK_TOP_AUD1PLL_98M>,
> + <&topckgen CLK_TOP_AUD2PLL_90M>;
> + assigned-clock-rates = <0>, <0>, <49152000>, <45158400>;
> + };
> };
>
> mmsys: syscon@14000000 {
> diff --git a/arch/arm/boot/dts/mt7623.dtsi b/arch/arm/boot/dts/mt7623.dtsi
> index b750da5..e4dd31d 100644
> --- a/arch/arm/boot/dts/mt7623.dtsi
> +++ b/arch/arm/boot/dts/mt7623.dtsi
> @@ -538,105 +538,99 @@
> status = "disabled";
> };
>
> - afe: audio-controller@11220000 {
> - compatible = "mediatek,mt7623-audio",
> - "mediatek,mt2701-audio";
> - reg = <0 0x11220000 0 0x2000>,
> - <0 0x112a0000 0 0x20000>;
> - interrupts = <GIC_SPI 104 IRQ_TYPE_LEVEL_LOW>,
> - <GIC_SPI 132 IRQ_TYPE_LEVEL_LOW>;
> - interrupt-names = "afe", "asys";
> - power-domains = <&scpsys MT2701_POWER_DOMAIN_IFR_MSC>;
> + audsys: clock-controller@11220000 {
> + compatible = "mediatek,mt7623-audsys",
> + "mediatek,mt2701-audsys",
> + "syscon";
> + reg = <0 0x11220000 0 0x2000>;
> + #clock-cells = <1>;
>
> - clocks = <&infracfg CLK_INFRA_AUDIO>,
> - <&topckgen CLK_TOP_AUD_MUX1_SEL>,
> - <&topckgen CLK_TOP_AUD_MUX2_SEL>,
> - <&topckgen CLK_TOP_AUD_MUX1_DIV>,
> - <&topckgen CLK_TOP_AUD_MUX2_DIV>,
> - <&topckgen CLK_TOP_AUD_48K_TIMING>,
> - <&topckgen CLK_TOP_AUD_44K_TIMING>,
> - <&topckgen CLK_TOP_AUDPLL_MUX_SEL>,
> - <&topckgen CLK_TOP_APLL_SEL>,
> - <&topckgen CLK_TOP_AUD1PLL_98M>,
> - <&topckgen CLK_TOP_AUD2PLL_90M>,
> - <&topckgen CLK_TOP_HADDS2PLL_98M>,
> - <&topckgen CLK_TOP_HADDS2PLL_294M>,
> - <&topckgen CLK_TOP_AUDPLL>,
> - <&topckgen CLK_TOP_AUDPLL_D4>,
> - <&topckgen CLK_TOP_AUDPLL_D8>,
> - <&topckgen CLK_TOP_AUDPLL_D16>,
> - <&topckgen CLK_TOP_AUDPLL_D24>,
> - <&topckgen CLK_TOP_AUDINTBUS_SEL>,
> - <&clk26m>,
> - <&topckgen CLK_TOP_SYSPLL1_D4>,
> - <&topckgen CLK_TOP_AUD_K1_SRC_SEL>,
> - <&topckgen CLK_TOP_AUD_K2_SRC_SEL>,
> - <&topckgen CLK_TOP_AUD_K3_SRC_SEL>,
> - <&topckgen CLK_TOP_AUD_K4_SRC_SEL>,
> - <&topckgen CLK_TOP_AUD_K5_SRC_SEL>,
> - <&topckgen CLK_TOP_AUD_K6_SRC_SEL>,
> - <&topckgen CLK_TOP_AUD_K1_SRC_DIV>,
> - <&topckgen CLK_TOP_AUD_K2_SRC_DIV>,
> - <&topckgen CLK_TOP_AUD_K3_SRC_DIV>,
> - <&topckgen CLK_TOP_AUD_K4_SRC_DIV>,
> - <&topckgen CLK_TOP_AUD_K5_SRC_DIV>,
> - <&topckgen CLK_TOP_AUD_K6_SRC_DIV>,
> - <&topckgen CLK_TOP_AUD_I2S1_MCLK>,
> - <&topckgen CLK_TOP_AUD_I2S2_MCLK>,
> - <&topckgen CLK_TOP_AUD_I2S3_MCLK>,
> - <&topckgen CLK_TOP_AUD_I2S4_MCLK>,
> - <&topckgen CLK_TOP_AUD_I2S5_MCLK>,
> - <&topckgen CLK_TOP_AUD_I2S6_MCLK>,
> - <&topckgen CLK_TOP_ASM_M_SEL>,
> - <&topckgen CLK_TOP_ASM_H_SEL>,
> - <&topckgen CLK_TOP_UNIVPLL2_D4>,
> - <&topckgen CLK_TOP_UNIVPLL2_D2>,
> - <&topckgen CLK_TOP_SYSPLL_D5>;
> -
> - clock-names = "infra_sys_audio_clk",
> - "top_audio_mux1_sel",
> - "top_audio_mux2_sel",
> - "top_audio_mux1_div",
> - "top_audio_mux2_div",
> - "top_audio_48k_timing",
> - "top_audio_44k_timing",
> - "top_audpll_mux_sel",
> - "top_apll_sel",
> - "top_aud1_pll_98M",
> - "top_aud2_pll_90M",
> - "top_hadds2_pll_98M",
> - "top_hadds2_pll_294M",
> - "top_audpll",
> - "top_audpll_d4",
> - "top_audpll_d8",
> - "top_audpll_d16",
> - "top_audpll_d24",
> - "top_audintbus_sel",
> - "clk_26m",
> - "top_syspll1_d4",
> - "top_aud_k1_src_sel",
> - "top_aud_k2_src_sel",
> - "top_aud_k3_src_sel",
> - "top_aud_k4_src_sel",
> - "top_aud_k5_src_sel",
> - "top_aud_k6_src_sel",
> - "top_aud_k1_src_div",
> - "top_aud_k2_src_div",
> - "top_aud_k3_src_div",
> - "top_aud_k4_src_div",
> - "top_aud_k5_src_div",
> - "top_aud_k6_src_div",
> - "top_aud_i2s1_mclk",
> - "top_aud_i2s2_mclk",
> - "top_aud_i2s3_mclk",
> - "top_aud_i2s4_mclk",
> - "top_aud_i2s5_mclk",
> - "top_aud_i2s6_mclk",
> - "top_asm_m_sel",
> - "top_asm_h_sel",
> - "top_univpll2_d4",
> - "top_univpll2_d2",
> - "top_syspll_d5";
> + afe: audio-controller {
> + compatible = "mediatek,mt7623-audio",
> + "mediatek,mt2701-audio";
> + interrupts = <GIC_SPI 104 IRQ_TYPE_LEVEL_LOW>,
> + <GIC_SPI 132 IRQ_TYPE_LEVEL_LOW>;
> + interrupt-names = "afe", "asys";
> + power-domains = <&scpsys MT2701_POWER_DOMAIN_IFR_MSC>;
> +
> + clocks = <&infracfg CLK_INFRA_AUDIO>,
> + <&topckgen CLK_TOP_AUD_MUX1_SEL>,
> + <&topckgen CLK_TOP_AUD_MUX2_SEL>,
> + <&topckgen CLK_TOP_AUD_48K_TIMING>,
> + <&topckgen CLK_TOP_AUD_44K_TIMING>,
> + <&topckgen CLK_TOP_AUD_K1_SRC_SEL>,
> + <&topckgen CLK_TOP_AUD_K2_SRC_SEL>,
> + <&topckgen CLK_TOP_AUD_K3_SRC_SEL>,
> + <&topckgen CLK_TOP_AUD_K4_SRC_SEL>,
> + <&topckgen CLK_TOP_AUD_K1_SRC_DIV>,
> + <&topckgen CLK_TOP_AUD_K2_SRC_DIV>,
> + <&topckgen CLK_TOP_AUD_K3_SRC_DIV>,
> + <&topckgen CLK_TOP_AUD_K4_SRC_DIV>,
> + <&topckgen CLK_TOP_AUD_I2S1_MCLK>,
> + <&topckgen CLK_TOP_AUD_I2S2_MCLK>,
> + <&topckgen CLK_TOP_AUD_I2S3_MCLK>,
> + <&topckgen CLK_TOP_AUD_I2S4_MCLK>,
> + <&audsys CLK_AUD_I2SO1>,
> + <&audsys CLK_AUD_I2SO2>,
> + <&audsys CLK_AUD_I2SO3>,
> + <&audsys CLK_AUD_I2SO4>,
> + <&audsys CLK_AUD_I2SIN1>,
> + <&audsys CLK_AUD_I2SIN2>,
> + <&audsys CLK_AUD_I2SIN3>,
> + <&audsys CLK_AUD_I2SIN4>,
> + <&audsys CLK_AUD_ASRCO1>,
> + <&audsys CLK_AUD_ASRCO2>,
> + <&audsys CLK_AUD_ASRCO3>,
> + <&audsys CLK_AUD_ASRCO4>,
> + <&audsys CLK_AUD_AFE>,
> + <&audsys CLK_AUD_AFE_CONN>,
> + <&audsys CLK_AUD_A1SYS>,
> + <&audsys CLK_AUD_A2SYS>,
> + <&audsys CLK_AUD_AFE_MRGIF>;
> +
> + clock-names = "infra_sys_audio_clk",
> + "top_audio_mux1_sel",
> + "top_audio_mux2_sel",
> + "top_audio_a1sys_hp",
> + "top_audio_a2sys_hp",
> + "i2s0_src_sel",
> + "i2s1_src_sel",
> + "i2s2_src_sel",
> + "i2s3_src_sel",
> + "i2s0_src_div",
> + "i2s1_src_div",
> + "i2s2_src_div",
> + "i2s3_src_div",
> + "i2s0_mclk_en",
> + "i2s1_mclk_en",
> + "i2s2_mclk_en",
> + "i2s3_mclk_en",
> + "i2so0_hop_ck",
> + "i2so1_hop_ck",
> + "i2so2_hop_ck",
> + "i2so3_hop_ck",
> + "i2si0_hop_ck",
> + "i2si1_hop_ck",
> + "i2si2_hop_ck",
> + "i2si3_hop_ck",
> + "asrc0_out_ck",
> + "asrc1_out_ck",
> + "asrc2_out_ck",
> + "asrc3_out_ck",
> + "audio_afe_pd",
> + "audio_afe_conn_pd",
> + "audio_a1sys_pd",
> + "audio_a2sys_pd",
> + "audio_mrgif_pd";
> +
> + assigned-clocks = <&topckgen CLK_TOP_AUD_MUX1_SEL>,
> + <&topckgen CLK_TOP_AUD_MUX2_SEL>,
> + <&topckgen CLK_TOP_AUD_MUX1_DIV>,
> + <&topckgen CLK_TOP_AUD_MUX2_DIV>;
> + assigned-clock-parents = <&topckgen CLK_TOP_AUD1PLL_98M>,
> + <&topckgen CLK_TOP_AUD2PLL_90M>;
> + assigned-clock-rates = <0>, <0>, <49152000>, <45158400>;
> + };
> };
>
> mmc0: mmc@11230000 {
>