RE: [PATCH] ASoC: amd: yc: Update DMI table entries

[Limonciello, Mario]

[Public]

> -----Original Message-----
> From: syed sabakareem <Syed.SabaKareem@xxxxxxx>
> Sent: Friday, July 22, 2022 08:46
> To: broonie@xxxxxxxxxx; alsa-devel@xxxxxxxxxxxxxxxx
> Cc: Mukunda, Vijendar <Vijendar.Mukunda@xxxxxxx>; Deucher,
> Alexander <Alexander.Deucher@xxxxxxx>; Hiregoudar, Basavaraj
> <Basavaraj.Hiregoudar@xxxxxxx>; Dommati, Sunil-kumar <Sunil-
> kumar.Dommati@xxxxxxx>; markpearson@xxxxxxxxxx; Limonciello, Mario
> <Mario.Limonciello@xxxxxxx>; Saba Kareem, Syed
> <Syed.SabaKareem@xxxxxxx>; Liam Girdwood <lgirdwood@xxxxxxxxx>;
> Jaroslav Kysela <perex@xxxxxxxx>; Takashi Iwai <tiwai@xxxxxxxx>;
> Mukunda, Vijendar <Vijendar.Mukunda@xxxxxxx>; open list <linux-
> kernel@xxxxxxxxxxxxxxx>
> Subject: [PATCH] ASoC: amd: yc: Update DMI table entries
> 
> Removed intel DMI product id's 21AW/21AX/21D8/21D9/21BN/21BQ
> in DMI table and updated DMI entry for AMD platform X13 Gen 3
> platform 21CM/21CN.
> 
> Link: https://bugzilla.kernel.org/show_bug.cgi?id=216267

Syed thanks for the quick patch on this issue.
I just noticed one more tag to add for the reporter of this that Mark can pick up when he pulls this in.

Reported-by: David Korth <gerbilsoft@xxxxxxxxxxxxxx>

Also suggest Mark this to consider to pull back to @stable, mostly because of the X13 IDs missing.

> 
> Signed-off-by: syed sabakareem <Syed.SabaKareem@xxxxxxx>
> Reviewed-by: Mario Limonciello <mario.limonciello@xxxxxxx>
> ---
>  sound/soc/amd/yc/acp6x-mach.c | 32 ++------------------------------
>  1 file changed, 2 insertions(+), 30 deletions(-)
> 
> diff --git a/sound/soc/amd/yc/acp6x-mach.c b/sound/soc/amd/yc/acp6x-
> mach.c
> index f06e6c1a7799..ecfe7a790790 100644
> --- a/sound/soc/amd/yc/acp6x-mach.c
> +++ b/sound/soc/amd/yc/acp6x-mach.c
> @@ -105,28 +105,14 @@ static const struct dmi_system_id
> yc_acp_quirk_table[] = {
>  		.driver_data = &acp6x_card,
>  		.matches = {
>  			DMI_MATCH(DMI_BOARD_VENDOR, "LENOVO"),
> -			DMI_MATCH(DMI_PRODUCT_NAME, "21AW"),
> +			DMI_MATCH(DMI_PRODUCT_NAME, "21CM"),
>  		}
>  	},
>  	{
>  		.driver_data = &acp6x_card,
>  		.matches = {
>  			DMI_MATCH(DMI_BOARD_VENDOR, "LENOVO"),
> -			DMI_MATCH(DMI_PRODUCT_NAME, "21AX"),
> -		}
> -	},
> -	{
> -		.driver_data = &acp6x_card,
> -		.matches = {
> -			DMI_MATCH(DMI_BOARD_VENDOR, "LENOVO"),
> -			DMI_MATCH(DMI_PRODUCT_NAME, "21BN"),
> -		}
> -	},
> -	{
> -		.driver_data = &acp6x_card,
> -		.matches = {
> -			DMI_MATCH(DMI_BOARD_VENDOR, "LENOVO"),
> -			DMI_MATCH(DMI_PRODUCT_NAME, "21BQ"),
> +			DMI_MATCH(DMI_PRODUCT_NAME, "21CN"),
>  		}
>  	},
>  	{
> @@ -157,20 +143,6 @@ static const struct dmi_system_id
> yc_acp_quirk_table[] = {
>  			DMI_MATCH(DMI_PRODUCT_NAME, "21CL"),
>  		}
>  	},
> -	{
> -		.driver_data = &acp6x_card,
> -		.matches = {
> -			DMI_MATCH(DMI_BOARD_VENDOR, "LENOVO"),
> -			DMI_MATCH(DMI_PRODUCT_NAME, "21D8"),
> -		}
> -	},
> -	{
> -		.driver_data = &acp6x_card,
> -		.matches = {
> -			DMI_MATCH(DMI_BOARD_VENDOR, "LENOVO"),
> -			DMI_MATCH(DMI_PRODUCT_NAME, "21D9"),
> -		}
> -	},
>  	{}
>  };
> 
> --
> 2.25.1