This define is not actually used anywhere that I know of. I just wanted
to be consistent and correct, following what was previously done. I
have been wondering if I should be adding devices to the pci_ids.h file
that are not being currently used. It seems like most drivers are not
using these defines and are just using the DID's directly. In the
future, should I only be add devices that are actually using the defines
somewhere?
Thanks,
Jason
>-----Original Message-----
>From: Jeff Garzik [mailto:[email protected]]
>Sent: Friday, July 29, 2005 2:49 PM
>To: Gaston, Jason D
>Cc: [email protected]; [email protected]; [email protected]
>Subject: Re: [PATCH 2.6.13-rc4 1/1] pci_ids: patch for Intel ICH7R
>
>Jason Gaston wrote:
>> Hello,
>>
>> This patch adds the Intel ICH7R SATA RAID DID to the pci_ids.h file.
>This patch was built against the 2.6.13-rc4 kernel.
>> If acceptable, please apply.
>>
>> Thanks,
>>
>> Jason Gaston
>>
>> Signed-off-by: Jason Gaston <[email protected]>
>>
>> --- linux-2.6.13-rc4/include/linux/pci_ids.h.orig 2005-07-29
>09:06:03.841520568 -0700
>> +++ linux-2.6.13-rc4/include/linux/pci_ids.h 2005-07-29
>09:06:42.256680576 -0700
>> @@ -2454,6 +2454,7 @@
>> #define PCI_DEVICE_ID_INTEL_ICH7_3 0x27c1
>> #define PCI_DEVICE_ID_INTEL_ICH7_30 0x27b0
>> #define PCI_DEVICE_ID_INTEL_ICH7_31 0x27bd
>> +#define PCI_DEVICE_ID_INTEL_ICH7_4 0x27c3
>> #define PCI_DEVICE_ID_INTEL_ICH7_5 0x27c4
>> #define PCI_DEVICE_ID_INTEL_ICH7_6 0x27c5
>> #define PCI_DEVICE_ID_INTEL_ICH7_7 0x27c8
>
>Where is this actually used?
>
>I purposefully do not use PCI_DEVICE_ID_xxx in my drivers, because I
>feel that linux/pci_ids.h is constantly patched for little value.
>
>Device ids, unlike vendor ids, are largely single-use constants.
>
> Jeff
>
>
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
[Index of Archives]
[Kernel Newbies]
[Netfilter]
[Bugtraq]
[Photo]
[Gimp]
[Yosemite News]
[MIPS Linux]
[ARM Linux]
[Linux Security]
[Linux RAID]
[Video 4 Linux]
[Linux for the blind]
| |
 |