[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <CAA+D8AM7hEcyVaZF7MHN-jY_NOS+cdhrXGdTXeQ48-WV=bnLMA@mail.gmail.com>
Date: Wed, 6 Aug 2025 11:49:57 +0800
From: Shengjiu Wang <shengjiu.wang@...il.com>
To: Alexander Stein <alexander.stein@...tq-group.com>
Cc: andrzej.hajda@...el.com, neil.armstrong@...aro.org, rfoss@...nel.org,
Laurent.pinchart@...asonboard.com, jonas@...boo.se, jernej.skrabec@...il.com,
maarten.lankhorst@...ux.intel.com, mripard@...nel.org, tzimmermann@...e.de,
airlied@...il.com, simona@...ll.ch, lumag@...nel.org, dianders@...omium.org,
cristian.ciocaltea@...labora.com, luca.ceresoli@...tlin.com,
dri-devel@...ts.freedesktop.org, linux-kernel@...r.kernel.org,
victor.liu@....com, shawnguo@...nel.org, s.hauer@...gutronix.de,
kernel@...gutronix.de, festevam@...il.com, imx@...ts.linux.dev,
linux-arm-kernel@...ts.infradead.org, robh@...nel.org, krzk+dt@...nel.org,
conor+dt@...nel.org, p.zabel@...gutronix.de, devicetree@...r.kernel.org,
l.stach@...gutronix.de, perex@...ex.cz, tiwai@...e.com,
linux-sound@...r.kernel.org, Shengjiu Wang <shengjiu.wang@....com>
Subject: Re: [PATCH v3 6/6] arm64: dts: imx8mp: Add hdmi parallel audio
interface node
On Tue, Aug 5, 2025 at 3:10 PM Alexander Stein
<alexander.stein@...tq-group.com> wrote:
>
> Am Montag, 4. August 2025, 12:47:22 CEST schrieb Shengjiu Wang:
> > The HDMI TX Parallel Audio Interface (HTX_PAI) is a bridge between the
> > Audio Subsystem to the HDMI TX Controller.
> >
> > Shrink register map size of hdmi_pvi to avoid overlapped hdmi_pai device.
> >
> > Signed-off-by: Shengjiu Wang <shengjiu.wang@....com>
> > ---
> > arch/arm64/boot/dts/freescale/imx8mp-evk.dts | 4 +++
> > arch/arm64/boot/dts/freescale/imx8mp.dtsi | 28 +++++++++++++++++++-
>
> Please separate commits for SoC and board files. Thanks
Ok.
Best regards
Shengjiu Wang
>
> Best regards,
> Alexander
>
> > 2 files changed, 31 insertions(+), 1 deletion(-)
> >
> > diff --git a/arch/arm64/boot/dts/freescale/imx8mp-evk.dts b/arch/arm64/boot/dts/freescale/imx8mp-evk.dts
> > index c0cc5611048e..cc9351a5bd65 100644
> > --- a/arch/arm64/boot/dts/freescale/imx8mp-evk.dts
> > +++ b/arch/arm64/boot/dts/freescale/imx8mp-evk.dts
> > @@ -442,6 +442,10 @@ &flexcan2 {
> > status = "disabled";/* can2 pin conflict with pdm */
> > };
> >
> > +&hdmi_pai {
> > + status = "okay";
> > +};
> > +
> > &hdmi_pvi {
> > status = "okay";
> > };
> > diff --git a/arch/arm64/boot/dts/freescale/imx8mp.dtsi b/arch/arm64/boot/dts/freescale/imx8mp.dtsi
> > index 841d155685ee..00d8474bd1b1 100644
> > --- a/arch/arm64/boot/dts/freescale/imx8mp.dtsi
> > +++ b/arch/arm64/boot/dts/freescale/imx8mp.dtsi
> > @@ -2066,7 +2066,7 @@ irqsteer_hdmi: interrupt-controller@...c2000 {
> >
> > hdmi_pvi: display-bridge@...c4000 {
> > compatible = "fsl,imx8mp-hdmi-pvi";
> > - reg = <0x32fc4000 0x1000>;
> > + reg = <0x32fc4000 0x800>;
> > interrupt-parent = <&irqsteer_hdmi>;
> > interrupts = <12>;
> > power-domains = <&hdmi_blk_ctrl IMX8MP_HDMIBLK_PD_PVI>;
> > @@ -2092,6 +2092,24 @@ pvi_to_hdmi_tx: endpoint {
> > };
> > };
> >
> > + hdmi_pai: audio-bridge@...c4800 {
> > + compatible = "fsl,imx8mp-hdmi-pai";
> > + reg = <0x32fc4800 0x800>;
> > + interrupt-parent = <&irqsteer_hdmi>;
> > + interrupts = <14>;
> > + clocks = <&clk IMX8MP_CLK_HDMI_APB>;
> > + clock-names = "apb";
> > + power-domains = <&hdmi_blk_ctrl IMX8MP_HDMIBLK_PD_PAI>;
> > + status = "disabled";
> > +
> > + port {
> > +
> > + pai_to_hdmi_tx: endpoint {
> > + remote-endpoint = <&hdmi_tx_from_pai>;
> > + };
> > + };
> > + };
> > +
> > lcdif3: display-controller@...c6000 {
> > compatible = "fsl,imx8mp-lcdif";
> > reg = <0x32fc6000 0x1000>;
> > @@ -2143,6 +2161,14 @@ port@1 {
> > reg = <1>;
> > /* Point endpoint to the HDMI connector */
> > };
> > +
> > + port@2 {
> > + reg = <2>;
> > +
> > + hdmi_tx_from_pai: endpoint {
> > + remote-endpoint = <&pai_to_hdmi_tx>;
> > + };
> > + };
> > };
> > };
> >
> >
>
>
> --
> TQ-Systems GmbH | Mühlstraße 2, Gut Delling | 82229 Seefeld, Germany
> Amtsgericht München, HRB 105018
> Geschäftsführer: Detlef Schneider, Rüdiger Stahl, Stefan Schneider
> http://www.tq-group.com/
>
>
Powered by blists - more mailing lists