Re: [PATCH 1/2] arm64: dts: mt8183: add audio node

From: Kansho Nishida
Date: Tue Jul 06 2021 - 05:38:34 EST

Hi Chen,

On Mon, Jul 5, 2021 at 6:55 PM Chen-Yu Tsai <wenst@xxxxxxxxxxxx> wrote:
>
> Hi,
>
> On Mon, Jul 5, 2021 at 5:40 PM Kansho Nishida <kansho@xxxxxxxxxxxx> wrote:
> >
> > On Fri, Jul 2, 2021 at 3:20 AM Matthias Brugger <matthias.bgg@xxxxxxxxx> wrote:
> > >
> > >
> > >
> > > On 30/06/2021 11:07, Kansho Nishida wrote:
> > > > Add afe (audio front end) device node to the MT8183 dtsi.
> > > >
> > > > Signed-off-by: Kansho Nishida <kansho@xxxxxxxxxxxx>
> > > > ---
> > > >
> > > > arch/arm64/boot/dts/mediatek/mt8183.dtsi | 94 +++++++++++++++++++++++-
> > > > 1 file changed, 93 insertions(+), 1 deletion(-)
> > > >
> > > > diff --git a/arch/arm64/boot/dts/mediatek/mt8183.dtsi b/arch/arm64/boot/dts/mediatek/mt8183.dtsi
> > > > index f90df6439c08..b06acb8d6527 100644
> > > > --- a/arch/arm64/boot/dts/mediatek/mt8183.dtsi
> > > > +++ b/arch/arm64/boot/dts/mediatek/mt8183.dtsi
> > > > @@ -1115,10 +1115,102 @@ usb_host: usb@11200000 {
> > > > };
> > > > };
> > > >
> > > > - audiosys: syscon@11220000 {
> > > > + audiosys: clock-controller@11220000 {
> > >
> > > Why do you change it from sysecon to clock-controller?
> > >
> > > Regards,
> > > Matthias
> >
> > Hi Matthias,
> >
> > Thanks for the comments!
> > This should be "clock-controller" according to the binding description (*1).
> > Moreover, you suggested doing so even though it's a long time ago (*2).
>
> The address space really covers the whole audio subsystem though. The clock
> controls only span a few registers. In hindsight the clock controls shouldn't
> have been modelled separately, since they are only used internally within the
> audio subsystem.
>
> See https://lore.kernel.org/linux-mediatek/YOLKxrJin5kkwiIl@xxxxxxxxxx/T/#u
> for issues I raised on the MT8195.
>
> Now we are probably not going to be able to fix this for existing bindings,
> so let's just get the node name right. It should probably be audio-controller
> or something close instead.

OK, let's use audio-controller instead. I'll submit the v3 patch.

Regards,
Kansho

>
>
> Regards
> ChenYu
>
>
> > *1) https://www.kernel.org/doc/Documentation/devicetree/bindings/arm/mediatek/mediatek%2Caudsys.txt
> > *2) https://patchwork.kernel.org/project/linux-mediatek/patch/1559360354-22974-1-git-send-email-jiaxin.yu@xxxxxxxxxxxx/
> >
> > Regards,
> > Kansho
> >
> > >
> > > > compatible = "mediatek,mt8183-audiosys", "syscon";
> > > > reg = <0 0x11220000 0 0x1000>;
> > > > #clock-cells = <1>;
> > > > + afe: mt8183-afe-pcm {
> > > > + compatible = "mediatek,mt8183-audio";
> > > > + interrupts = <GIC_SPI 161 IRQ_TYPE_LEVEL_LOW>;
> > > > + resets = <&watchdog MT8183_TOPRGU_AUDIO_SW_RST>;
> > > > + reset-names = "audiosys";
> > > > + power-domains =
> > > > + <&spm MT8183_POWER_DOMAIN_AUDIO>;
> > > > + clocks = <&audiosys CLK_AUDIO_AFE>,
> > > > + <&audiosys CLK_AUDIO_DAC>,
> > > > + <&audiosys CLK_AUDIO_DAC_PREDIS>,
> > > > + <&audiosys CLK_AUDIO_ADC>,
> > > > + <&audiosys CLK_AUDIO_PDN_ADDA6_ADC>,
> > > > + <&audiosys CLK_AUDIO_22M>,
> > > > + <&audiosys CLK_AUDIO_24M>,
> > > > + <&audiosys CLK_AUDIO_APLL_TUNER>,
> > > > + <&audiosys CLK_AUDIO_APLL2_TUNER>,
> > > > + <&audiosys CLK_AUDIO_I2S1>,
> > > > + <&audiosys CLK_AUDIO_I2S2>,
> > > > + <&audiosys CLK_AUDIO_I2S3>,
> > > > + <&audiosys CLK_AUDIO_I2S4>,
> > > > + <&audiosys CLK_AUDIO_TDM>,
> > > > + <&audiosys CLK_AUDIO_TML>,
> > > > + <&infracfg CLK_INFRA_AUDIO>,
> > > > + <&infracfg CLK_INFRA_AUDIO_26M_BCLK>,
> > > > + <&topckgen CLK_TOP_MUX_AUDIO>,
> > > > + <&topckgen CLK_TOP_MUX_AUD_INTBUS>,
> > > > + <&topckgen CLK_TOP_SYSPLL_D2_D4>,
> > > > + <&topckgen CLK_TOP_MUX_AUD_1>,
> > > > + <&topckgen CLK_TOP_APLL1_CK>,
> > > > + <&topckgen CLK_TOP_MUX_AUD_2>,
> > > > + <&topckgen CLK_TOP_APLL2_CK>,
> > > > + <&topckgen CLK_TOP_MUX_AUD_ENG1>,
> > > > + <&topckgen CLK_TOP_APLL1_D8>,
> > > > + <&topckgen CLK_TOP_MUX_AUD_ENG2>,
> > > > + <&topckgen CLK_TOP_APLL2_D8>,
> > > > + <&topckgen CLK_TOP_MUX_APLL_I2S0>,
> > > > + <&topckgen CLK_TOP_MUX_APLL_I2S1>,
> > > > + <&topckgen CLK_TOP_MUX_APLL_I2S2>,
> > > > + <&topckgen CLK_TOP_MUX_APLL_I2S3>,
> > > > + <&topckgen CLK_TOP_MUX_APLL_I2S4>,
> > > > + <&topckgen CLK_TOP_MUX_APLL_I2S5>,
> > > > + <&topckgen CLK_TOP_APLL12_DIV0>,
> > > > + <&topckgen CLK_TOP_APLL12_DIV1>,
> > > > + <&topckgen CLK_TOP_APLL12_DIV2>,
> > > > + <&topckgen CLK_TOP_APLL12_DIV3>,
> > > > + <&topckgen CLK_TOP_APLL12_DIV4>,
> > > > + <&topckgen CLK_TOP_APLL12_DIVB>,
> > > > + /*<&topckgen CLK_TOP_APLL12_DIV5>,*/
> > > > + <&clk26m>;
> > > > + clock-names = "aud_afe_clk",
> > > > + "aud_dac_clk",
> > > > + "aud_dac_predis_clk",
> > > > + "aud_adc_clk",
> > > > + "aud_adc_adda6_clk",
> > > > + "aud_apll22m_clk",
> > > > + "aud_apll24m_clk",
> > > > + "aud_apll1_tuner_clk",
> > > > + "aud_apll2_tuner_clk",
> > > > + "aud_i2s1_bclk_sw",
> > > > + "aud_i2s2_bclk_sw",
> > > > + "aud_i2s3_bclk_sw",
> > > > + "aud_i2s4_bclk_sw",
> > > > + "aud_tdm_clk",
> > > > + "aud_tml_clk",
> > > > + "aud_infra_clk",
> > > > + "mtkaif_26m_clk",
> > > > + "top_mux_audio",
> > > > + "top_mux_aud_intbus",
> > > > + "top_syspll_d2_d4",
> > > > + "top_mux_aud_1",
> > > > + "top_apll1_ck",
> > > > + "top_mux_aud_2",
> > > > + "top_apll2_ck",
> > > > + "top_mux_aud_eng1",
> > > > + "top_apll1_d8",
> > > > + "top_mux_aud_eng2",
> > > > + "top_apll2_d8",
> > > > + "top_i2s0_m_sel",
> > > > + "top_i2s1_m_sel",
> > > > + "top_i2s2_m_sel",
> > > > + "top_i2s3_m_sel",
> > > > + "top_i2s4_m_sel",
> > > > + "top_i2s5_m_sel",
> > > > + "top_apll12_div0",
> > > > + "top_apll12_div1",
> > > > + "top_apll12_div2",
> > > > + "top_apll12_div3",
> > > > + "top_apll12_div4",
> > > > + "top_apll12_divb",
> > > > + /*"top_apll12_div5",*/
> > > > + "top_clk26m_clk";
> > > > + };
> > > > };
> > > >
> > > > mmc0: mmc@11230000 {
> > > >
> >
> > _______________________________________________
> > Linux-mediatek mailing list
> > Linux-mediatek@xxxxxxxxxxxxxxxxxxx
> > http://lists.infradead.org/mailman/listinfo/linux-mediatek