Re: [PATCH v2 4/4] ASoC: dt-bindings: ti,ts3a227e.yaml: add jack-type

From: Rob Herring
Date: Mon Jan 23 2023 - 17:10:57 EST


On Fri, Jan 20, 2023 at 11:25:54AM +0100, Astrid Rost wrote:
> Add jack-type: Bitmap value of snd_jack_type to allow combining
> card drivers to create a jack for it.
>
> Signed-off-by: Astrid Rost <astrid.rost@xxxxxxxx>
> ---
> Documentation/devicetree/bindings/sound/ti,ts3a227e.yaml | 8 ++++++++
> 1 file changed, 8 insertions(+)
>
> diff --git a/Documentation/devicetree/bindings/sound/ti,ts3a227e.yaml b/Documentation/devicetree/bindings/sound/ti,ts3a227e.yaml
> index 785930658029..1d949b805f98 100644
> --- a/Documentation/devicetree/bindings/sound/ti,ts3a227e.yaml
> +++ b/Documentation/devicetree/bindings/sound/ti,ts3a227e.yaml
> @@ -27,6 +27,14 @@ properties:
> interrupts:
> maxItems: 1
>
> + jack-type:
> + $ref: /schemas/types.yaml#/definitions/uint32
> + description: Bitmap value of snd_jack_type to allow combining

snd_jack_type? Is that a Linux thing? Bindings are independent of Linux.

> + card drivers to create a jack for it. Supported is
> + 1 SND_JACK_HEADPHONE
> + 2 SND_JACK_MICROPHONE
> + default: 3

I'm pretty sure jack properties are more complicated than just headphone
and/or microphone. There's buttons which are detected in differing ways.

Rob