> > + { > > + .id = "10EC5650", > > + .drv_name = "jsl_rt5650", > > + .machine_quirk = snd_soc_acpi_codec_list, > > + .quirk_data = &rt5650_spk, > > + .sof_tplg_filename = "sof-jsl-rt5650-dual.tplg", > > what does 'dual' mean in this context? it's not something we've used before > so a comment would be needed IMHO. > > two DAIs, two codecs? Dual I2S. Should we remove it? Just sof-jsl-rt5650.tplg? Regards, Brent