Re: [PATCH v3 1/2] arm64: dts: qcom: eliza: Add LPASS macro and SoundWire support

[Ravi Hothi]

On 7/29/2026 3:05 PM, Konrad Dybcio wrote:
> On 7/23/26 2:15 PM, Ravi Hothi wrote:
> > Add SoC-level LPASS WSA macro, VA macro, SoundWire controller and LPASS
> > LPI pin controller nodes. DMIC and WSA SoundWire pinctrl states are
> > defined inside the LPASS LPI pin controller node. The hardware is similar
> > to the SM8750 platform.
> >
> > The WSA macro, VA macro and SoundWire controller are kept disabled so
> > board DTS files can selectively enable and configure them.
> >
> > Signed-off-by: Ravi Hothi <ravi.hothi@xxxxxxxxxxxxxxxx>
> > ---
>
> [...]
>
> > +		lpass_wsamacro: codec@6b00000 {
> > +			compatible = "qcom,eliza-lpass-wsa-macro",
> > +				     "qcom,sm8550-lpass-wsa-macro";
> > +			reg = <0x0 0x06b00000 0x0 0x1000>;
> > +
> > +			clocks = <&q6prmcc LPASS_CLK_ID_WSA_CORE_TX_MCLK
> > +					   LPASS_CLK_ATTRIBUTE_COUPLE_NO>,
> > +				 <&q6prmcc LPASS_HW_MACRO_VOTE
> > +					   LPASS_CLK_ATTRIBUTE_COUPLE_NO>,
> > +				 <&q6prmcc LPASS_HW_DCODEC_VOTE
> > +					   LPASS_CLK_ATTRIBUTE_COUPLE_NO>,
> > +				 <&lpass_vamacro>;
> > +			clock-names = "mclk",
> > +				      "macro",
> > +				      "dcodec",
> > +				      "fsgen";
> > +
> > +			#clock-cells = <0>;
> > +			clock-output-names = "mclk";
> > +			#sound-dai-cells = <1>;
> > +
> > +			status = "disabled";
>
> All macros (but not swr hosts) are usually enabled by default on other
> socs, let's do the same here

ACK, Will update in next version.

Thanks,
Ravi Hothi
> Konrad