Re: [PATCH v1] arm64: dts: qcom: qcm6490-idp: Fix WCD9370 reset GPIO polarity

[Krzysztof Kozlowski]

On 20/02/2026 10:02, Ravi Hothi wrote:
> The WCD9370 audio codec reset line on QCM6490 IDP should be active-low, but
> the device tree described it as active-high. As a result, the codec is
> kept in reset and fails to reset the SoundWire, leading to timeouts
> and ASoC card probe failure (-ETIMEDOUT).
> 
> Fix the reset GPIO polarity to GPIO_ACTIVE_LOW so the codec can properly
> initialize.
> 
> Fixes: aa04c298619f ("arm64: dts: qcom: qcm6490-idp: Add WSA8830 speakers and WCD9370 headset codec")

So above was never tested on mainline.

I fixed the driver in 2251db28edcc70b7ee8a8c6bcbaecf752b3ea5ec, so back
in 2024, thus sound on qcm6490-idp COULD NOT WORK when that DTS change
was posted...

...unless it was developed on downstream.

And a BASIC probe test would discover that - whether device probes
and/or appears in sysfs soundwire bus.

Since basic test would discover it, then I really believe this was done
on some downstream-style clone.

I think this is wider problem with @qualcomm patches.


> Signed-off-by: Ravi Hothi <ravi.hothi@xxxxxxxxxxxxxxxx>
> ---
>  arch/arm64/boot/dts/qcom/qcm6490-idp.dts | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 

Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@xxxxxxxxxxxxxxxx>

Best regards,
Krzysztof