On Fri, 17 Nov 2023 08:17:48 +0100, Oliver Sieber wrote:
Hi,
in the following I provide my proposed patch for the bug report on Bugzilla [1]. Let me quote from it:
Hey,sound/soc/amd/renoir/rn-pci-acp3x.c has a hard-coded list of quirks. It seems like some product IDs may be missing. In my case, I am using a Lenovo Thinkpad t14s gen2 with an AMD Ryzen 7 PRO 5850U. My audio output is fine but my internal microphone cannot be found. Running `dmidecode` yields: Handle 0x000F, DMI type 1, 27 bytes System Information Manufacturer: LENOVO Product Name: 20XGS1KT02 Version: ThinkPad T14s Gen 2a Could it be that one may have to add the product ID "20XGS1KT02" to the hard-coded list of quirks in rn-pci-acp3x.c? It seems as there have been similar problems in the past, e.g. https://bugzilla.kernel.org/show_bug.cgi?id=216270.My proposed diff looks as follows:
diff --git a/sound/soc/amd/renoir/rn-pci-acp3x.c b/sound/soc/amd/renoir/ rn-pci-acp3x.c index b3812b70f5f9..754e24d55e6f 100644 --- a/sound/soc/amd/renoir/rn-pci-acp3x.c +++ b/sound/soc/amd/renoir/rn-pci-acp3x.c @@ -199,6 +199,13 @@ static const struct dmi_system_id rn_acp_quirk_table[] = { DMI_EXACT_MATCH(DMI_BOARD_NAME, "20NLCTO1WW"), } }, + { + /* Lenovo ThinkPad T14s Gen 2 */ + .matches = { + DMI_EXACT_MATCH(DMI_BOARD_VENDOR, "LENOVO"), + DMI_EXACT_MATCH(DMI_BOARD_NAME, "20XGS1KT02"), + } + }, {} };
Please have in mind that this is my first patch for the kernel.
The kernel patch should be in a proper format, so that it can be merged to the upstream. e.g. The Subject line should have a brief description with the proper prefix for the corresponding subsystem (in your case, something like "ASoC: amd:"). It'll follow the patch description, as well as your proper Signed-off-by tag.
The details are found in https://docs.kernel.org/process/submitting-patches.html
Could you resubmit the patch in a proper format?
thanks,
Takashi