On Sat, Feb 28, 2009 at 07:13:03PM +0100, Jiri Slaby wrote:
>> +            (id->subdevice == (u16) PCI_ANY_ID ||
>> +             id->subdevice == dev->subsystem_device));
>> +}
>
> This is wrong. Subids might legally be 0xffff which is (u16)PCI_ANY_ID.
>
> Why not use pci_device_id table with pci_match_id and the info encoded  
> in driver_data?

Ok, good point.  Yeah, that would work -- it's ok to use that for purposes
other than driver match?

-- 
Bob Copeland %% www.bobcopeland.com

_______________________________________________
ath5k-devel mailing list
ath5k-devel@lists.ath5k.org
https://lists.ath5k.org/mailman/listinfo/ath5k-devel

Reply via email to