Linux Sound subsystem development
 help / color / mirror / Atom feed
From: "Amadeusz Sławiński" <amadeuszx.slawinski@linux.intel.com>
To: Jaroslav Kysela <perex@perex.cz>,
	Cezary Rojewski <cezary.rojewski@intel.com>,
	Takashi Iwai <tiwai@suse.com>, Mark Brown <broonie@kernel.org>
Cc: linux-sound@vger.kernel.org
Subject: Re: [PATCH] ASoC: core: Change device numbering
Date: Tue, 11 Feb 2025 11:53:41 +0100	[thread overview]
Message-ID: <4c46ebfa-2892-462f-9f6f-98bec8620e5a@linux.intel.com> (raw)
In-Reply-To: <675dba40-e559-45bf-b9ec-65f4d4c60923@perex.cz>

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.


  reply	other threads:[~2025-02-11 10:53 UTC|newest]

Thread overview: 15+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2025-01-27 14:44 [PATCH] ASoC: core: Change device numbering Amadeusz Sławiński
2025-01-27 14:45 ` Amadeusz Sławiński
2025-01-27 14:54   ` Jaroslav Kysela
2025-01-29  9:25     ` Cezary Rojewski
2025-01-29 14:51       ` Amadeusz Sławiński
2025-01-31  9:03         ` Jaroslav Kysela
2025-01-31 12:41           ` Amadeusz Sławiński
2025-01-31 16:18             ` Jaroslav Kysela
2025-02-11 10:53               ` Amadeusz Sławiński [this message]
2025-02-11 13:30                 ` Jaroslav Kysela
2025-02-11 13:33                   ` Takashi Iwai
2025-02-11 13:56                   ` Amadeusz Sławiński
2025-02-11 14:50                     ` Jaroslav Kysela
2025-02-11 15:12                       ` Amadeusz Sławiński
2025-02-12 13:39                         ` Jaroslav Kysela

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=4c46ebfa-2892-462f-9f6f-98bec8620e5a@linux.intel.com \
    --to=amadeuszx.slawinski@linux.intel.com \
    --cc=broonie@kernel.org \
    --cc=cezary.rojewski@intel.com \
    --cc=linux-sound@vger.kernel.org \
    --cc=perex@perex.cz \
    --cc=tiwai@suse.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox