* [PATCH v3] Add Dell G15 5525 Ryzen Edition to quirks list for acp6x so that internal DMIC works.
@ 2023-04-10 16:03 Cem Kaya
2023-04-10 16:39 ` Takashi Iwai
0 siblings, 1 reply; 3+ messages in thread
From: Cem Kaya @ 2023-04-10 16:03 UTC (permalink / raw)
To: perex, tiwai; +Cc: mario.limonciello, alsa-devel, linux-kernel
From 972ad712cfda9453a720b73c645601db23d708c6 Mon Sep 17 00:00:00 2001
From: Cem Kaya <cemkaya.boun@gmail.com>
Date: Sat, 8 Apr 2023 16:20:42 +0200
Subject: [PATCH v3] Add Dell G15 5525 Ryzen Edition to quirks list for acp6x so
that internal DMIC works.
Signed-off-by: Cem Kaya <cemkaya.boun@gmail.com>
Commit: Add Dell G15 5525 Ryzen Edition to quirks list for acp6x so that internal mic works.
Link: https://bugzilla.kernel.org/show_bug.cgi?id=217155
---
sound/soc/amd/yc/acp6x-mach.c | 7 +++++++
1 file changed, 7 insertions(+)
diff --git a/sound/soc/amd/yc/acp6x-mach.c b/sound/soc/amd/yc/acp6x-mach.c
index a428e17f0325..e044d811496e 100644
--- a/sound/soc/amd/yc/acp6x-mach.c
+++ b/sound/soc/amd/yc/acp6x-mach.c
@@ -45,6 +45,13 @@ static struct snd_soc_card acp6x_card = {
};
static const struct dmi_system_id yc_acp_quirk_table[] = {
+ {
+ .driver_data = &acp6x_card,
+ .matches = {
+ DMI_MATCH(DMI_BOARD_VENDOR, "Dell Inc."),
+ DMI_MATCH(DMI_PRODUCT_NAME, "Dell G15 5525"),
+ }
+ },
{
.driver_data = &acp6x_card,
.matches = {
--
2.40.0
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH v3] Add Dell G15 5525 Ryzen Edition to quirks list for acp6x so that internal DMIC works.
2023-04-10 16:03 [PATCH v3] Add Dell G15 5525 Ryzen Edition to quirks list for acp6x so that internal DMIC works Cem Kaya
@ 2023-04-10 16:39 ` Takashi Iwai
2023-04-10 16:44 ` Takashi Iwai
0 siblings, 1 reply; 3+ messages in thread
From: Takashi Iwai @ 2023-04-10 16:39 UTC (permalink / raw)
To: Cem Kaya; +Cc: tiwai, mario.limonciello, alsa-devel, linux-kernel
On Mon, 10 Apr 2023 18:03:36 +0200,
Cem Kaya wrote:
>
>
> From 972ad712cfda9453a720b73c645601db23d708c6 Mon Sep 17 00:00:00 2001
> From: Cem Kaya <cemkaya.boun@gmail.com>
> Date: Sat, 8 Apr 2023 16:20:42 +0200
> Subject: [PATCH v3] Add Dell G15 5525 Ryzen Edition to quirks list for acp6x
> so
> that internal DMIC works.
> Signed-off-by: Cem Kaya <cemkaya.boun@gmail.com>
> Commit: Add Dell G15 5525 Ryzen Edition to quirks list for acp6x so that
> internal mic works.
> Link: https://bugzilla.kernel.org/show_bug.cgi?id=217155
Looks better, but still a few things to be correctly formatted.
The subject line needs the proper prefix, depending on the subsystem.
In your case, put "ASoC: amd:", for example. Also, the subject line
should be concise.
The patch description is no such a tag like "Commit:", but it should
be filled in the commit log. This can be as long as you like, but put
the line breaks appropriately.
The Link and Signed-off-by tags follow after the patch description
with a blank line.
In your case, it'd be worth to put Cc-to-stable, too.
So, it should be like:
-- 8< --
From: Cem Kaya <cemkaya.boun@gmail.com>
Subject: [PATCH v3] ASoC: amd: Add Dell G15 5525 to quirks list for internal DMIC
Add Dell G15 5525 Ryzen Edition to quirks list for acp6x so that
internal mic works.
Link: https://bugzilla.kernel.org/show_bug.cgi?id=217155
Cc: <stable@vger.kernel.org>
Signed-off-by: Cem Kaya <cemkaya.boun@gmail.com>
-- 8< --
..., and get the right maintainer via scripts/get_maintainer.pl, put
them to Cc.
For details, please read
Documentation/process/submitting-patches.rst.
*HOWEVER*: the most serious problem is in this patch submission is...
> @@ -45,6 +45,13 @@ static struct snd_soc_card acp6x_card = {
> };
>
> static const struct dmi_system_id yc_acp_quirk_table[] = {
> + {
> + .driver_data = &acp6x_card,
> + .matches = {
> + DMI_MATCH(DMI_BOARD_VENDOR, "Dell Inc."),
> + DMI_MATCH(DMI_PRODUCT_NAME, "Dell G15 5525"),
> + }
> + },
... somehow the patch itself became broken (while v2 patch looked
OK). Please verify how this got broken and try to submit correctly at
the next time.
thanks,
Takashi
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH v3] Add Dell G15 5525 Ryzen Edition to quirks list for acp6x so that internal DMIC works.
2023-04-10 16:39 ` Takashi Iwai
@ 2023-04-10 16:44 ` Takashi Iwai
0 siblings, 0 replies; 3+ messages in thread
From: Takashi Iwai @ 2023-04-10 16:44 UTC (permalink / raw)
To: Cem Kaya; +Cc: tiwai, mario.limonciello, alsa-devel, linux-kernel
On Mon, 10 Apr 2023 18:39:14 +0200,
Takashi Iwai wrote:
>
> ..., and get the right maintainer via scripts/get_maintainer.pl, put
> them to Cc.
Note that, in your case, put rather "supports" (especially Mark) to Cc
in addition to maintainers.
Takashi
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2023-04-12 5:27 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2023-04-10 16:03 [PATCH v3] Add Dell G15 5525 Ryzen Edition to quirks list for acp6x so that internal DMIC works Cem Kaya
2023-04-10 16:39 ` Takashi Iwai
2023-04-10 16:44 ` Takashi Iwai
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).