All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] ALSA: hda/realtek: add new framework to quirk chain
@ 2023-12-05 23:42 Tim Bosse
  2023-12-06  2:33 ` Geraldo Nascimento
  0 siblings, 1 reply; 6+ messages in thread
From: Tim Bosse @ 2023-12-05 23:42 UTC (permalink / raw)
  To: alsa-devel; +Cc: Tim Bosse

The Framework Laptop 13 (AMD Ryzen 7040Series) has an ALC 295 with
a disconnected or faulty headset mic presence detect and works with the
same quirk chain as 309d7363ca3d9fcdb92ff2d958be14d7e8707f68. This model
has a VID:PID of f111:0006.

Signed-off-by: Tim Bosse <flinn@timbos.se>
---
 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 f9ddacfd920e..59f2578839ed 100644
--- a/sound/pci/hda/patch_realtek.c
+++ b/sound/pci/hda/patch_realtek.c
@@ -10271,6 +10271,7 @@ static const struct snd_pci_quirk alc269_fixup_tbl[] = {
 	SND_PCI_QUIRK(0x8086, 0x2081, "Intel NUC 10", ALC256_FIXUP_INTEL_NUC10),
 	SND_PCI_QUIRK(0x8086, 0x3038, "Intel NUC 13", ALC295_FIXUP_CHROME_BOOK),
 	SND_PCI_QUIRK(0xf111, 0x0001, "Framework Laptop", ALC295_FIXUP_FRAMEWORK_LAPTOP_MIC_NO_PRESENCE),
+	SND_PCI_QUIRK(0xf111, 0x0006, "Framework Laptop", ALC295_FIXUP_FRAMEWORK_LAPTOP_MIC_NO_PRESENCE),
 
 #if 0
 	/* Below is a quirk table taken from the old code.
-- 
2.43.0


^ permalink raw reply related	[flat|nested] 6+ messages in thread

* Re: [PATCH] ALSA: hda/realtek: add new framework to quirk chain
  2023-12-05 23:42 [PATCH] ALSA: hda/realtek: add new framework to quirk chain Tim Bosse
@ 2023-12-06  2:33 ` Geraldo Nascimento
  2023-12-06 14:26   ` [PATCH v2] ALSA: hda/realtek: add new Framework laptop to quirks Tim Bosse
  0 siblings, 1 reply; 6+ messages in thread
From: Geraldo Nascimento @ 2023-12-06  2:33 UTC (permalink / raw)
  To: Tim Bosse; +Cc: alsa-devel, Takashi Iwai

On Tue, Dec 05, 2023 at 06:42:00PM -0500, Tim Bosse wrote:
> The Framework Laptop 13 (AMD Ryzen 7040Series) has an ALC 295 with
> a disconnected or faulty headset mic presence detect and works with the
> same quirk chain as 309d7363ca3d9fcdb92ff2d958be14d7e8707f68. This model
> has a VID:PID of f111:0006.
> 
> Signed-off-by: Tim Bosse <flinn@timbos.se>
> ---
>  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 f9ddacfd920e..59f2578839ed 100644
> --- a/sound/pci/hda/patch_realtek.c
> +++ b/sound/pci/hda/patch_realtek.c
> @@ -10271,6 +10271,7 @@ static const struct snd_pci_quirk alc269_fixup_tbl[] = {
>  	SND_PCI_QUIRK(0x8086, 0x2081, "Intel NUC 10", ALC256_FIXUP_INTEL_NUC10),
>  	SND_PCI_QUIRK(0x8086, 0x3038, "Intel NUC 13", ALC295_FIXUP_CHROME_BOOK),
>  	SND_PCI_QUIRK(0xf111, 0x0001, "Framework Laptop", ALC295_FIXUP_FRAMEWORK_LAPTOP_MIC_NO_PRESENCE),
> +	SND_PCI_QUIRK(0xf111, 0x0006, "Framework Laptop", ALC295_FIXUP_FRAMEWORK_LAPTOP_MIC_NO_PRESENCE),
>  
>  #if 0
>  	/* Below is a quirk table taken from the old code.
> -- 
> 2.43.0

Hi Tim, you forgot to add at least Takashi to the Cc: and your patch
will most likely be missed, so putting him in the loop.

Also, not much to say here since it's a simple desirable fix but would
you mind sending v2 with a rewording of the commit info? The way you put
it sounds like a new framework is being added to the realtek quirk
chain, which is not the case, you're adding a new Framework-brand laptop
to the quirks.

Thank you,
Geraldo Nascimento

^ permalink raw reply	[flat|nested] 6+ messages in thread

* [PATCH v2] ALSA: hda/realtek: add new Framework laptop to quirks
  2023-12-06  2:33 ` Geraldo Nascimento
@ 2023-12-06 14:26   ` Tim Bosse
  2023-12-06 14:31     ` Takashi Iwai
  0 siblings, 1 reply; 6+ messages in thread
From: Tim Bosse @ 2023-12-06 14:26 UTC (permalink / raw)
  To: alsa-devel; +Cc: tiwai, Tim Bosse

The Framework Laptop 13 (AMD Ryzen 7040Series) has an ALC295 with
a disconnected or faulty headset mic presence detect similar to the
previous models.  It works with the same quirk chain as
309d7363ca3d9fcdb92ff2d958be14d7e8707f68.  This model has a VID:PID
of f111:0006.

Signed-off-by: Tim Bosse <flinn@timbos.se>
---
 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 f9ddacfd920e..59f2578839ed 100644
--- a/sound/pci/hda/patch_realtek.c
+++ b/sound/pci/hda/patch_realtek.c
@@ -10271,6 +10271,7 @@ static const struct snd_pci_quirk alc269_fixup_tbl[] = {
 	SND_PCI_QUIRK(0x8086, 0x2081, "Intel NUC 10", ALC256_FIXUP_INTEL_NUC10),
 	SND_PCI_QUIRK(0x8086, 0x3038, "Intel NUC 13", ALC295_FIXUP_CHROME_BOOK),
 	SND_PCI_QUIRK(0xf111, 0x0001, "Framework Laptop", ALC295_FIXUP_FRAMEWORK_LAPTOP_MIC_NO_PRESENCE),
+	SND_PCI_QUIRK(0xf111, 0x0006, "Framework Laptop", ALC295_FIXUP_FRAMEWORK_LAPTOP_MIC_NO_PRESENCE),
 
 #if 0
 	/* Below is a quirk table taken from the old code.
-- 
2.43.0


^ permalink raw reply related	[flat|nested] 6+ messages in thread

* Re: [PATCH v2] ALSA: hda/realtek: add new Framework laptop to quirks
  2023-12-06 14:26   ` [PATCH v2] ALSA: hda/realtek: add new Framework laptop to quirks Tim Bosse
@ 2023-12-06 14:31     ` Takashi Iwai
  0 siblings, 0 replies; 6+ messages in thread
From: Takashi Iwai @ 2023-12-06 14:31 UTC (permalink / raw)
  To: Tim Bosse; +Cc: alsa-devel

On Wed, 06 Dec 2023 15:26:29 +0100,
Tim Bosse wrote:
> 
> The Framework Laptop 13 (AMD Ryzen 7040Series) has an ALC295 with
> a disconnected or faulty headset mic presence detect similar to the
> previous models.  It works with the same quirk chain as
> 309d7363ca3d9fcdb92ff2d958be14d7e8707f68.  This model has a VID:PID
> of f111:0006.
> 
> Signed-off-by: Tim Bosse <flinn@timbos.se>

Thanks, applied now.


Takashi

^ permalink raw reply	[flat|nested] 6+ messages in thread

* [PATCH] ALSA: hda/realtek: add new framework to quirk chain
@ 2023-12-05 17:10 Tim Bosse
  0 siblings, 0 replies; 6+ messages in thread
From: Tim Bosse @ 2023-12-05 17:10 UTC (permalink / raw)
  To: alsa-devel; +Cc: Tim Bosse

The Framework Laptop 13 (AMD Ryzen 7040Series) has an ALC 295 with
a disconnected or faulty headset mic presence detect and works with the
same quirk chain as 309d7363ca3d9fcdb92ff2d958be14d7e8707f68. This model
has a VID:PID of f111:0006.

Signed-off-by: Tim Bosse <flinn@timbos.se>
---
 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 f9ddacfd920e..59f2578839ed 100644
--- a/sound/pci/hda/patch_realtek.c
+++ b/sound/pci/hda/patch_realtek.c
@@ -10271,6 +10271,7 @@ static const struct snd_pci_quirk alc269_fixup_tbl[] = {
 	SND_PCI_QUIRK(0x8086, 0x2081, "Intel NUC 10", ALC256_FIXUP_INTEL_NUC10),
 	SND_PCI_QUIRK(0x8086, 0x3038, "Intel NUC 13", ALC295_FIXUP_CHROME_BOOK),
 	SND_PCI_QUIRK(0xf111, 0x0001, "Framework Laptop", ALC295_FIXUP_FRAMEWORK_LAPTOP_MIC_NO_PRESENCE),
+	SND_PCI_QUIRK(0xf111, 0x0006, "Framework Laptop", ALC295_FIXUP_FRAMEWORK_LAPTOP_MIC_NO_PRESENCE),
 
 #if 0
 	/* Below is a quirk table taken from the old code.
-- 
2.43.0


^ permalink raw reply related	[flat|nested] 6+ messages in thread

* [PATCH] ALSA: hda/realtek: add new framework to quirk chain
@ 2023-12-05  2:47 Tim Bosse
  0 siblings, 0 replies; 6+ messages in thread
From: Tim Bosse @ 2023-12-05  2:47 UTC (permalink / raw)
  To: alsa-devel; +Cc: Tim Bosse

The Framework Laptop 13 (AMD Ryzen 7040Series) also has an ALC 295 with
a disconnected or faulty headset mic presence detect.  This model has a
VID:PID of f111:0006.

Signed-off-by: Tim Bosse <flinn@timbos.se>
---
 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 f9ddacfd920e..59f2578839ed 100644
--- a/sound/pci/hda/patch_realtek.c
+++ b/sound/pci/hda/patch_realtek.c
@@ -10271,6 +10271,7 @@ static const struct snd_pci_quirk alc269_fixup_tbl[] = {
 	SND_PCI_QUIRK(0x8086, 0x2081, "Intel NUC 10", ALC256_FIXUP_INTEL_NUC10),
 	SND_PCI_QUIRK(0x8086, 0x3038, "Intel NUC 13", ALC295_FIXUP_CHROME_BOOK),
 	SND_PCI_QUIRK(0xf111, 0x0001, "Framework Laptop", ALC295_FIXUP_FRAMEWORK_LAPTOP_MIC_NO_PRESENCE),
+	SND_PCI_QUIRK(0xf111, 0x0006, "Framework Laptop", ALC295_FIXUP_FRAMEWORK_LAPTOP_MIC_NO_PRESENCE),
 
 #if 0
 	/* Below is a quirk table taken from the old code.
-- 
2.43.0


^ permalink raw reply related	[flat|nested] 6+ messages in thread

end of thread, other threads:[~2023-12-06 14:32 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2023-12-05 23:42 [PATCH] ALSA: hda/realtek: add new framework to quirk chain Tim Bosse
2023-12-06  2:33 ` Geraldo Nascimento
2023-12-06 14:26   ` [PATCH v2] ALSA: hda/realtek: add new Framework laptop to quirks Tim Bosse
2023-12-06 14:31     ` Takashi Iwai
  -- strict thread matches above, loose matches on Subject: below --
2023-12-05 17:10 [PATCH] ALSA: hda/realtek: add new framework to quirk chain Tim Bosse
2023-12-05  2:47 Tim Bosse

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.