All of lore.kernel.org
 help / color / mirror / Atom feed
From: Takashi Iwai <tiwai@suse.de>
To: Kaden Berger <kadenb816@gmail.com>
Cc: alsa-devel@alsa-project.org
Subject: Re: [PATCH] ALSA: hda/realtek: Add quirk for HP Spectre 14t-ea100
Date: Fri, 19 Sep 2025 09:41:14 +0200	[thread overview]
Message-ID: <87h5wyanz9.wl-tiwai@suse.de> (raw)
In-Reply-To: <aMxdGAmfOQ6VPNU8@archlinux>

On Thu, 18 Sep 2025 21:27:20 +0200,
Kaden Berger wrote:
> 

Could you give a proper patch description?  It should describe what
this patch does, why it's needed, etc.
Then put your Signed-off-by tag.  It's a legal requirement.

> --- patch_realtek.c	2025-06-22 11:36:47.365528716 -0500
> +++ patch_realtek.c.new	2025-09-18 13:59:13.610449218 -0500

The patch should be against a full path with a prefix to be applied
with -p1 option.  And, since 6.17 kernel, the code was moved to
another file path, so the patch should appear like:

 --- a/sound/hda/codecs/realtek/alc269.c ....
 +++ a/sound/hda/codecs/realtek/alc269.c ....

For the details about the patch submission, please refer to
Documentation/submitting-patches.rst.

> @@ -10779,6 +10779,7 @@
>  	SND_PCI_QUIRK(0x103c, 0x89c6, "Zbook Fury 17 G9", ALC245_FIXUP_CS35L41_SPI_2_HP_GPIO_LED),
>  	SND_PCI_QUIRK(0x103c, 0x89ca, "HP", ALC236_FIXUP_HP_MUTE_LED_MICMUTE_VREF),
>  	SND_PCI_QUIRK(0x103c, 0x89d3, "HP EliteBook 645 G9 (MB 89D2)", ALC236_FIXUP_HP_MUTE_LED_MICMUTE_VREF),
> +	SND_PCI_QUIRK(0x103c, 0x89da, "HP Spectre x360 14t-ea100", ALC245_FIXUP_HP_SPECTRE_X360_EU0XXX),
>  	SND_PCI_QUIRK(0x103c, 0x89e7, "HP Elite x2 G9", ALC245_FIXUP_CS35L41_SPI_2_HP_GPIO_LED),
>  	SND_PCI_QUIRK(0x103c, 0x8a0f, "HP Pavilion 14-ec1xxx", ALC287_FIXUP_HP_GPIO_LED),
>  	SND_PCI_QUIRK(0x103c, 0x8a20, "HP Laptop 15s-fq5xxx", ALC236_FIXUP_HP_MUTE_LED_COEFBIT2),

The change itself is pretty trivial, and if you rather let other people
fixing it, let us know.  If you'd like to resubmit in a proper format,
that'll be appreciated, too, of course.


thanks,

Takashi

  reply	other threads:[~2025-09-19  7:42 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2025-09-18 19:27 [PATCH] ALSA: hda/realtek: Add quirk for HP Spectre 14t-ea100 Kaden Berger
2025-09-19  7:41 ` Takashi Iwai [this message]
  -- strict thread matches above, loose matches on Subject: below --
2025-09-27  9:47 Takashi Iwai

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=87h5wyanz9.wl-tiwai@suse.de \
    --to=tiwai@suse.de \
    --cc=alsa-devel@alsa-project.org \
    --cc=kadenb816@gmail.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.