linux-kernel - Re: [PATCH linux-2.6] hda_intel: ALSA HD Audio patch for Intel Cougar Point DeviceIDs, enables snoop

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] [thread-next>] [day] [month] [year] [list]

Message-ID: <s5h4ol82xhd.wl%tiwai@suse.de>
Date:	Tue, 23 Feb 2010 08:59:58 +0100
From:	Takashi Iwai <tiwai@...e.de>
To:	Seth Heasley <seth.heasley@...el.com>
Cc:	perex@...ex.cz, alsa-devel@...a-project.org,
	linux-kernel@...r.kernel.org
Subject: Re: [PATCH linux-2.6] hda_intel: ALSA HD Audio patch for Intel Cougar Point DeviceIDs, enables snoop

At Mon, 22 Feb 2010 17:31:09 -0800,
Seth Heasley wrote:
> 
> This patch adds the Intel Cougar Point (PCH) HD Audio Controller DeviceIDs, and enables snoop, eliminating static during
> playback.  This patch supersedes the previous Cougar Point audio patch.

We can't replace the patch in the published tree.  This has to be an
additional fix.

Anyway, I fixed it manually now.  Thanks.


Takashi

> Signed-off-by: Seth Heasley <seth.heasley@...el.com>
> 
> --- sound/pci/hda/hda_intel.c.orig	2010-02-11 11:37:43.000000000 -0800
> +++ sound/pci/hda/hda_intel.c	2010-02-22 10:05:48.000000000 -0800
> @@ -449,6 +449,7 @@
>  /* driver types */
>  enum {
>  	AZX_DRIVER_ICH,
> +	AZX_DRIVER_PCH,
>  	AZX_DRIVER_SCH,
>  	AZX_DRIVER_ATI,
>  	AZX_DRIVER_ATIHDMI,
> @@ -463,6 +464,7 @@
>  
>  static char *driver_short_names[] __devinitdata = {
>  	[AZX_DRIVER_ICH] = "HDA Intel",
> +	[AZX_DRIVER_PCH] = "HDA Intel PCH",
>  	[AZX_DRIVER_SCH] = "HDA Intel MID",
>  	[AZX_DRIVER_ATI] = "HDA ATI SB",
>  	[AZX_DRIVER_ATIHDMI] = "HDA ATI HDMI",
> @@ -1065,6 +1067,7 @@
>  				0x01, NVIDIA_HDA_ENABLE_COHBIT);
>  		break;
>  	case AZX_DRIVER_SCH:
> +	case AZX_DRIVER_PCH:
>  		pci_read_config_word(chip->pci, INTEL_SCH_HDA_DEVC, &snoop);
>  		if (snoop & INTEL_SCH_HDA_DEVC_NOSNOOP) {
>  			pci_write_config_word(chip->pci, INTEL_SCH_HDA_DEVC,
> @@ -2415,6 +2418,7 @@
>  	if (bdl_pos_adj[dev] < 0) {
>  		switch (chip->driver_type) {
>  		case AZX_DRIVER_ICH:
> +		case AZX_DRIVER_PCH:
>  			bdl_pos_adj[dev] = 1;
>  			break;
>  		default:
> @@ -2693,6 +2697,7 @@
>  	{ PCI_DEVICE(0x8086, 0x3a6e), .driver_data = AZX_DRIVER_ICH },
>  	/* PCH */
>  	{ PCI_DEVICE(0x8086, 0x3b56), .driver_data = AZX_DRIVER_ICH },
> +	{ PCI_DEVICE(0x8086, 0x1c20), .driver_data = AZX_DRIVER_PCH },
>  	/* SCH */
>  	{ PCI_DEVICE(0x8086, 0x811b), .driver_data = AZX_DRIVER_SCH },
>  	/* ATI SB 450/600 */
> 
--
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/