Re: [PATCH 2/6] arm64: dts: qcom: sc8280xp: disable sound nodes
From: Johan Hovold
Date: Mon Jan 02 2023 - 10:16:01 EST
On Mon, Jan 02, 2023 at 01:29:38PM +0100, Krzysztof Kozlowski wrote:
> On 02/01/2023 11:50, Johan Hovold wrote:
> > The sound nodes in the SoC dtsi should be disabled by default.
> >
> > Signed-off-by: Johan Hovold <johan+linaro@xxxxxxxxxx>
> > ---
> > arch/arm64/boot/dts/qcom/sc8280xp.dtsi | 16 ++++++++++++++++
> > 1 file changed, 16 insertions(+)
> >
> > diff --git a/arch/arm64/boot/dts/qcom/sc8280xp.dtsi b/arch/arm64/boot/dts/qcom/sc8280xp.dtsi
> > index ed1e2bee86ee..c1ce2d7b3675 100644
> > --- a/arch/arm64/boot/dts/qcom/sc8280xp.dtsi
> > +++ b/arch/arm64/boot/dts/qcom/sc8280xp.dtsi
> > @@ -1733,6 +1733,8 @@ rxmacro: rxmacro@3200000 {
> >
> > pinctrl-names = "default";
> > pinctrl-0 = <&rx_swr_default>;
> > +
> > + status = "disabled";
>
> There is no reason this should be disabled by default. No external (per
> board) configuration is needed and by default SoC components should be
> enabled if they do not need anything from the board.
This node is one of the nodes for which driver support is not yet in
place so that's one reason for at least disabling it temporarily.
Since all other pinconfig lives in the board dts, if we decide to move
also the sound pinconfig then then that may be a second.
Johan