2021-03-04 06:15:23

by Werner Sembach

[permalink] [raw]
Subject: [PATCH] ALSA: hda/realtek: Add quirk for Clevo NH55RZQ

From: Eckhart Mohr <[email protected]>

ALSA: hda/realtek: Add quirk for Clevo NH55RZQ

This applies a SND_PCI_QUIRK(...) to the Clevo NH55RZQ barebone. This
fixes the issue of the device not recognizing a pluged in microphone.

The device has both, a microphone only jack, and a speaker + microphone
combo jack. The combo jack already works. The microphone-only jack does
not recognize when a device is pluged in without this patch.

Signed-off-by: Eckhart Mohr <[email protected]>
Co-developed-by: Werner Sembach <[email protected]>
Signed-off-by: Werner Sembach <[email protected]>
---
Hi,
this is my first ever submitted kernel patch, feel free to criticise me if I made an error or missed a best practise bullet point.
Also: I'm unsure if this would fit the requirements for [email protected] and/or [email protected], but I think not (correct me if I'm wrong).
Kind regards
Werner Sembach

From 2835edd753fd19c72a644dccb7e941cfc0ecdf8e Mon Sep 17 00:00:00 2001
From: Werner Sembach <[email protected]>
Date: Fri, 26 Feb 2021 13:50:15 +0100
Subject: [PATCH] Fix device detection on microphone jack of Clevo NH55RZQ

---
 sound/pci/hda/patch_realtek.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/sound/pci/hda/patch_realtek.c b/sound/pci/hda/patch_realtek.c
index 290645516313..8014e80d72c3 100644
--- a/sound/pci/hda/patch_realtek.c
+++ b/sound/pci/hda/patch_realtek.c
@@ -8089,6 +8089,7 @@ static const struct snd_pci_quirk alc269_fixup_tbl[] = {
     SND_PCI_QUIRK(0x1558, 0x8551, "System76 Gazelle (gaze14)", ALC293_FIXUP_SYSTEM76_MIC_NO_PRESENCE),
     SND_PCI_QUIRK(0x1558, 0x8560, "System76 Gazelle (gaze14)", ALC269_FIXUP_HEADSET_MIC),
     SND_PCI_QUIRK(0x1558, 0x8561, "System76 Gazelle (gaze14)", ALC269_FIXUP_HEADSET_MIC),
+    SND_PCI_QUIRK(0x1558, 0x8562, "Clevo NH[5|7][0-9]RZ[Q]", ALC269_FIXUP_DMIC),
     SND_PCI_QUIRK(0x1558, 0x8668, "Clevo NP50B[BE]", ALC293_FIXUP_SYSTEM76_MIC_NO_PRESENCE),
     SND_PCI_QUIRK(0x1558, 0x8680, "Clevo NJ50LU", ALC293_FIXUP_SYSTEM76_MIC_NO_PRESENCE),
     SND_PCI_QUIRK(0x1558, 0x8686, "Clevo NH50[CZ]U", ALC293_FIXUP_SYSTEM76_MIC_NO_PRESENCE),
--
2.25.1







2021-03-04 06:28:43

by Takashi Iwai

[permalink] [raw]
Subject: Re: [PATCH] ALSA: hda/realtek: Add quirk for Clevo NH55RZQ

On Tue, 02 Mar 2021 12:19:03 +0100,
Werner Sembach wrote:
>
> From: Eckhart Mohr <[email protected]>
>
> ALSA: hda/realtek: Add quirk for Clevo NH55RZQ
>
> This applies a SND_PCI_QUIRK(...) to the Clevo NH55RZQ barebone. This
> fixes the issue of the device not recognizing a pluged in microphone.
>
> The device has both, a microphone only jack, and a speaker + microphone
> combo jack. The combo jack already works. The microphone-only jack does
> not recognize when a device is pluged in without this patch.
>
> Signed-off-by: Eckhart Mohr <[email protected]>
> Co-developed-by: Werner Sembach <[email protected]>
> Signed-off-by: Werner Sembach <[email protected]>
> ---
> Hi,
> this is my first ever submitted kernel patch, feel free to criticise me if I made an error or missed a best practise bullet point.

The patch isn't cleanly applicable. Could you try to submit via
git-send-email instead?

> Also: I'm unsure if this would fit the requirements for [email protected] and/or [email protected], but I think not (correct me if I'm wrong).

Cc to stable is worthwhile. You can simply add
Cc: <[email protected]>

around your signed-off-by lines in the patch.


thanks,

Takashi

2021-03-04 07:16:51

by Werner Sembach

[permalink] [raw]
Subject: Re: [PATCH] ALSA: hda/realtek: Add quirk for Clevo NH55RZQ

Hello Takashi,
> The patch isn't cleanly applicable. Could you try to submit via
> git-send-email instead?

I'm sorry the editor + mailer i used replaced tabs with spaces and made a html mail.

Already resend the patch with tabs, but only after that I realized that even the "plain" mails are converted to html.

It's probably still broken so don't waste your time with it, i will read the man page of git send-mail now ^^.

> Cc to stable is worthwhile. You can simply add
> Cc: <[email protected]>
>
> around your signed-off-by lines in the patch.

Ok (also forgot that in my resend patch)

Kind regards,

Werner Sembach