2023-01-20 10:53:14

by Astrid Rost

[permalink] [raw]
Subject: [PATCH v2 0/4] ASoC: simple-card-utils: create jack inputs for aux_devs

Add a generic way to create jack inputs for auxiliary jack detection
drivers (e.g. via i2c, spi), which are not part of any real codec.
The simple-card can be used as combining card driver to add the jacks,
no new one is required.

Create a jack (for input-events) for jack devices in the auxiliary
device list (aux_devs). A device which returns a valid value on
get_jack_type counts as jack device; set_jack is required
to add the jack to the device.

v1 -> v2: Auxiliary jack drivers return the correct snd_jack_type with
the function: get_jack_type.
- No devicetree changes in simple-card.
- soc-component: changed name to: get_jack_type.
- simple-jack-utils: updated algorithm to add jack devices.
A device which returns a valid value on get_jack_type counts as jack
device
- ts3a227e: added devicetree property jack-type, added NULL check for jack.

Astrid Rost (4):
[PATCH v2 1/4] ASoC: soc-component: add get_jack_type
[PATCH v2 2/4] ASoC: simple-card-utils: create jack inputs for aux_devs
[PATCH v2 3/4] ASoC: ts3a227e: add set_jack and get_jack_type
[PATCH v2 4/4] ASoC: dt-bindings: ti,ts3a227e.yaml: add jack-type

.../bindings/sound/ti,ts3a227e.yaml | 8 +++
include/sound/simple_card_utils.h | 3 ++
include/sound/soc-component.h | 2 +
sound/soc/codecs/ts3a227e.c | 29 ++++++++++-
sound/soc/generic/simple-card-utils.c | 49 +++++++++++++++++++
sound/soc/generic/simple-card.c | 4 ++
sound/soc/soc-component.c | 20 ++++++++
7 files changed, 114 insertions(+), 1 deletion(-)

--
2.30.2


2023-01-20 10:57:46

by Astrid Rost

[permalink] [raw]
Subject: [PATCH v2 2/4] ASoC: simple-card-utils: create jack inputs for aux_devs

Add a generic way to create jack inputs for auxiliary jack detection
drivers (e.g. via i2c, spi), which are not part of any real codec.
The simple-card can be used as combining card driver to add the jacks,
no new one is required.

Create a jack (for input-events) for jack devices in the auxiliary
device list (aux_devs). A device which returns a valid value on
get_jack_type counts as jack device; set_jack is required
to add the jack to the device.

Signed-off-by: Astrid Rost <[email protected]>
---
include/sound/simple_card_utils.h | 3 ++
sound/soc/generic/simple-card-utils.c | 49 +++++++++++++++++++++++++++
sound/soc/generic/simple-card.c | 4 +++
3 files changed, 56 insertions(+)

diff --git a/include/sound/simple_card_utils.h b/include/sound/simple_card_utils.h
index 38590f1ae9ee..a3f3f3aa9e6e 100644
--- a/include/sound/simple_card_utils.h
+++ b/include/sound/simple_card_utils.h
@@ -69,6 +69,7 @@ struct asoc_simple_priv {
} *dai_props;
struct asoc_simple_jack hp_jack;
struct asoc_simple_jack mic_jack;
+ struct snd_soc_jack *aux_jacks;
struct snd_soc_dai_link *dai_link;
struct asoc_simple_dai *dais;
struct snd_soc_dai_link_component *dlcs;
@@ -187,6 +188,8 @@ int asoc_simple_parse_pin_switches(struct snd_soc_card *card,
int asoc_simple_init_jack(struct snd_soc_card *card,
struct asoc_simple_jack *sjack,
int is_hp, char *prefix, char *pin);
+int asoc_simple_init_aux_jacks(struct asoc_simple_priv *priv,
+ char *prefix);
int asoc_simple_init_priv(struct asoc_simple_priv *priv,
struct link_info *li);
int asoc_simple_remove(struct platform_device *pdev);
diff --git a/sound/soc/generic/simple-card-utils.c b/sound/soc/generic/simple-card-utils.c
index e35becce9635..56552a616f21 100644
--- a/sound/soc/generic/simple-card-utils.c
+++ b/sound/soc/generic/simple-card-utils.c
@@ -786,6 +786,55 @@ int asoc_simple_init_jack(struct snd_soc_card *card,
}
EXPORT_SYMBOL_GPL(asoc_simple_init_jack);

+int asoc_simple_init_aux_jacks(struct asoc_simple_priv *priv, char *prefix)
+{
+ struct snd_soc_card *card = simple_priv_to_card(priv);
+ struct snd_soc_component *component;
+ int found_jack_index = 0;
+ int type = 0;
+ int num = 0;
+ int ret;
+
+ if (priv->aux_jacks)
+ return 0;
+
+ for_each_card_auxs(card, component) {
+ type = snd_soc_component_get_jack_type(component);
+ if (type > 0)
+ num++;
+ }
+ if (num < 1)
+ return 0;
+
+ priv->aux_jacks = devm_kcalloc(card->dev, num,
+ sizeof(struct snd_soc_jack), GFP_KERNEL);
+ if (!priv->aux_jacks)
+ return -ENOMEM;
+
+ for_each_card_auxs(card, component) {
+ char id[128];
+ struct snd_soc_jack *jack;
+
+ if (found_jack_index >= num)
+ break;
+
+ type = snd_soc_component_get_jack_type(component);
+ if (type <= 0)
+ continue;
+
+ /* create jack */
+ jack = &(priv->aux_jacks[found_jack_index++]);
+ snprintf(id, sizeof(id), "%s-jack", component->name);
+ ret = snd_soc_card_jack_new(card, id, type, jack);
+ if (ret)
+ continue;
+
+ (void)snd_soc_component_set_jack(component, jack, NULL);
+ }
+ return 0;
+}
+EXPORT_SYMBOL_GPL(asoc_simple_init_aux_jacks);
+
int asoc_simple_init_priv(struct asoc_simple_priv *priv,
struct link_info *li)
{
diff --git a/sound/soc/generic/simple-card.c b/sound/soc/generic/simple-card.c
index feb55b66239b..e98932c16754 100644
--- a/sound/soc/generic/simple-card.c
+++ b/sound/soc/generic/simple-card.c
@@ -623,6 +623,10 @@ static int simple_soc_probe(struct snd_soc_card *card)
if (ret < 0)
return ret;

+ ret = asoc_simple_init_aux_jacks(priv, PREFIX);
+ if (ret < 0)
+ return ret;
+
return 0;
}

--
2.30.2

2023-01-20 11:15:02

by Astrid Rost

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

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 <[email protected]>
---
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
+ card drivers to create a jack for it. Supported is
+ 1 SND_JACK_HEADPHONE
+ 2 SND_JACK_MICROPHONE
+ default: 3
+
ti,micbias:
$ref: /schemas/types.yaml#/definitions/uint32
description: Intended MICBIAS voltage (datasheet section 9.6.7).
--
2.30.2

2023-01-22 14:16:49

by Krzysztof Kozlowski

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

On 20/01/2023 11:25, Astrid Rost wrote:
> Add jack-type: Bitmap value of snd_jack_type to allow combining
> card drivers to create a jack for it.

Subject: drop "yaml". We do not filename extensions to subject prefix.
Nowhere.

>
> Signed-off-by: Astrid Rost <[email protected]>
> ---
> 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
> + card drivers to create a jack for it. Supported is

Why the device would once support (allow) headphone and once not? Device
either always supports them or never...


> + 1 SND_JACK_HEADPHONE
> + 2 SND_JACK_MICROPHONE

minimum and maximum

> + default: 3
> +
> ti,micbias:
> $ref: /schemas/types.yaml#/definitions/uint32
> description: Intended MICBIAS voltage (datasheet section 9.6.7).

Best regards,
Krzysztof


2023-01-23 08:39:43

by Astrid Rost

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

Hello Krzysztof,

On 1/22/23 15:16, Krzysztof Kozlowski wrote:
> On 20/01/2023 11:25, Astrid Rost wrote:
>> Add jack-type: Bitmap value of snd_jack_type to allow combining
>> card drivers to create a jack for it.
>
> Subject: drop "yaml". We do not filename extensions to subject prefix.
> Nowhere.
>

yes, true.

>>
>> Signed-off-by: Astrid Rost <[email protected]>
>> ---
>> 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
>> + card drivers to create a jack for it. Supported is
>
> Why the device would once support (allow) headphone and once not? Device
> either always supports them or never...
>

If a device has two connectors (pink and green), one for the microphone
and one for the headset. It would be easier to see from the available
events, which is which. But of course it is possible to give it good names.
My first approach was, that it returned all supported types, so no
devicetree change needed. But by colleges agreed that it would be nice
to remove unused flags. I am happy to remove it and someone who requires
it can add it.

>> + 1 SND_JACK_HEADPHONE
>> + 2 SND_JACK_MICROPHONE
>
> minimum and maximum

I do not understand this? It is a bitmap. I can put it as an
enum:
- 1 # SND_JACK_HEADPHONE
- 2 # SND_JACK_MICROPHONE
- 3 # SND_JACK_HEADPHONE | SND_JACK_MICROPHONE
>
>> + default: 3
>> +
>> ti,micbias:
>> $ref: /schemas/types.yaml#/definitions/uint32
>> description: Intended MICBIAS voltage (datasheet section 9.6.7).
>
> Best regards,
> Krzysztof
>

Thank you for your comments

Astrid

2023-01-23 09:05:22

by Krzysztof Kozlowski

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

On 23/01/2023 09:39, Astrid Rost wrote:
> Hello Krzysztof,
>
> On 1/22/23 15:16, Krzysztof Kozlowski wrote:
>> On 20/01/2023 11:25, Astrid Rost wrote:
>>> Add jack-type: Bitmap value of snd_jack_type to allow combining
>>> card drivers to create a jack for it.
>>
>> Subject: drop "yaml". We do not filename extensions to subject prefix.
>> Nowhere.
>>
>
> yes, true.
>
>>>
>>> Signed-off-by: Astrid Rost <[email protected]>
>>> ---
>>> 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
>>> + card drivers to create a jack for it. Supported is
>>
>> Why the device would once support (allow) headphone and once not? Device
>> either always supports them or never...
>>
>
> If a device has two connectors (pink and green), one for the microphone
> and one for the headset.

We talk about "ts3a227" here, which has always two connectors (pins)...
unless you refer to the case when these are e.g. grounded?


> It would be easier to see from the available
> events, which is which. But of course it is possible to give it good names.
> My first approach was, that it returned all supported types, so no
> devicetree change needed. But by colleges agreed that it would be nice
> to remove unused flags. I am happy to remove it and someone who requires
> it can add it.
>
>>> + 1 SND_JACK_HEADPHONE
>>> + 2 SND_JACK_MICROPHONE
>>
>> minimum and maximum
>
> I do not understand this? It is a bitmap. I can put it as an
> enum:
> - 1 # SND_JACK_HEADPHONE
> - 2 # SND_JACK_MICROPHONE
> - 3 # SND_JACK_HEADPHONE | SND_JACK_MICROPHONE


Then maximum is OR of them, isn't it?

Best regards,
Krzysztof


2023-01-23 09:19:49

by Astrid Rost

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

Hello,

On 1/23/23 10:05, Krzysztof Kozlowski wrote:
> On 23/01/2023 09:39, Astrid Rost wrote:
>> Hello Krzysztof,
>>
>> On 1/22/23 15:16, Krzysztof Kozlowski wrote:
>>> On 20/01/2023 11:25, Astrid Rost wrote:
>>>> Add jack-type: Bitmap value of snd_jack_type to allow combining
>>>> card drivers to create a jack for it.
>>>
>>> Subject: drop "yaml". We do not filename extensions to subject prefix.
>>> Nowhere.
>>>
>>
>> yes, true.
>>
>>>>
>>>> Signed-off-by: Astrid Rost <[email protected]>
>>>> ---
>>>> 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
>>>> + card drivers to create a jack for it. Supported is
>>>
>>> Why the device would once support (allow) headphone and once not? Device
>>> either always supports them or never...
>>>
>>
>> If a device has two connectors (pink and green), one for the microphone
>> and one for the headset.
>
> We talk about "ts3a227" here, which has always two connectors (pins)...
> unless you refer to the case when these are e.g. grounded?
>

yes, that is what I meant.
I push a version, where I remove this.

>
>> It would be easier to see from the available
>> events, which is which. But of course it is possible to give it good names.
>> My first approach was, that it returned all supported types, so no
>> devicetree change needed. But by colleges agreed that it would be nice
>> to remove unused flags. I am happy to remove it and someone who requires
>> it can add it.
>>
>>>> + 1 SND_JACK_HEADPHONE
>>>> + 2 SND_JACK_MICROPHONE
>>>
>>> minimum and maximum
>>
>> I do not understand this? It is a bitmap. I can put it as an
>> enum:
>> - 1 # SND_JACK_HEADPHONE
>> - 2 # SND_JACK_MICROPHONE
>> - 3 # SND_JACK_HEADPHONE | SND_JACK_MICROPHONE
>
>
> Then maximum is OR of them, isn't it?

yes 1-3
>
> Best regards,
> Krzysztof
>

Astrid

2023-01-23 22:10:56

by Rob Herring

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

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 <[email protected]>
> ---
> 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

2023-01-24 08:46:33

by Astrid Rost

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

Hello Rob,

On 1/23/23 23:09, Rob Herring wrote:
> 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 <[email protected]>
>> ---
>> 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.

Yes, you are right. On the buttons, someone would like to have an
keymapping like KEY_VOLUMEDOWN, KEY_VOLUMEUP for the different buttons
SND_JACK_BTN_X. This is always added to the jack by the ts3a227e driver.

I pushed yesterday a v3, which returns all supported jack types by this
device. I am happy with this.

>
> Rob

Best regards

Astrid