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

[Srinivasa Rao Mandadapu]

Hi Stephen!!

Thanks for reminder.

On 7/21/2021 10:14 AM, Stephen Boyd wrote:
> Quoting Srinivasa Rao Mandadapu (2020-09-29 23:42:48)
> > From: V Sujith Kumar Reddy <vsujithk@xxxxxxxxxxxxxx>
> >
> > Add dai link in sc7180-trogdor.dtsi for supporting audio over DP
> >
> > Signed-off-by: V Sujith Kumar Reddy <vsujithk@xxxxxxxxxxxxxx>
> > Signed-off-by: Srinivasa Rao Mandadapu <srivasam@xxxxxxxxxxxxxx>
> > ---
> >   arch/arm64/boot/dts/qcom/sc7180-trogdor.dtsi | 15 +++++++++++++++
> >   1 file changed, 15 insertions(+)
> >
> > diff --git a/arch/arm64/boot/dts/qcom/sc7180-trogdor.dtsi b/arch/arm64/boot/dts/qcom/sc7180-trogdor.dtsi
> > index 5724982..850b43e 100644
> > --- a/arch/arm64/boot/dts/qcom/sc7180-trogdor.dtsi
> > +++ b/arch/arm64/boot/dts/qcom/sc7180-trogdor.dtsi
> > @@ -231,6 +231,7 @@
> >
> >                  audio-jack = <&alc5682>;
> >
> > +               #sound-dai-cells = <0>;
> >                  #address-cells = <1>;
> >                  #size-cells = <0>;
> >
> > @@ -257,6 +258,17 @@
> >                                  sound-dai = <&max98357a>;
> >                          };
> >                  };
> > +               dai-link@2 {
> > +                       link-name = "MultiMedia2";
> > +                       reg = <2>;
> > +                       cpu {
> > +                               sound-dai = <&lpass_cpu 2>;
> > +                       };
> > +
> > +                       codec {
> > +                               sound-dai = <&msm_dp>;
> I see qcom maintainers have picked up the dp node now. Can you resend?
> This would need to change to mdss_dp. Also incorporate my comments from
> January this year please.
Resent the patches. Please check V5 version of patches.

--
Qualcomm India Private Limited, on behalf of Qualcomm Innovation Center, Inc.,
is a member of Code Aurora Forum, a Linux Foundation Collaborative Project.