Re: [PATCH v3] ALSA: hda/realtek - Add HW8326 support

["huangwenhui" <huangwenhuia@xxxxxxxxxxxxx>]

I send a patch now.


Thanks.
 
------------------ Original ------------------
From: &nbsp;"Takashi&nbsp;Iwai"<tiwai@xxxxxxx&gt;;
Date: &nbsp;Wed, Jun 8, 2022 03:56 PM
To: &nbsp;"jeremy.szu"<jeremy.szu@xxxxxxxxxxxxx&gt;; 
Cc: &nbsp;"huangwenhui"<huangwenhuia@xxxxxxxxxxxxx&gt;; "tiwai"<tiwai@xxxxxxxx&gt;; "perex"<perex@xxxxxxxx&gt;; "alsa-devel"<alsa-devel@xxxxxxxxxxxxxxxx&gt;; "kailang"<kailang@xxxxxxxxxxx&gt;; "tanureal"<tanureal@xxxxxxxxxxxxxxxxxxxxx&gt;; "linux-kernel"<linux-kernel@xxxxxxxxxxxxxxx&gt;; "wse"<wse@xxxxxxxxxxxxxxxxxxx&gt;; "hui.wang"<hui.wang@xxxxxxxxxxxxx&gt;; "sami"<sami@xxxxxxxx&gt;; "cam"<cam@xxxxxxxxxxx&gt;; 
Subject: &nbsp;Re: [PATCH v3] ALSA: hda/realtek - Add HW8326 support

&nbsp;

On Wed, 08 Jun 2022 09:53:32 +0200,
Jeremy Szu wrote:
&gt; 
&gt; and better to follow the order to easy to find the codec once the list
&gt; growing next time, e.g.
&gt; 
&gt; @@ -651,6 +651,7 @@ static const struct hda_vendor_id hda_vendor_ids[] = {
&gt;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; { 0x1095, "Silicon Image" },
&gt;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; { 0x10de, "Nvidia" },
&gt;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; { 0x10ec, "Realtek" },
&gt; +&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; { 0x19e5, "Huawei" },
&gt;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; { 0x1102, "Creative" },
&gt;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; { 0x1106, "VIA" },
&gt;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; { 0x111d, "IDT" },
&gt; 
&gt; to
&gt; 
&gt;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; { 0x14f1, "Conexant" },
&gt;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; { 0x17e8, "Chrontel" },
&gt;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; { 0x1854, "LG" },
&gt; +&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; { 0x19e5, "Huawei" },
&gt;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; { 0x1aec, "Wolfson Microelectronics" },
&gt;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; { 0x1af4, "QEMU" },
&gt; 
&gt; and so on.

Indeed.&nbsp; Care to send an additional patch?


thanks,

Takashi