Re: [PATCH v2 1/2] ASoC: dt-bindings: nau8822: Add supply regulators

[sashiko-bot@kernel.org]

Thank you for your contribution! Sashiko AI review found 1 potential issue(s) to consider:
- [Low] Typo in the description for `vddspk-supply`.
--

commit 2a0698dd10cf90a92b31e252ae2cfd2a236cf2d9
Author: Alexey Charkov <alchark@flipper.net>
ASoC: dt-bindings: nau8822: Add supply regulators

This commit adds optional power supply pins (VDDA, VDDB, VDDC, and VDDSPK)
to the NAU8822 device tree binding. This allows device tree users to ensure
correct power sequencing before communicating with the device.

> diff --git a/Documentation/devicetree/bindings/sound/nuvoton,nau8822.yaml b/Documentation/devicetree/bindings/sound/nuvoton,nau8822.yaml
> index cb8182bbc491f..cf4c130382411 100644
> --- a/Documentation/devicetree/bindings/sound/nuvoton,nau8822.yaml
> +++ b/Documentation/devicetree/bindings/sound/nuvoton,nau8822.yaml

[ ... ]

> @@ -30,6 +30,20 @@ properties:
>    clock-names:
>      const: mclk
>  
> +  vdda-supply:
> +    description: Analog power supply
> +
> +  vddb-supply:
> +    description: Digital buffer (input/output) supply
> +
> +  vddc-supply:
> +    description: Digital core supply
> +
> +  vddspk-supply:
> +    description:
> +      Speaker supply (power supply pin for RSPKOUT, LSPKOUT, AUXOUT2 and
> +      AUXTOUT1 output drivers)

[Severity: Low]
This isn't a bug, but there appears to be a typo here. Should this be
AUXOUT1 instead of AUXTOUT1 to match the actual pin and driver definition?

-- 
Sashiko AI review · https://sashiko.dev/#/patchset/20260525-nau8822-reg-v2-0-7d37ae393e46@flipper.net?part=1