> 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.