Re: [PATCH] ALSA: hda/realtek: fix mute/micmute LEDs for a HP ZBook

[Andy Chi]

On Tue, Jan 2, 2024 at 2:42 PM Geraldo Nascimento
<geraldogabriel@xxxxxxxxx> 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@xxxxxxxxxxxxx> 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@xxxxxxxxx> 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@xxxxxxxxxxxxx>
> > >> > ---
> > >> >  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
> > >> >
> > >> >