-----Original Message----- From: Takashi Iwai [mailto:tiwai@suse.de] Sent: Tuesday, February 27, 2018 10:25 PM To: Ughreja, Rakesh A rakesh.a.ughreja@intel.com Cc: Pierre-Louis Bossart pierre-louis.bossart@linux.intel.com; alsa-devel@alsa- project.org; broonie@kernel.org; liam.r.girdwood@linux.intel.com; Koul, Vinod vinod.koul@intel.com; Patches Audio patches.audio@intel.com Subject: Re: [alsa-devel] [PATCH v1 9/9] ASoC: Intel: Boards: add support for HDA codecs
On Tue, 27 Feb 2018 17:20:05 +0100, Ughreja, Rakesh A wrote:
The hdac_hda is just a wrapper around the legacy codec driver and so it relies on the functionality of the legacy HDA codec driver for all the functionality including pin re-tasking.
The widget names that you see above is just to complete the DAPM route. Based on your comment I am planning to rename it as following
Analog In Endpoint Analog Output Endpoint Digital In Endpoint Digital Out Endpoint
and will connect it to the Codec Pins.
Also I think it makes sense to rename the codec Pin names accordingly
Codec Analog Input Pin Codec Analog Output Pin Codec Digital Input Pin Codec Digital Output Pin
Humm, what if you have more than one analog input? It's almost as if this list should be created dynamically based on what is exposed by the codec, I don't see how a static list will cover all configurations.
If it is really required it can be done, the codec->pcm_list_head has got entries stored.
But I am not sure what is the behavior of the legacy HDA codec driver when it sees more than one Analog inputs.
Takashi, will I see two Analog entries in the pcm_list_head ?
Yes, in a few cases, the generic parser creates another PCM for analog I/O as "Alt Analog":
But this again is static and its named as "Alt Analog". So, in the current code if I just create one more as "Alt Analog" DAI it should be fine ?
I do map them by searching the pcm_list_head when the application opens them.
Will I see more than one "Analog" and "Alt Analog" ever in the pcm_list_head ? or at max I am going to see one "Analog" and an additional "Alt Analog" PCM.
Regards, Rakesh