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

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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 <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
+      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



[Index of Archives]     [ALSA User]     [Linux Audio Users]     [Pulse Audio]     [Kernel Archive]     [Asterisk PBX]     [Photo Sharing]     [Linux Sound]     [Video 4 Linux]     [Gimp]     [Yosemite News]

  Powered by Linux