Re: [PATCH] pci_ids.h: add new Atheros USB vendor ID

From: Luis R. Rodriguez
Date: Thu Aug 06 2009 - 00:27:23 EST


On Wed, Aug 5, 2009 at 9:16 PM, Marcel Holtmann<marcel@xxxxxxxxxxxx> wrote:
> Hi Luiz,
>
>> New Atheros USB devices will use 0x0cf3. The first device
>> will be ar9271, for which we are adding support for.
>>
>> Cc: zhifeng.cai@xxxxxxxxxxx
>> Cc: stephen.chen@xxxxxxxxxxx
>> Signed-off-by: Luis R. Rodriguez <lrodriguez@xxxxxxxxxxx>
>> ---
>>
>> John, sending this through you as this is only
>> specific to wireless, at least at the moment.
>>
>> Âinclude/linux/pci_ids.h | Â Â1 +
>> Â1 files changed, 1 insertions(+), 0 deletions(-)
>>
>> diff --git a/include/linux/pci_ids.h b/include/linux/pci_ids.h
>> index 73b46b6..27f4e6b 100644
>> --- a/include/linux/pci_ids.h
>> +++ b/include/linux/pci_ids.h
>> @@ -2170,6 +2170,7 @@
>> Â#define PCI_DEVICE_ID_BCM1250_HT Â Â 0x0002
>>
>> Â#define PCI_VENDOR_ID_ATHEROS Â Â Â Â Â Â Â Â0x168c
>> +#define PCI_VENDOR_ID_ATHEROS_USB Â Â0x0cf3
>
> this is confusing the hell out of me. What are you adding here.

No you're right, my fault.

> A USB
> vendor id or a PCI one. Is the ar9271 a PCI device or a USB device?

Its only USB.

Thanks... hm I see no USB vendor id headers. Guess we won't need to add this.

Luis
--
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/