RE: [RESEND,v3] ASoC: Intel: boards: add max98390 2/4 speakers support

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



> This is technically not a 'RESEND, v3' since it's different from v3.
> Please don't do this.
> 
> > ---
> > changes in resend v3:
> >   - use .comp_ids to scan acpi support ALC5682I-VD or ALC5682I-VS
> 
> and this indeed shows you modified the code...

Sorry for the confusion. My mistakes... It is not a resend case because I did changes. I will send new one and also fix below you mention.

> 
> > +static const struct snd_soc_acpi_codecs adl_max98390_amp = {
> > +	.num_codecs = 1,
> > +	.codecs = {"MX98390"}
> > +};
> > +
> >  struct snd_soc_acpi_mach snd_soc_acpi_intel_adl_machines[] = {
> >  	{
> >  		.comp_ids = &adl_rt5682_rt5682s_hp, @@ -434,6 +439,22
> @@ struct
> > snd_soc_acpi_mach snd_soc_acpi_intel_adl_machines[] = {
> >  		.sof_fw_filename = "sof-adl.ri",
> >  		.sof_tplg_filename = "sof-adl-nau8825.tplg",
> >  	},
> > +	{
> > +		.comp_ids = &adl_rt5682_rt5682s_hp,
> > +		.drv_name = "adl_max98390_rt5682",
> > +		.machine_quirk = snd_soc_acpi_codec_list,
> > +		.quirk_data = &adl_max98390_amp,
> > +		.sof_fw_filename = "sof-adl.ri",
> > +		.sof_tplg_filename = "sof-adl-max98390-rt5682.tplg",
> > +	},
> > +	{
> > +		.comp_ids = &adl_rt5682_rt5682s_hp,
> > +		.drv_name = "adl_mx98390_rt5682s",
> > +		.machine_quirk = snd_soc_acpi_codec_list,
> > +		.quirk_data = &adl_max98390_amp,
> > +		.sof_fw_filename = "sof-adl.ri",
> > +		.sof_tplg_filename = "sof-adl-max98390-rt5682.tplg",
> > +	},
> 
> I don't follow the last two entries.. They use
> 
> the same 'comp_ids':
> 
> static const struct snd_soc_acpi_codecs adl_rt5682_rt5682s_hp = {
> 	.num_codecs = 2,
> 	.codecs = {"10EC5682", "RTL5682"},
> };
> 
> the same machine quirk:
> snd_soc_acpi_codec_list,
> 
> the same quirk_data:
> &adl_max98390_amp,
> 
> but a different driver.
> .drv_name = "adl_max98390_rt5682",
> .drv_name = "adl_mx98390_rt5682s",
> 
> it looks like the second entry will never be selected.
> 
> Can you please explain how this works if it's intentional or double-check if this
> is correct. I think it's a copy-paste from the case where the amplifier is
> different, but here only the headset codec is different.
Yes, you are right! Here is only hp difference. I don't have to copy two.
With single drv_name and remove the redundant entry in sof_rt5682 board_id table.




[Index of Archives]     [ALSA User]     [Linux Audio Users]     [Pulse Audio]     [Kernel Archive]     [Asterisk PBX]     [Photo Sharing]     [Linux Sound]     [Video 4 Linux]     [Gimp]     [Yosemite News]

  Powered by Linux