RE: [PATCH v3] ASoC: Intel: sof_rt5682: add jsl_rt5650 board config

[Lu, Brent]

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