The code is fine but this patch is problematic on three counts: a) you didn't Cc: maintainers b) by sending this directly, you lost Bard and my Reviewed-by tags c) the timing is bad, we don't send patches during the merge window (unless they are critical fixes) On 11/2/23 11:48, Mac Chiang wrote: > RT5650 is I2S codec integrated with HP and SPK. > The HW board connects SoC I2S to RT5650 codec as below: > > I2S0: ALC5650 aif1 for Speaker > I2S2: ALC5650 aif2 for Headphone > > Signed-off-by: Mac Chiang <mac.chiang@xxxxxxxxx> > --- > pick up from Link: https://github.com/thesofproject/linux/pull/4661 > for Chrome MTL projects. > > Changes v2: > - regen format patch due to v1 from github remote > --- > sound/soc/intel/boards/sof_rt5682.c | 9 +++++++++ > sound/soc/intel/common/soc-acpi-intel-mtl-match.c | 12 ++++++++++++ > 2 files changed, 21 insertions(+) > > diff --git a/sound/soc/intel/boards/sof_rt5682.c b/sound/soc/intel/boards/sof_rt5682.c > index 06ad15af46de..9723479f43da 100644 > --- a/sound/soc/intel/boards/sof_rt5682.c > +++ b/sound/soc/intel/boards/sof_rt5682.c > @@ -1147,6 +1147,15 @@ static const struct platform_device_id board_ids[] = { > SOF_RT5682_SSP_AMP(0) | > SOF_RT5682_NUM_HDMIDEV(3)), > }, > + { > + .name = "mtl_rt5650", > + .driver_data = (kernel_ulong_t)(SOF_RT5682_MCLK_EN | > + SOF_RT5682_SSP_CODEC(2) | > + SOF_RT5682_SSP_AMP(0) | > + SOF_RT5682_NUM_HDMIDEV(3) | > + SOF_BT_OFFLOAD_SSP(1) | > + SOF_SSP_BT_OFFLOAD_PRESENT), > + }, > { } > }; > MODULE_DEVICE_TABLE(platform, board_ids); > diff --git a/sound/soc/intel/common/soc-acpi-intel-mtl-match.c b/sound/soc/intel/common/soc-acpi-intel-mtl-match.c > index 301b8142d554..af4224bff718 100644 > --- a/sound/soc/intel/common/soc-acpi-intel-mtl-match.c > +++ b/sound/soc/intel/common/soc-acpi-intel-mtl-match.c > @@ -40,6 +40,11 @@ static const struct snd_soc_acpi_codecs mtl_lt6911_hdmi = { > .codecs = {"INTC10B0"} > }; > > +static const struct snd_soc_acpi_codecs mtl_rt5650_amp = { > + .num_codecs = 1, > + .codecs = {"10EC5650"} > +}; > + > struct snd_soc_acpi_mach snd_soc_acpi_intel_mtl_machines[] = { > { > .comp_ids = &mtl_rt5682_rt5682s_hp, > @@ -77,6 +82,13 @@ struct snd_soc_acpi_mach snd_soc_acpi_intel_mtl_machines[] = { > SND_SOC_ACPI_TPLG_INTEL_SSP_MSB | > SND_SOC_ACPI_TPLG_INTEL_DMIC_NUMBER, > }, > + { > + .id = "10EC5650", > + .drv_name = "mtl_rt5650", > + .machine_quirk = snd_soc_acpi_codec_list, > + .quirk_data = &mtl_rt5650_amp, > + .sof_tplg_filename = "sof-mtl-rt5650.tplg", > + }, > /* place amp-only boards in the end of table */ > { > .id = "INTC10B0",