Re: [PATCH] ASoC: core: Change device numbering

[Jaroslav Kysela <perex@perex.cz>]

On 11. 02. 25 11:53, Amadeusz Sławiński wrote:
> On 1/31/2025 5:18 PM, Jaroslav Kysela wrote:
>> On 31. 01. 25 13:41, Amadeusz Sławiński wrote:
>>
>>> So, as I understand you want something like the following and to
>>> differentiate cards in UCM using component?
>>
>>> - card: 0
>>>      id: avshdaudio
>>>      name: avs_hdaudio
>>>      longname: LENOVO-20KH006LPB-ThinkPadX1Carbon6th
>>>      driver_name: avs_hdaudio
>>>      mixer_name: Realtek ALC285
>>>      components: HDA:10ec0285,17aa225c,00100002
>>
>> The short name (name:) may be better like 'AVS HD-Audio' (almost all
>> ASoC drivers suffers here), but the proposed driver_name is fine from my
>> view.
>>
> 
> We had some more discussions, what about something like:
> 
> card X: HDAudio [AVS HD-Audio], device X: HDAudio Analog () []
> card X: HDMI [AVS HDMI], device X: HDMI 1 () []
> card X: ALC274 [AVS I2S ALC274], device X: Audio () []
> 
> - card: X
>     id: HDAudio
>     name: AVS HD-Audio
>     longname: LENOVO-20KH006LPB-ThinkPadX1Carbon6th
>     driver_name: avs_hdaudio
>     mixer_name: Realtek ALC285
>     components: HDA:10ec0285,17aa225c,00100002
>     controls_count: 14
> 
> - card: X
>     id: HDMI
>     name: AVS HDMI
>     longname: LENOVO-20KH006LPB-ThinkPadX1Carbon6th
>     driver_name: avs_hdaudio
>     mixer_name: Intel Kabylake HDMI
>     components: HDA:8086280b,80860101,00100000
>     controls_count: 21
> 
> - card: X
>     id: ALC274
>     name: AVS I2S ALC274
>     longname:
> IntelCorporation-CannonLakeClientPlatform-0.1-CannonLakeYLPDDR4RVP
>     driver_name: avs_rt274
>     mixer_name:
>     components:
>     controls_count: 17
> 
> it feels like it is more descriptive.

It looks much better for driver name and card (short) name. It also shows, how 
the DMI auto-generated long names are really broken and confusing for users.

				Jaroslav

-- 
Jaroslav Kysela <perex@perex.cz>
Linux Sound Maintainer; ALSA Project; Red Hat, Inc.