From: Chen-Yu Tsai <wenst@chromium.org>
To: Kansho Nishida <kansho@chromium.org>
Cc: Matthias Brugger <matthias.bgg@gmail.com>,
linux-mediatek@lists.infradead.org,
LKML <linux-kernel@vger.kernel.org>,
Jiaxin Yu <jiaxin.yu@mediatek.com>,
devicetree@vger.kernel.org, Rob Herring <robh+dt@kernel.org>,
Shunli Wang <shunli.wang@mediatek.com>,
linux-arm-kernel@lists.infradead.org,
Eddie Huang <eddie.huang@mediatek.com>
Subject: Re: [PATCH 1/2] arm64: dts: mt8183: add audio node
Date: Mon, 5 Jul 2021 17:55:00 +0800 [thread overview]
Message-ID: <CAGXv+5FVf3hu5Y+aFazqBwAcvWMGkyqXB-2VVnskmMsYKj2_yA@mail.gmail.com> (raw)
In-Reply-To: <CAP3OrSLyoKNr7fMOx5sUtWi7PVQGuN-5w7k_0D2MhDUeXXmYCg@mail.gmail.com>
Hi,
On Mon, Jul 5, 2021 at 5:40 PM Kansho Nishida <kansho@chromium.org> wrote:
>
> On Fri, Jul 2, 2021 at 3:20 AM Matthias Brugger <matthias.bgg@gmail.com> 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@chromium.org>
> > > ---
> > >
> > > 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@google.com/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.
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@mediatek.com/
>
> 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@lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/linux-mediatek
next prev parent reply other threads:[~2021-07-05 9:55 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-06-30 9:07 [PATCH 0/2] arm64: dts: mt8183: Add kukui platform audio node to the device tree Kansho Nishida
2021-06-30 9:07 ` [PATCH 1/2] arm64: dts: mt8183: add audio node Kansho Nishida
2021-07-01 18:20 ` Matthias Brugger
2021-07-05 9:39 ` Kansho Nishida
2021-07-05 9:55 ` Chen-Yu Tsai [this message]
2021-07-06 9:38 ` Kansho Nishida
2021-06-30 9:07 ` [PATCH 2/2] arm64: dts: mt8183: add kukui platform " Kansho Nishida
2021-07-01 18:23 ` Matthias Brugger
2021-07-05 9:44 ` Kansho Nishida
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=CAGXv+5FVf3hu5Y+aFazqBwAcvWMGkyqXB-2VVnskmMsYKj2_yA@mail.gmail.com \
--to=wenst@chromium.org \
--cc=devicetree@vger.kernel.org \
--cc=eddie.huang@mediatek.com \
--cc=jiaxin.yu@mediatek.com \
--cc=kansho@chromium.org \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-mediatek@lists.infradead.org \
--cc=matthias.bgg@gmail.com \
--cc=robh+dt@kernel.org \
--cc=shunli.wang@mediatek.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).