Re: [PATCH v4 2/2] arm64: dts: qcom: sc7180-trogdor: Add lpass dai link for HDMI

From: Doug Anderson
Date: Wed Aug 11 2021 - 18:10:28 EST


Hi,

On Tue, Aug 10, 2021 at 11:06 PM Stephen Boyd <swboyd@xxxxxxxxxxxx> wrote:
>
> Quoting Srinivasa Rao Mandadapu (2021-07-21 01:05:49)
> > diff --git a/arch/arm64/boot/dts/qcom/sc7180-trogdor.dtsi b/arch/arm64/boot/dts/qcom/sc7180-trogdor.dtsi
> > index 31bf7c698b8f..a4cb9ee567ff 100644
> > --- a/arch/arm64/boot/dts/qcom/sc7180-trogdor.dtsi
> > +++ b/arch/arm64/boot/dts/qcom/sc7180-trogdor.dtsi
> > @@ -288,6 +288,7 @@ sound: sound {
> > "Headphone Jack", "HPOL",
> > "Headphone Jack", "HPOR";
> >
> > + #sound-dai-cells = <0>;
> > #address-cells = <1>;
> > #size-cells = <0>;
> >
> > @@ -314,6 +315,18 @@ sound_multimedia1_codec: codec {
> > sound-dai = <&max98357a>;
> > };
> > };
> > +
> > + dai-link@2 {
> > + link-name = "MultiMedia2";
> > + reg = <2>;
>
> Should this be
>
> reg = <LPASS_DP_RX>;
>
> ? And then the dai-link@2 should change to dai-link@5? It doesn't seem
> to really matter though, so maybe not.
>
> > + cpu {
> > + sound-dai = <&lpass_cpu 2>;
>
> This should be
>
> sound-dai = <&lpass_cpu LPASS_DP_RX>;
>
> ? At least from what I can tell without having it be 5 it doesn't work
> properly and external audio over DP doesn't enumerate.

I swear I provided feedback just like that. Yeah, here:

https://lore.kernel.org/r/CAD=FV=W3X8W90vPdrDAymzTKj-J7QPyn4ukaLSOhEkCme3+r_Q@xxxxxxxxxxxxxx

I think the problem is that the wrong patch landed. The one I see that
landed last week was v4 but I also see v7 of the same patch:

https://lore.kernel.org/r/20210726120910.20335-3-srivasam@xxxxxxxxxxxxxx/