diff mbox series

[RESEND,v3] ASoC: Intel: sof_rt5682: add mtl_rt5650 support

Message ID 20231113115907.18539-1-mac.chiang@intel.com (mailing list archive)
State Accepted
Commit a55ea47bb8743ee044550c5dfdd3cb3147602e05
Headers show
Series [RESEND,v3] ASoC: Intel: sof_rt5682: add mtl_rt5650 support | expand

Commit Message

Chiang, Mac Nov. 13, 2023, 11:59 a.m. UTC 
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

Reviewed-by: Bard Liao <yung-chuan.liao@linux.intel.com>
Signed-off-by: Mac Chiang <mac.chiang@intel.com>
---
Hi Pierre, I resend the patch for your review and cc maintainers. Thanks.
Pick up for Chrome MTL projects support.

Changes v3:
  - missing Reviewed-by, add it.
---

 sound/soc/intel/boards/sof_rt5682.c               |  9 +++++++++
 sound/soc/intel/common/soc-acpi-intel-mtl-match.c | 12 ++++++++++++
 2 files changed, 21 insertions(+)

Comments

Pierre-Louis Bossart Nov. 13, 2023, 2:40 p.m. UTC | #1 
On 11/13/23 05:59, 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
> 
> Reviewed-by: Bard Liao <yung-chuan.liao@linux.intel.com>
> Signed-off-by: Mac Chiang <mac.chiang@intel.com>
> ---
> Hi Pierre, I resend the patch for your review and cc maintainers. Thanks.
> Pick up for Chrome MTL projects support.

The code was submitted as
https://github.com/thesofproject/linux/pull/4661, approved and merged in
the topic/sof-dev branch. By submitting the same code multiple times
using two different paths, you are just adding more confusion and noise
for what should be a straightforward process on a minor change.

Moving forward, please pick one of:
a) submit to alsa-devel
b) submit to the SOF GitHub and let SOF maintainers upstream.

Thank you.

Acked-by: Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com>

> 
> Changes v3:
>   - missing Reviewed-by, add it.
> ---
> 
>  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",
Chiang, Mac Nov. 14, 2023, 7:36 a.m. UTC | #2 
> -----Original Message-----
> From: Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com>
> Sent: Monday, November 13, 2023 10:41 PM
> To: Chiang, Mac <mac.chiang@intel.com>; alsa-devel@alsa-project.org
> Cc: Rojewski, Cezary <cezary.rojewski@intel.com>;
> liam.r.girdwood@linux.intel.com; peter.ujfalusi@linux.intel.com; yung-
> chuan.liao@linux.intel.com; ranjani.sridharan@linux.intel.com;
> kai.vehmanen@linux.intel.com; broonie@kernel.org; perex@perex.cz;
> tiwai@suse.com
> Subject: Re: [RESEND,v3] ASoC: Intel: sof_rt5682: add mtl_rt5650 support
> 
> 
> 
> On 11/13/23 05:59, 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
> >
> > Reviewed-by: Bard Liao <yung-chuan.liao@linux.intel.com>
> > Signed-off-by: Mac Chiang <mac.chiang@intel.com>
> > ---
> > Hi Pierre, I resend the patch for your review and cc maintainers. Thanks.
> > Pick up for Chrome MTL projects support.
> 
> The code was submitted as
> https://github.com/thesofproject/linux/pull/4661, approved and merged in
> the topic/sof-dev branch. By submitting the same code multiple times using
> two different paths, you are just adding more confusion and noise for what
> should be a straightforward process on a minor change.
> 
> Moving forward, please pick one of:
> a) submit to alsa-devel
> b) submit to the SOF GitHub and let SOF maintainers upstream.
Thanks for reminding, Pierre. Sure, next time I will follow the minor change process.
> Thank you.
> 
> Acked-by: Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com>
> 
> >
> > Changes v3:
> >   - missing Reviewed-by, add it.
> > ---
> >
> >  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",
Mark Brown Nov. 14, 2023, 11:39 a.m. UTC | #3 
On Mon, 13 Nov 2023 06:59:07 -0500, 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
> 
> 
> [...]

Applied to

   https://git.kernel.org/pub/scm/linux/kernel/git/broonie/sound.git for-next

Thanks!

[1/1] ASoC: Intel: sof_rt5682: add mtl_rt5650 support
      commit: a55ea47bb8743ee044550c5dfdd3cb3147602e05

All being well this means that it will be integrated into the linux-next
tree (usually sometime in the next 24 hours) and sent to Linus during
the next merge window (or sooner if it is a bug fix), however if
problems are discovered then the patch may be dropped or reverted.

You may get further e-mails resulting from automated or manual testing
and review of the tree, please engage with people reporting problems and
send followup patches addressing any issues that are reported if needed.

If any updates are required or you are submitting further changes they
should be sent as incremental updates against current git, existing
patches will not be replaced.

Please add any relevant lists and maintainers to the CCs when replying
to this mail.

Thanks,
Mark
diff mbox series

Patch

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",