linux-kernel - Re: [PATCH v4] ALSA: hda/realtek: Add quirks for Unis H3C Desktop B760 & Q760

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <87mt0e7ne2.wl-tiwai@suse.de>
Date:   Sun, 02 Jul 2023 10:01:25 +0200
From:   Takashi Iwai <tiwai@...e.de>
To:     dengxiang <dengxiang@...china.com>
Cc:     tiwai@...e.com, yangyingliang@...wei.com, perex@...ex.cz,
        linux-kernel@...r.kernel.org
Subject: Re: [PATCH v4] ALSA: hda/realtek: Add quirks for Unis H3C Desktop B760 & Q760

On Sat, 01 Jul 2023 10:04:30 +0200,
dengxiang wrote:
> 
> These models use NSIWAY amplifiers for internal speaker, but cannot put
> sound outside from these amplifiers. So eapd verbs are needed to initialize
> the amplifiers. They can be added during boot to get working sound out
> of internal speaker.
> 
> ---
> V4:
> - put at the right position to keep the PCI SSID order for the whole.
> V3:
> - do some adjustment for eapd verbs and keep the capital letters for
>   ALC_* enums.
> v2:
> - remove warning about unused variable 'spec'.
> 
> Signed-off-by: dengxiang <dengxiang@...china.com>

Signed-off-by tag must be put before the line '---'.
But, above all....

> @@ -11772,6 +11780,7 @@ static const struct snd_pci_quirk alc662_fixup_tbl[] = {
>  	SND_PCI_QUIRK(0x1043, 0x1bf3, "ASUS N76VZ", ALC662_FIXUP_BASS_MODE4_CHMAP),
>  	SND_PCI_QUIRK(0x1043, 0x8469, "ASUS mobo", ALC662_FIXUP_NO_JACK_DETECT),
>  	SND_PCI_QUIRK(0x105b, 0x0cd6, "Foxconn", ALC662_FIXUP_ASUS_MODE2),
> +	SND_PCI_QUIRK(0x10ec, 0x0897, "UNIS H3C X500S", ALC897_FIXUP_UNIS_H3C_X500S),

Now I noticed that 10ec is a generic vendor id for Realtek, so it's
not really specific to your device.  It implies that your BIOS is
still not mature.

I'm fine to take the change for the model name, so that the workaround
can be added via model option, but rather avoid applying the quirk
from this SSID.

Note that there are already a few similar entries, but those are only
for the early reference boards.


thanks,

Takashi