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

[Jaroslav Kysela <perex@xxxxxxxx>]

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@xxxxxxxx>
Linux Sound Maintainer; ALSA Project; Red Hat, Inc.