linux-kernel - Re: [PATCH] ALSA: hda - Add new GPU codec ID 0x10de007d to snd-hda

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 for Android: free password hash cracker in your pocket

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

Message-ID: <55ADF924.5050401@nvidia.com>
Date:	Tue, 21 Jul 2015 00:47:48 -0700
From:	Aaron Plattner <aplattner@...dia.com>
To:	Takashi Iwai <tiwai@...e.de>
CC:	Jaroslav Kysela <perex@...ex.cz>, <alsa-devel@...a-project.org>,
	<linux-kernel@...r.kernel.org>
Subject: Re: [PATCH] ALSA: hda - Add new GPU codec ID 0x10de007d to snd-hda

On 07/21/2015 12:39 AM, Takashi Iwai wrote:
> On Tue, 21 Jul 2015 02:14:14 +0200,
> Aaron Plattner wrote:
>>
>> Vendor ID 0x10de007d is used by a yet-to-be-named GPU chip.
>>
>> Signed-off-by: Aaron Plattner <aplattner@...dia.com>
> 
> Applied, thanks.  But...
> 
>> ---
>> This chip also has the 2-ch audio swapping bug, so patch_nvhdmi is appropriate
>> here.
> 
> Please include such important information into the changelog itself at
> the next time.  I copied it manually.

Oh, okay, sorry.  I figured that since the whole point of patch_nvhdmi
is the 2ch workaround, it should be obvious from context but it
certainly can't hurt to have it in the commit itself too.  Thanks for
taking care of it!

Sincerely,
Aaron

> Takashi
> 
>>
>>  sound/pci/hda/patch_hdmi.c | 2 ++
>>  1 file changed, 2 insertions(+)
>>
>> diff --git a/sound/pci/hda/patch_hdmi.c b/sound/pci/hda/patch_hdmi.c
>> index 95158914cc6c..a97db5fc8a15 100644
>> --- a/sound/pci/hda/patch_hdmi.c
>> +++ b/sound/pci/hda/patch_hdmi.c
>> @@ -3512,6 +3512,7 @@ static const struct hda_codec_preset snd_hda_preset_hdmi[] = {
>>  { .id = 0x10de0070, .name = "GPU 70 HDMI/DP",	.patch = patch_nvhdmi },
>>  { .id = 0x10de0071, .name = "GPU 71 HDMI/DP",	.patch = patch_nvhdmi },
>>  { .id = 0x10de0072, .name = "GPU 72 HDMI/DP",	.patch = patch_nvhdmi },
>> +{ .id = 0x10de007d, .name = "GPU 7d HDMI/DP",	.patch = patch_nvhdmi },
>>  { .id = 0x10de8001, .name = "MCP73 HDMI",	.patch = patch_nvhdmi_2ch },
>>  { .id = 0x11069f80, .name = "VX900 HDMI/DP",	.patch = patch_via_hdmi },
>>  { .id = 0x11069f81, .name = "VX900 HDMI/DP",	.patch = patch_via_hdmi },
>> @@ -3576,6 +3577,7 @@ MODULE_ALIAS("snd-hda-codec-id:10de0067");
>>  MODULE_ALIAS("snd-hda-codec-id:10de0070");
>>  MODULE_ALIAS("snd-hda-codec-id:10de0071");
>>  MODULE_ALIAS("snd-hda-codec-id:10de0072");
>> +MODULE_ALIAS("snd-hda-codec-id:10de007d");
>>  MODULE_ALIAS("snd-hda-codec-id:10de8001");
>>  MODULE_ALIAS("snd-hda-codec-id:11069f80");
>>  MODULE_ALIAS("snd-hda-codec-id:11069f81");
>> -- 
>> 2.4.6
>>
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/