On Wed, Oct 6, 2021 at 7:23 PM Brent Lu <brent.lu@xxxxxxxxx> wrote: > > Use the id_alt field to enumerate rt5682s headphone codec and remove > redundant entries in tables. ... > +static struct snd_soc_acpi_codecs adl_rt5682s_hp = { > + .num_codecs = 1, > + .codecs = {"RTL5682"} Keeping commas in non-terminator entries is always good to avoid churn in the future. > +}; ... > +static struct snd_soc_acpi_codecs rt5682s_hp = { > + .num_codecs = 1, > + .codecs = {"RTL5682"} Ditto. > +}; ... > +static struct snd_soc_acpi_codecs rt5682s_hp = { > + .num_codecs = 1, > + .codecs = {"RTL5682"} Ditto. > +}; ... > +static struct snd_soc_acpi_codecs rt5682s_hp = { > + .num_codecs = 1, > + .codecs = {"RTL5682"} Ditto. > +}; ... > +static struct snd_soc_acpi_codecs rt5682s_hp = { > + .num_codecs = 1, > + .codecs = {"RTL5682"} Ditto. > +}; ...and so on. -- With Best Regards, Andy Shevchenko