Message-ID: <0b0c0030-3587-5501-c7b2-eccbbe4551e0@axis.com>
Date:   Mon, 23 Jan 2023 09:39:29 +0100
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101
 Thunderbird/102.6.0
Subject: Re: [PATCH v2 4/4] ASoC: dt-bindings: ti,ts3a227e.yaml: add jack-type
Content-Language: en-US
To:     Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>,
        Astrid Rost <astrid.rost@axis.com>,
        Mark Brown <broonie@kernel.org>,
        Liam Girdwood <lgirdwood@gmail.com>,
        Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>,
        Rob Herring <robh+dt@kernel.org>,
        Dylan Reid <dgreid@chromium.org>
CC:     <kernel@axis.com>, <alsa-devel@alsa-project.org>,
        <linux-kernel@vger.kernel.org>, <devicetree@vger.kernel.org>
References: <20230120102555.1523394-1-astrid.rost@axis.com>
 <20230120102555.1523394-5-astrid.rost@axis.com>
 <2d05a943-3510-5ee9-9906-247a6344190a@linaro.org>
From:   Astrid Rost <astridr@axis.com>
In-Reply-To: <2d05a943-3510-5ee9-9906-247a6344190a@linaro.org>
Content-Type: text/plain; charset=UTF-8; format=flowed
Content-Transfer-Encoding: 7bit
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?aU00amVSU25SYWZJZTNZakRrS3ZRd0htMmJwWEhSRkw1REQvaUZ3YmZkRjhp?=
 =?utf-8?B?dGZIcVdvN0hKVWZiR3pCWUt4NzcveFpGdkJUT1gyZkpoMVBNSExwd2FTWG15?=
 =?utf-8?B?eUJ5dlhVOGpXN1VHbDFEUVhzR3pxcGJ2bHlWdHNNSlF2S29HOWFkTmxWbHg2?=
 =?utf-8?B?VEhIUUFBNTFDS1lwSFlNZ2thYWU5TlFsQWdQd0FIQ1pyOEJRYlkvdDVMOG5Q?=
 =?utf-8?B?NW4yeHVSSHd6NVEwQkFhRXFpZ2REbWF2YmRvdVpuT3RWVUM1MVpHckkvTWlX?=
 =?utf-8?B?OUFJbDhseUpEQk8wQi9YTkx2aW4xeUFNUjd2RXhmdGpRWFpvQ1JmVldxeTJ4?=
 =?utf-8?B?ZXhoOXRqbXBTVUptYSs3VU44UUxKWlJSdDVEWnIrLzJCdmVzZ094K3Q2VkZP?=
 =?utf-8?B?YlEzTks4aDVmaSt1Y3J3UVByTjEvN0VzQ0JpOUFTTzRIMG9qdlhqY0lYSWpQ?=
 =?utf-8?B?R1pHM3FLNkNnTzZaT1hUUU02RDMxalA0UDVPTkorU3QvOGIycHdEYnBNeHAr?=
 =?utf-8?B?VjRBR2NiaXgzOVhFYmlGMzllNytNcTNMVWJQLzZ2UFlDODdaSFpXamJrZXZa?=
 =?utf-8?B?ejF4SlM1bmdpcEt5K216dzZGSktVRjN5NWtzVzd3dC9xV3JHeUpUaUROMXFx?=
 =?utf-8?B?elNjNGVJU2VZS2dNd2JST2c1djVVVlhONUVJMWxOMTZUMWdQYlFtdWJEMHpm?=
 =?utf-8?B?QmpZQW91MEVTeVhTMk05L1RhR2lYUEpkaXRUR1QzT05YOGQyRlBkZnpMN0Yz?=
 =?utf-8?B?SVBvbVBDMXBpUysweWJkTHczQm1VSmhFNVZlRWNIUHBoUUZPU2lNSndGZVVD?=
 =?utf-8?B?Z28zSUg5NlgzclhpY0dkNUhSdEdTb3FaRkhmTW9LakJ2dVBGcC9IRjF0Qmor?=
 =?utf-8?B?dDVvaHlQWExVNUF2R083YnR3bG56dGNJRGhWamlJL1lzdHVsVFJidGRQN01P?=
 =?utf-8?B?Wk9iMjIzVU9wdjdxY3lBOWMzY3A3ZFBlOEZmZVBzRHdxdVFTbjF3ODB1QmtC?=
 =?utf-8?B?L3g3K29LRlZaYUVVdFdtdmw3ZE9JbmliVlAvVEF5dU1Ma2JFZDBQcEZlL2ls?=
 =?utf-8?B?T2ZMMjBZM3kyc1FGVHZrUXZyTS94US9Gd2RrK0Q0aTFqZDhMclFQVXZtNlFY?=
 =?utf-8?B?d04ySGs1a2M1dG9UUnhmdlVzVFBwWEFmdlFNaTBDd0tDaTN5ZFdneTR2dE1k?=
 =?utf-8?B?VmFySXl0VmxuaFVpcmliMEZhN0Q0dDJXWkwrMlp3YXptZDVHNENZUUZvNmZ0?=
 =?utf-8?B?dytLaEdwNmtZci9vVzFQMUd1NzlRNWVKU296ZU5EclFEa0F3R2dmRFd3ditE?=
 =?utf-8?B?T2tnQy93TmM0anZHMFdueVMyc3hiT0xYMVkrakZWeGp3eVlydzdRcUVLUmZI?=
 =?utf-8?B?U3liV0FUK1gzWEZ5OFYxVFBTOHNKMWJocVJubGxEZ1VTdlZHaWhQSmF2K2R6?=
 =?utf-8?B?dnF6eTEyUDU5MzMzaExyR3FhdXFlVHliSVZVUkwwVlRCMkplMUpKbDhRcTJk?=
 =?utf-8?B?K2pWK0gwc0JEbmtXdjE0MkNXczlvTUZmd0orU2ZGUUdLQUxIVzdySGxEbnhK?=
 =?utf-8?B?bzdxYnBWUXBmcnFFSjFqU3Fzc2NxZkVaSTZNcGpzTys0dSs2YlQxV2ZLaDBz?=
 =?utf-8?B?NG1KWitQOE5rbXZuSS9VL3VpdnVJNDJpMjhYSC92ZllwUlkzemZIYlhaMlEw?=
 =?utf-8?B?NUdMMUY2RmROQThGQ2N4ekRmTjRTMmJvUEcwaXhWazVBbE16Q0UzTHp2WGZ4?=
 =?utf-8?B?ZnJGUTFNbkZwYk9jMEw1YjdON1FuQko0SXBjZ0tkaXRTL2VoRGptbytIY0Rs?=
 =?utf-8?B?REpKSkQwY3h2MS9WKzdOUEQzaERoZGFQWUphRDN5cDdCRGlha3NtN1BTVXlh?=
 =?utf-8?B?SGthMHNjY3VSUnJWWXhNMkFYNFNXaG9xQXBpTGRyNS9LVHZXNERjZjBHWEFn?=
 =?utf-8?B?SEgweVdhM0U4ZkRURjNEYnQ3ZklBeXZXRVBYOEFXTmRqdDJaMTVoRU1UUXVi?=
 =?utf-8?B?Zk5uakl6OVJ6WENCOXZiREoxRmhpQXJEbnN1aU13TFhuVWN3YmpBbFNrQng5?=
 =?utf-8?B?ZlBIQjZVZXdmaVExeisyTFhqSXZtelZiTG8yV1VMVHl5NURXY3l5OFMxY0FO?=
 =?utf-8?Q?iUr8tS7Oa9lEPmPRCJoHEVXnP?=
X-MS-Exchange-CrossTenant-Network-Message-Id: be671f4f-2cef-4e89-a0f9-08dafd1d58e8
X-MS-Exchange-CrossTenant-AuthSource: DU0PR02MB9467.eurprd02.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 23 Jan 2023 08:39:31.5115
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 78703d3c-b907-432f-b066-88f7af9ca3af
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: AlJBUDdw+YsmsbWCTDacJsyesbCPu36xakCaooQZRC4GeEkbwdjpqTeTGbkor51dSWUnCarC2tspFqhrniC6qg==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: DU2PR02MB10332
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

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@axis.com>
>> ---
>>   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