Re: [PATCH 6/6] ASoC: dt-bindings: wcd938x: fix codec reset line polarity in example

From: Rob Herring
Date: Fri Oct 28 2022 - 15:39:58 EST


On Thu, Oct 27, 2022 at 12:46:52AM -0700, Dmitry Torokhov wrote:
> The driver for the codec, when resetting the chip, first drives the line
> low, and then high. This means that the line is active low. Change the
> annotation in the example DTS accordingly.
>
> Signed-off-by: Dmitry Torokhov <dmitry.torokhov@xxxxxxxxx>
> ---
> Documentation/devicetree/bindings/sound/qcom,wcd938x.yaml | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/Documentation/devicetree/bindings/sound/qcom,wcd938x.yaml b/Documentation/devicetree/bindings/sound/qcom,wcd938x.yaml
> index 51547190f709..2f5e0df93872 100644
> --- a/Documentation/devicetree/bindings/sound/qcom,wcd938x.yaml
> +++ b/Documentation/devicetree/bindings/sound/qcom,wcd938x.yaml
> @@ -112,7 +112,7 @@ examples:
> - |
> codec {
> compatible = "qcom,wcd9380-codec";
> - reset-gpios = <&tlmm 32 0>;
> + reset-gpios = <&tlmm 32 GPIO_ACTIVE_LOW>;

Missing a header for this.

> #sound-dai-cells = <1>;
> qcom,tx-device = <&wcd938x_tx>;
> qcom,rx-device = <&wcd938x_rx>;
> --
> 2.38.0.135.g90850a2211-goog
>
>