Re: [PATCH v2 1/3] ASoC: dt-bindings: everest,es8316: Document routing strings

[Binbin Zhou <zhoubinbin@xxxxxxxxxxx>]

On 2025/9/4 00:01, Jihed Chaibi wrote:
> Add a list of the es8316 pin names to the binding's description to make
> it self-contained and improve the user experience for board developers.
>
> Signed-off-by: Jihed Chaibi <jihed.chaibi.dev@xxxxxxxxx>


Reviewed-by: Binbin Zhou <zhoubinbin@xxxxxxxxxxx>

> ---
> Changes in v2:
>   - Rephrased description to refer to device pins instead of the
>     driver.
> ---
>   .../bindings/sound/everest,es8316.yaml           | 16 ++++++++++++++++
>   1 file changed, 16 insertions(+)
>
> diff --git a/Documentation/devicetree/bindings/sound/everest,es8316.yaml b/Documentation/devicetree/bindings/sound/everest,es8316.yaml
> index e4b2eb5fae2..81a0215050e 100644
> --- a/Documentation/devicetree/bindings/sound/everest,es8316.yaml
> +++ b/Documentation/devicetree/bindings/sound/everest,es8316.yaml
> @@ -12,6 +12,22 @@ maintainers:
>     - Matteo Martelli <matteomartelli3@xxxxxxxxx>
>     - Binbin Zhou <zhoubinbin@xxxxxxxxxxx>
>   
> +description: |
> +  Everest ES8311, ES8316 and ES8323 audio CODECs
> +
> +  Pins on the device (for linking into audio routes):
> +
> +    Outputs:
> +      * LOUT:     Left Analog Output
> +      * ROUT:     Right Analog Output
> +      * MICBIAS:  Microphone Bias
> +
> +    Inputs:
> +      * MIC1P:    Microphone 1 Positive Analog Input
> +      * MIC1N:    Microphone 1 Negative Analog Input
> +      * MIC2P:    Microphone 2 Positive Analog Input
> +      * MIC2N:    Microphone 2 Negative Analog Input
> +
>   allOf:
>     - $ref: dai-common.yaml#
>   

Thanks.
Binbin