Re: [PATCH 2.6.8-rc2] intel8x0.c to include CK804 audio support

[Andrew Morton]

"Andrew Chew" <achew@xxxxxxxxxx> wrote:
>
> This patch updates include/linux/pci_ids.h with the CK804 audio
>  controller ID, and adds the CK804 audio controller to the
>  sound/pci/intel8x0.c audio driver.

I'm getting many workwrapped and tab-replaced patches nowadays.  Could
people pleeeeze ensure that their email clients are sending unmangled
patches?

I fixed this one up.  I usually do :(


>  --- linux-2.6.8-rc2/include/linux/pci_ids.h	2004-07-21
>  15:22:57.000000000 -0700
>  +++ linux/include/linux/pci_ids.h	2004-07-20 18:49:22.000000000
>  -0700
>  @@ -1071,6 +1071,7 @@
>   #define PCI_DEVICE_ID_NVIDIA_NFORCE_CK804_SATA2	0x0055
>   #define PCI_DEVICE_ID_NVIDIA_NVENET_8		0x0056
>   #define PCI_DEVICE_ID_NVIDIA_NVENET_9		0x0057
>  +#define PCI_DEVICE_ID_NVIDIA_CK804_AUDIO	0x0059
>   #define PCI_DEVICE_ID_NVIDIA_NFORCE2_IDE	0x0065
>   #define PCI_DEVICE_ID_NVIDIA_NVENET_2		0x0066
>   #define PCI_DEVICE_ID_NVIDIA_MCP2_AUDIO		0x006a
>  --- linux-2.6.8-rc2/sound/pci/intel8x0.c	2004-07-21
>  15:22:59.000000000 -0700
>  +++ linux/sound/pci/intel8x0.c	2004-07-20 18:52:07.000000000 -0700
>  @@ -155,6 +155,9 @@
>   #ifndef PCI_DEVICE_ID_NVIDIA_CK8S_AUDIO
>   #define PCI_DEVICE_ID_NVIDIA_CK8S_AUDIO	0x00ea
>   #endif
>  +#ifndef PCI_DEVICE_ID_NVIDIA_CK804_AUDIO
>  +#define PCI_DEVICE_ID_NVIDIA_CK804_AUDIO 0x0059
>  +#endif
>   

Why does the driver need this ifdef?  We just added the ID to pci_ids.h?
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/