Message ID | 20240102024916.19093-1-andy.chi@canonical.com |
---|---|
State | New |
Headers |
Return-Path: <linux-kernel+bounces-13993-ouuuleilei=gmail.com@vger.kernel.org> Delivered-To: ouuuleilei@gmail.com Received: by 2002:a05:7301:6f82:b0:100:9c79:88ff with SMTP id tb2csp4244218dyb; Mon, 1 Jan 2024 19:00:28 -0800 (PST) X-Google-Smtp-Source: AGHT+IGAy4oow/XV2ifehN/VmMxJrVqy4gmwztF60HDZz/ZqeOQ0Zqz7CNhuO91DYSz2grOrPEnL X-Received: by 2002:a05:6870:a54a:b0:1fb:75b:1325 with SMTP id p10-20020a056870a54a00b001fb075b1325mr17973243oal.119.1704164427852; Mon, 01 Jan 2024 19:00:27 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1704164427; cv=none; d=google.com; s=arc-20160816; b=SAXnPB3FtudcKWrsr4at9IAlCc2vt3fGwUTcGVkaxfyv72G1D7EtdvkCA+AY5sNvyq y+ernDRR/k5OB6PnWxg8a/S28hJjuaT+3dopnWhRkSVVp9BIC0OifHCCtM/l51uMWINz MCRHG3/PfrpglLcyc9fQerZgyVUAS0GlBbNxjedASBCDkvfj32picM2DDFBxQmt2IMw9 pAwz9kZMDJUiKXdxJIzzpNXK0/9IJJRjm0MXxMazZJu0ky6JG7n0/Xqr1V8c4Dt/VpLK +W+JwSd3v+wJvVWfI03BqBiRJlxK1/8iO/noazgXIma0fE7f2uaHQzAxdHRbHV9YnvKt SqwA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=content-transfer-encoding:mime-version:list-unsubscribe :list-subscribe:list-id:precedence:message-id:date:subject:cc:to :from:dkim-signature; bh=EWkKSm+nN6fBmS9JU50EaFN7HE3LdA/HGHMNIUbvCq4=; fh=GPExLTdS1Uyj969FDWZodjsSdxQafoxrbLArZdggiKs=; b=0JeqfgH/tOehMGKPNeblWHoUQw1kEDgjqOcFjRLrvoAMkx3ezTFTQgqopJ7aMxDP+G yVQn4XxFXbO8JH1SHRIxQlJvip9X9IPVbcniuf1pVlLiTRtKM9xa+bQ7pWRipyj+6+nX EKBbH/+XUUys7rqq3Z7hRg+cWvoxj9ACq7YLsCKFuyemk6vggZSJ4XkiwiTZUby7kHBc BCyDxIG2Okpkxcl5EAKuvLzTNBiU0P4qK0ylpnlAJQFAnP1N84JvO0pK4qP6D+SBIiH6 BCzadsVo041fNHfcd+Wbocp6irUY9mvxO/jj9YsRDc9xuMGMGbUBy4z9Qc7YP6xlkw9J CzBg== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@canonical.com header.s=20210705 header.b=ls3Hry8I; spf=pass (google.com: domain of linux-kernel+bounces-13993-ouuuleilei=gmail.com@vger.kernel.org designates 147.75.48.161 as permitted sender) smtp.mailfrom="linux-kernel+bounces-13993-ouuuleilei=gmail.com@vger.kernel.org"; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=canonical.com Received: from sy.mirrors.kernel.org (sy.mirrors.kernel.org. [147.75.48.161]) by mx.google.com with ESMTPS id ci21-20020a17090afc9500b0028c8db12828si9787860pjb.3.2024.01.01.19.00.27 for <ouuuleilei@gmail.com> (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 01 Jan 2024 19:00:27 -0800 (PST) Received-SPF: pass (google.com: domain of linux-kernel+bounces-13993-ouuuleilei=gmail.com@vger.kernel.org designates 147.75.48.161 as permitted sender) client-ip=147.75.48.161; Authentication-Results: mx.google.com; dkim=pass header.i=@canonical.com header.s=20210705 header.b=ls3Hry8I; spf=pass (google.com: domain of linux-kernel+bounces-13993-ouuuleilei=gmail.com@vger.kernel.org designates 147.75.48.161 as permitted sender) smtp.mailfrom="linux-kernel+bounces-13993-ouuuleilei=gmail.com@vger.kernel.org"; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=canonical.com Received: from smtp.subspace.kernel.org (wormhole.subspace.kernel.org [52.25.139.140]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by sy.mirrors.kernel.org (Postfix) with ESMTPS id 45561B211F3 for <ouuuleilei@gmail.com>; Tue, 2 Jan 2024 03:00:25 +0000 (UTC) Received: from localhost.localdomain (localhost.localdomain [127.0.0.1]) by smtp.subspace.kernel.org (Postfix) with ESMTP id 497A71383; Tue, 2 Jan 2024 03:00:10 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=canonical.com header.i=@canonical.com header.b="ls3Hry8I" X-Original-To: linux-kernel@vger.kernel.org Received: from smtp-relay-canonical-0.canonical.com (smtp-relay-canonical-0.canonical.com [185.125.188.120]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 4B397EBC; Tue, 2 Jan 2024 03:00:03 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=canonical.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=canonical.com Received: from localhost.localdomain (unknown [10.101.197.31]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (2048 bits) server-digest SHA256) (No client certificate requested) by smtp-relay-canonical-0.canonical.com (Postfix) with ESMTPSA id 872E83F2AE; Tue, 2 Jan 2024 02:50:04 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=canonical.com; s=20210705; t=1704163818; bh=EWkKSm+nN6fBmS9JU50EaFN7HE3LdA/HGHMNIUbvCq4=; h=From:To:Cc:Subject:Date:Message-Id:MIME-Version; b=ls3Hry8IzfVXSA836/9OxoK9XjBXgHxcKgvwcyT9rXghBgKEeSH0SFBaaNY4J0+J5 Uc0RmnN85cpCRc03ffP40NkY6h/EMqvYxvfVnWVu8XxHarQ+J6vnq3Gnk/hZJqWIxO czP7CaM8W65GAWDhTSm/2LnDUGIuUU+cbi42AnzkLWq9LEx4O/iiqvbUnwebzySr0n enZmEn9pfiHhl3xyFT2m/lHKPaw/vYw9btu4XDjaBNHr3qD162sdXlsa76pm/0XXIm B9g2ZryVXVWxrqV8JgWy21NB3UachkNE0sP0LO3qw+NhJPsveQ4deX4MNiU6J4it9l fLqsxjtIY03lA== From: Andy Chi <andy.chi@canonical.com> To: Cc: andy.chi@canonical.com, Jaroslav Kysela <perex@perex.cz>, Takashi Iwai <tiwai@suse.com>, Stefan Binding <sbinding@opensource.cirrus.com>, Kailang Yang <kailang@realtek.com>, "Luke D. Jones" <luke@ljones.dev>, Shenghao Ding <shenghao-ding@ti.com>, Matthew Anderson <ruinairas1992@gmail.com>, Luka Guzenko <l.guzenko@web.de>, Yuchi Yang <yangyuchi66@gmail.com>, Vitaly Rodionov <vitalyr@opensource.cirrus.com>, linux-sound@vger.kernel.org, linux-kernel@vger.kernel.org Subject: [PATCH] ALSA: hda/realtek: fix mute/micmute LEDs for a HP ZBook Date: Tue, 2 Jan 2024 10:49:15 +0800 Message-Id: <20240102024916.19093-1-andy.chi@canonical.com> X-Mailer: git-send-email 2.34.1 Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: <linux-kernel.vger.kernel.org> List-Subscribe: <mailto:linux-kernel+subscribe@vger.kernel.org> List-Unsubscribe: <mailto:linux-kernel+unsubscribe@vger.kernel.org> MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-getmail-retrieved-from-mailbox: INBOX X-GMAIL-THRID: 1786945919447266933 X-GMAIL-MSGID: 1786945919447266933 |
Series |
ALSA: hda/realtek: fix mute/micmute LEDs for a HP ZBook
|
|
Commit Message
Andy Chi
Jan. 2, 2024, 2:49 a.m. UTC
There is a HP ZBook which using ALC236 codec and need the
ALC236_FIXUP_HP_MUTE_LED_MICMUTE_VREF quirk to make mute LED
and micmute LED work.
Signed-off-by: Andy Chi <andy.chi@canonical.com>
---
sound/pci/hda/patch_realtek.c | 1 +
1 file changed, 1 insertion(+)
Comments
On Tue, Jan 02, 2024 at 10:49:15AM +0800, Andy Chi wrote: > There is a HP ZBook which using ALC236 codec and need the > ALC236_FIXUP_HP_MUTE_LED_MICMUTE_VREF quirk to make mute LED > and micmute LED work. Hi Andy, I know Takashi in the name of smooth operation usually takes this kind of simple patch, quickly, since it's a simple bugfix, unless there are any objections, so I must raise one. Which HP ZBook? Which model? Is this a still unreleased model Canonical has had access under a NDA for example? Or perhaps there is some other reason why you cannot name the model yet? Either way, there is a need to publicly know the reason for the missing model information. > > Signed-off-by: Andy Chi <andy.chi@canonical.com> > --- > 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 7e9d8c1cedfc..1c8d5b58eb64 100644 > --- a/sound/pci/hda/patch_realtek.c > +++ b/sound/pci/hda/patch_realtek.c > @@ -9939,6 +9939,7 @@ static const struct snd_pci_quirk alc269_fixup_tbl[] = { > SND_PCI_QUIRK(0x103c, 0x8c70, "HP EliteBook 835 G11", ALC287_FIXUP_CS35L41_I2C_2_HP_GPIO_LED), > SND_PCI_QUIRK(0x103c, 0x8c71, "HP EliteBook 845 G11", ALC287_FIXUP_CS35L41_I2C_2_HP_GPIO_LED), > SND_PCI_QUIRK(0x103c, 0x8c72, "HP EliteBook 865 G11", ALC287_FIXUP_CS35L41_I2C_2_HP_GPIO_LED), > + SND_PCI_QUIRK(0x103c, 0x8c96, "HP", ALC236_FIXUP_HP_MUTE_LED_MICMUTE_VREF), Same questioning as above. "HP" is IMHO unacceptable without a clear reason for the missing model information. Thanks, Geraldo Nascimento > SND_PCI_QUIRK(0x103c, 0x8ca4, "HP ZBook Fury", ALC245_FIXUP_CS35L41_SPI_2_HP_GPIO_LED), > SND_PCI_QUIRK(0x103c, 0x8ca7, "HP ZBook Fury", ALC245_FIXUP_CS35L41_SPI_2_HP_GPIO_LED), > SND_PCI_QUIRK(0x103c, 0x8cf5, "HP ZBook Studio 16", ALC245_FIXUP_CS35L41_SPI_4_HP_GPIO_LED), > -- > 2.34.1 > >
Hi Geraldo, Yes, this model is not yet publicly available, so I can't type the full name on this patch. (change to text plain mode to send out again.) Kr, Andy On Tue, Jan 2, 2024 at 1:47 PM Andy Chi <andy.chi@canonical.com> wrote: > > Hi Geraldo, > Yes, this model is not yet publicly available, so I can't type the full name on this patch. > > Kr, > Andy > > On Tue, Jan 2, 2024 at 12:25 PM Geraldo Nascimento <geraldogabriel@gmail.com> wrote: >> >> On Tue, Jan 02, 2024 at 10:49:15AM +0800, Andy Chi wrote: >> > There is a HP ZBook which using ALC236 codec and need the >> > ALC236_FIXUP_HP_MUTE_LED_MICMUTE_VREF quirk to make mute LED >> > and micmute LED work. >> >> Hi Andy, >> >> I know Takashi in the name of smooth operation usually takes this kind >> of simple patch, quickly, since it's a simple bugfix, unless there are >> any objections, so I must raise one. >> >> Which HP ZBook? Which model? >> >> Is this a still unreleased model Canonical has had access under a NDA >> for example? Or perhaps there is some other reason why you cannot name >> the model yet? >> >> Either way, there is a need to publicly know the reason for the missing >> model information. >> >> > >> > Signed-off-by: Andy Chi <andy.chi@canonical.com> >> > --- >> > 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 7e9d8c1cedfc..1c8d5b58eb64 100644 >> > --- a/sound/pci/hda/patch_realtek.c >> > +++ b/sound/pci/hda/patch_realtek.c >> > @@ -9939,6 +9939,7 @@ static const struct snd_pci_quirk alc269_fixup_tbl[] = { >> > SND_PCI_QUIRK(0x103c, 0x8c70, "HP EliteBook 835 G11", ALC287_FIXUP_CS35L41_I2C_2_HP_GPIO_LED), >> > SND_PCI_QUIRK(0x103c, 0x8c71, "HP EliteBook 845 G11", ALC287_FIXUP_CS35L41_I2C_2_HP_GPIO_LED), >> > SND_PCI_QUIRK(0x103c, 0x8c72, "HP EliteBook 865 G11", ALC287_FIXUP_CS35L41_I2C_2_HP_GPIO_LED), >> > + SND_PCI_QUIRK(0x103c, 0x8c96, "HP", ALC236_FIXUP_HP_MUTE_LED_MICMUTE_VREF), >> >> Same questioning as above. "HP" is IMHO unacceptable without a clear >> reason for the missing model information. >> >> Thanks, >> Geraldo Nascimento >> >> > SND_PCI_QUIRK(0x103c, 0x8ca4, "HP ZBook Fury", ALC245_FIXUP_CS35L41_SPI_2_HP_GPIO_LED), >> > SND_PCI_QUIRK(0x103c, 0x8ca7, "HP ZBook Fury", ALC245_FIXUP_CS35L41_SPI_2_HP_GPIO_LED), >> > SND_PCI_QUIRK(0x103c, 0x8cf5, "HP ZBook Studio 16", ALC245_FIXUP_CS35L41_SPI_4_HP_GPIO_LED), >> > -- >> > 2.34.1 >> > >> >
On Tue, Jan 02, 2024 at 01:49:40PM +0800, Andy Chi wrote: > Hi Geraldo, > Yes, this model is not yet publicly available, so I can't type the > full name on this patch. > > (change to text plain mode to send out again.) Hi Andy, please be kind and next time refrain from top-posting, as it reverses the attention order for those reading the discussion for posterity. I think it is reasonable to mention it explicitly as an upcoming product in order not to cause any confusion and to make sure there's another janitorial patch later on filling in the model details. It's best to leave Takashi now to attend to your patch, and I congratulate both you as well as Canonical and HP for the effort towards bettering Linux. Thanks for addressing this quickly. Geraldo Nascimento > > Kr, > Andy > > > On Tue, Jan 2, 2024 at 1:47 PM Andy Chi <andy.chi@canonical.com> wrote: > > > > Hi Geraldo, > > Yes, this model is not yet publicly available, so I can't type the full name on this patch. > > > > Kr, > > Andy > > > > On Tue, Jan 2, 2024 at 12:25 PM Geraldo Nascimento <geraldogabriel@gmail.com> wrote: > >> > >> On Tue, Jan 02, 2024 at 10:49:15AM +0800, Andy Chi wrote: > >> > There is a HP ZBook which using ALC236 codec and need the > >> > ALC236_FIXUP_HP_MUTE_LED_MICMUTE_VREF quirk to make mute LED > >> > and micmute LED work. > >> > >> Hi Andy, > >> > >> I know Takashi in the name of smooth operation usually takes this kind > >> of simple patch, quickly, since it's a simple bugfix, unless there are > >> any objections, so I must raise one. > >> > >> Which HP ZBook? Which model? > >> > >> Is this a still unreleased model Canonical has had access under a NDA > >> for example? Or perhaps there is some other reason why you cannot name > >> the model yet? > >> > >> Either way, there is a need to publicly know the reason for the missing > >> model information. > >> > >> > > >> > Signed-off-by: Andy Chi <andy.chi@canonical.com> > >> > --- > >> > 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 7e9d8c1cedfc..1c8d5b58eb64 100644 > >> > --- a/sound/pci/hda/patch_realtek.c > >> > +++ b/sound/pci/hda/patch_realtek.c > >> > @@ -9939,6 +9939,7 @@ static const struct snd_pci_quirk alc269_fixup_tbl[] = { > >> > SND_PCI_QUIRK(0x103c, 0x8c70, "HP EliteBook 835 G11", ALC287_FIXUP_CS35L41_I2C_2_HP_GPIO_LED), > >> > SND_PCI_QUIRK(0x103c, 0x8c71, "HP EliteBook 845 G11", ALC287_FIXUP_CS35L41_I2C_2_HP_GPIO_LED), > >> > SND_PCI_QUIRK(0x103c, 0x8c72, "HP EliteBook 865 G11", ALC287_FIXUP_CS35L41_I2C_2_HP_GPIO_LED), > >> > + SND_PCI_QUIRK(0x103c, 0x8c96, "HP", ALC236_FIXUP_HP_MUTE_LED_MICMUTE_VREF), > >> > >> Same questioning as above. "HP" is IMHO unacceptable without a clear > >> reason for the missing model information. > >> > >> Thanks, > >> Geraldo Nascimento > >> > >> > SND_PCI_QUIRK(0x103c, 0x8ca4, "HP ZBook Fury", ALC245_FIXUP_CS35L41_SPI_2_HP_GPIO_LED), > >> > SND_PCI_QUIRK(0x103c, 0x8ca7, "HP ZBook Fury", ALC245_FIXUP_CS35L41_SPI_2_HP_GPIO_LED), > >> > SND_PCI_QUIRK(0x103c, 0x8cf5, "HP ZBook Studio 16", ALC245_FIXUP_CS35L41_SPI_4_HP_GPIO_LED), > >> > -- > >> > 2.34.1 > >> > > >> >
On Tue, Jan 2, 2024 at 2:42 PM Geraldo Nascimento <geraldogabriel@gmail.com> wrote: > > On Tue, Jan 02, 2024 at 01:49:40PM +0800, Andy Chi wrote: > > Hi Geraldo, > > Yes, this model is not yet publicly available, so I can't type the > > full name on this patch. > > > > (change to text plain mode to send out again.) > Hi Geraldo, > Hi Andy, please be kind and next time refrain from top-posting, > as it reverses the attention order for those reading the > discussion for posterity. > Got it, appreciate your feedback. > I think it is reasonable to mention it explicitly as an > upcoming product in order not to cause any confusion and > to make sure there's another janitorial patch later on > filling in the model details. > Should I mention it in the commit message? Or is it ok if I put the model name after it's on the market? > It's best to leave Takashi now to attend to your patch, and I > congratulate both you as well as Canonical and HP for the > effort towards bettering Linux. Cheers :) > > Thanks for addressing this quickly. > > Geraldo Nascimento > > > > > Kr, > > Andy > > > > > > On Tue, Jan 2, 2024 at 1:47 PM Andy Chi <andy.chi@canonical.com> wrote: > > > > > > Hi Geraldo, > > > Yes, this model is not yet publicly available, so I can't type the full name on this patch. > > > > > > Kr, > > > Andy > > > > > > On Tue, Jan 2, 2024 at 12:25 PM Geraldo Nascimento <geraldogabriel@gmail.com> wrote: > > >> > > >> On Tue, Jan 02, 2024 at 10:49:15AM +0800, Andy Chi wrote: > > >> > There is a HP ZBook which using ALC236 codec and need the > > >> > ALC236_FIXUP_HP_MUTE_LED_MICMUTE_VREF quirk to make mute LED > > >> > and micmute LED work. > > >> > > >> Hi Andy, > > >> > > >> I know Takashi in the name of smooth operation usually takes this kind > > >> of simple patch, quickly, since it's a simple bugfix, unless there are > > >> any objections, so I must raise one. > > >> > > >> Which HP ZBook? Which model? > > >> > > >> Is this a still unreleased model Canonical has had access under a NDA > > >> for example? Or perhaps there is some other reason why you cannot name > > >> the model yet? > > >> > > >> Either way, there is a need to publicly know the reason for the missing > > >> model information. > > >> > > >> > > > >> > Signed-off-by: Andy Chi <andy.chi@canonical.com> > > >> > --- > > >> > 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 7e9d8c1cedfc..1c8d5b58eb64 100644 > > >> > --- a/sound/pci/hda/patch_realtek.c > > >> > +++ b/sound/pci/hda/patch_realtek.c > > >> > @@ -9939,6 +9939,7 @@ static const struct snd_pci_quirk alc269_fixup_tbl[] = { > > >> > SND_PCI_QUIRK(0x103c, 0x8c70, "HP EliteBook 835 G11", ALC287_FIXUP_CS35L41_I2C_2_HP_GPIO_LED), > > >> > SND_PCI_QUIRK(0x103c, 0x8c71, "HP EliteBook 845 G11", ALC287_FIXUP_CS35L41_I2C_2_HP_GPIO_LED), > > >> > SND_PCI_QUIRK(0x103c, 0x8c72, "HP EliteBook 865 G11", ALC287_FIXUP_CS35L41_I2C_2_HP_GPIO_LED), > > >> > + SND_PCI_QUIRK(0x103c, 0x8c96, "HP", ALC236_FIXUP_HP_MUTE_LED_MICMUTE_VREF), > > >> > > >> Same questioning as above. "HP" is IMHO unacceptable without a clear > > >> reason for the missing model information. > > >> > > >> Thanks, > > >> Geraldo Nascimento > > >> > > >> > SND_PCI_QUIRK(0x103c, 0x8ca4, "HP ZBook Fury", ALC245_FIXUP_CS35L41_SPI_2_HP_GPIO_LED), > > >> > SND_PCI_QUIRK(0x103c, 0x8ca7, "HP ZBook Fury", ALC245_FIXUP_CS35L41_SPI_2_HP_GPIO_LED), > > >> > SND_PCI_QUIRK(0x103c, 0x8cf5, "HP ZBook Studio 16", ALC245_FIXUP_CS35L41_SPI_4_HP_GPIO_LED), > > >> > -- > > >> > 2.34.1 > > >> > > > >> >
On Tue, Jan 02, 2024 at 02:59:56PM +0800, Andy Chi wrote: > On Tue, Jan 2, 2024 at 2:42 PM Geraldo Nascimento > <geraldogabriel@gmail.com> wrote: > > > > On Tue, Jan 02, 2024 at 01:49:40PM +0800, Andy Chi wrote: > > > Hi Geraldo, > > > Yes, this model is not yet publicly available, so I can't type the > > > full name on this patch. > > > > > > (change to text plain mode to send out again.) > > > Hi Geraldo, > > Hi Andy, please be kind and next time refrain from top-posting, > > as it reverses the attention order for those reading the > > discussion for posterity. > > > Got it, appreciate your feedback. > > I think it is reasonable to mention it explicitly as an > > upcoming product in order not to cause any confusion and > > to make sure there's another janitorial patch later on > > filling in the model details. > > > Should I mention it in the commit message? Or is it ok if I put the > model name after it's on the market? Andy, in order to clear up any confusion that might arise later on, and to maintain high the quality of both code and git meta-information I think you should explicitly mention both in the commit info and commit message that this is an upcoming product. Also, instead of just "HP" in the commit proper, it should say something like "Upcoming HP ZBook". But that's just my advice, now you need to wait and hear Takashi's advice. We should always strive toward striking a healthy balance between companies' needs and the public nature of the development of the Linux kernel and ALSA. Unfortunately that means some details of upcoming models may end up catching up the attention of the specialized press. I encourage you to look on the bright side and think of it as free PR ;) Best Regards, Geraldo Nascimento > > > It's best to leave Takashi now to attend to your patch, and I > > congratulate both you as well as Canonical and HP for the > > effort towards bettering Linux. > Cheers :) > > > > > Thanks for addressing this quickly. > > > > Geraldo Nascimento > > > > > > > > Kr, > > > Andy > > > > > > > > > On Tue, Jan 2, 2024 at 1:47 PM Andy Chi <andy.chi@canonical.com> wrote: > > > > > > > > Hi Geraldo, > > > > Yes, this model is not yet publicly available, so I can't type the full name on this patch. > > > > > > > > Kr, > > > > Andy > > > > > > > > On Tue, Jan 2, 2024 at 12:25 PM Geraldo Nascimento <geraldogabriel@gmail.com> wrote: > > > >> > > > >> On Tue, Jan 02, 2024 at 10:49:15AM +0800, Andy Chi wrote: > > > >> > There is a HP ZBook which using ALC236 codec and need the > > > >> > ALC236_FIXUP_HP_MUTE_LED_MICMUTE_VREF quirk to make mute LED > > > >> > and micmute LED work. > > > >> > > > >> Hi Andy, > > > >> > > > >> I know Takashi in the name of smooth operation usually takes this kind > > > >> of simple patch, quickly, since it's a simple bugfix, unless there are > > > >> any objections, so I must raise one. > > > >> > > > >> Which HP ZBook? Which model? > > > >> > > > >> Is this a still unreleased model Canonical has had access under a NDA > > > >> for example? Or perhaps there is some other reason why you cannot name > > > >> the model yet? > > > >> > > > >> Either way, there is a need to publicly know the reason for the missing > > > >> model information. > > > >> > > > >> > > > > >> > Signed-off-by: Andy Chi <andy.chi@canonical.com> > > > >> > --- > > > >> > 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 7e9d8c1cedfc..1c8d5b58eb64 100644 > > > >> > --- a/sound/pci/hda/patch_realtek.c > > > >> > +++ b/sound/pci/hda/patch_realtek.c > > > >> > @@ -9939,6 +9939,7 @@ static const struct snd_pci_quirk alc269_fixup_tbl[] = { > > > >> > SND_PCI_QUIRK(0x103c, 0x8c70, "HP EliteBook 835 G11", ALC287_FIXUP_CS35L41_I2C_2_HP_GPIO_LED), > > > >> > SND_PCI_QUIRK(0x103c, 0x8c71, "HP EliteBook 845 G11", ALC287_FIXUP_CS35L41_I2C_2_HP_GPIO_LED), > > > >> > SND_PCI_QUIRK(0x103c, 0x8c72, "HP EliteBook 865 G11", ALC287_FIXUP_CS35L41_I2C_2_HP_GPIO_LED), > > > >> > + SND_PCI_QUIRK(0x103c, 0x8c96, "HP", ALC236_FIXUP_HP_MUTE_LED_MICMUTE_VREF), > > > >> > > > >> Same questioning as above. "HP" is IMHO unacceptable without a clear > > > >> reason for the missing model information. > > > >> > > > >> Thanks, > > > >> Geraldo Nascimento > > > >> > > > >> > SND_PCI_QUIRK(0x103c, 0x8ca4, "HP ZBook Fury", ALC245_FIXUP_CS35L41_SPI_2_HP_GPIO_LED), > > > >> > SND_PCI_QUIRK(0x103c, 0x8ca7, "HP ZBook Fury", ALC245_FIXUP_CS35L41_SPI_2_HP_GPIO_LED), > > > >> > SND_PCI_QUIRK(0x103c, 0x8cf5, "HP ZBook Studio 16", ALC245_FIXUP_CS35L41_SPI_4_HP_GPIO_LED), > > > >> > -- > > > >> > 2.34.1 > > > >> > > > > >> >
On Tue, 02 Jan 2024 08:45:07 +0100, Geraldo Nascimento wrote: > > On Tue, Jan 02, 2024 at 02:59:56PM +0800, Andy Chi wrote: > > On Tue, Jan 2, 2024 at 2:42 PM Geraldo Nascimento > > <geraldogabriel@gmail.com> wrote: > > > > > > On Tue, Jan 02, 2024 at 01:49:40PM +0800, Andy Chi wrote: > > > > Hi Geraldo, > > > > Yes, this model is not yet publicly available, so I can't type the > > > > full name on this patch. > > > > > > > > (change to text plain mode to send out again.) > > > > > Hi Geraldo, > > > Hi Andy, please be kind and next time refrain from top-posting, > > > as it reverses the attention order for those reading the > > > discussion for posterity. > > > > > Got it, appreciate your feedback. > > > I think it is reasonable to mention it explicitly as an > > > upcoming product in order not to cause any confusion and > > > to make sure there's another janitorial patch later on > > > filling in the model details. > > > > > Should I mention it in the commit message? Or is it ok if I put the > > model name after it's on the market? > > Andy, in order to clear up any confusion that might arise later on, and > to maintain high the quality of both code and git meta-information I think > you should explicitly mention both in the commit info and commit message > that this is an upcoming product. > > Also, instead of just "HP" in the commit proper, it should say something > like "Upcoming HP ZBook". > > But that's just my advice, now you need to wait and hear Takashi's advice. > > We should always strive toward striking a healthy balance between companies' > needs and the public nature of the development of the Linux kernel and ALSA. > Unfortunately that means some details of upcoming models may end up > catching up the attention of the specialized press. > > I encourage you to look on the bright side and think of it as free PR ;) I merged as is now with some comments about the new model. It's fine with the current patch if the entry will be updated later with a proper model name; having the code fix before the name definition would be more important. thanks, Takashi
On Tue, Jan 02, 2024 at 03:57:47PM +0100, Takashi Iwai wrote: > I merged as is now with some comments about the new model. > It's fine with the current patch if the entry will be updated later > with a proper model name; having the code fix before the name > definition would be more important. Thanks Takashi, actually it's a commit from the ideal world. I wish more integrators would work together with Linux companies to launch new products that work smoothly, like we see here. I appreciate you adding the note about it being a new model without a proper name. It's a middle path that doesn't overwork you and Andy, and my concern is still addressed that way. Regards, Geraldo Nascimento > > > thanks, > > Takashi
diff --git a/sound/pci/hda/patch_realtek.c b/sound/pci/hda/patch_realtek.c index 7e9d8c1cedfc..1c8d5b58eb64 100644 --- a/sound/pci/hda/patch_realtek.c +++ b/sound/pci/hda/patch_realtek.c @@ -9939,6 +9939,7 @@ static const struct snd_pci_quirk alc269_fixup_tbl[] = { SND_PCI_QUIRK(0x103c, 0x8c70, "HP EliteBook 835 G11", ALC287_FIXUP_CS35L41_I2C_2_HP_GPIO_LED), SND_PCI_QUIRK(0x103c, 0x8c71, "HP EliteBook 845 G11", ALC287_FIXUP_CS35L41_I2C_2_HP_GPIO_LED), SND_PCI_QUIRK(0x103c, 0x8c72, "HP EliteBook 865 G11", ALC287_FIXUP_CS35L41_I2C_2_HP_GPIO_LED), + SND_PCI_QUIRK(0x103c, 0x8c96, "HP", ALC236_FIXUP_HP_MUTE_LED_MICMUTE_VREF), SND_PCI_QUIRK(0x103c, 0x8ca4, "HP ZBook Fury", ALC245_FIXUP_CS35L41_SPI_2_HP_GPIO_LED), SND_PCI_QUIRK(0x103c, 0x8ca7, "HP ZBook Fury", ALC245_FIXUP_CS35L41_SPI_2_HP_GPIO_LED), SND_PCI_QUIRK(0x103c, 0x8cf5, "HP ZBook Studio 16", ALC245_FIXUP_CS35L41_SPI_4_HP_GPIO_LED),