On Sun, Apr 23, 2006 at 10:50:38AM +0300, Pekka Enberg wrote:
> On Thu, Apr 20, 2006 at 04:04:07PM +0100, Bert Thomas wrote:
> > > static const struct pci_device_id cif50_ids[] = {
> > > {
> > > .vendor = 0x10B5,
> > > .device = 0x9050,
> > > .subvendor = PCI_ANY_ID, //0x10B5,
> > > .subdevice = PCI_ANY_ID, //0x1080,
> > > .class = PCI_ANY_ID,
> > > .class_mask = PCI_ANY_ID
> > > },
>
> On 4/22/06, Greg KH <[email protected]> wrote:
> > Try the PCI_DEVICE() macro here instead.
> >
> > But that should not matter, this should work, I don't know why it
> > doesn't sorry.
>
> No device class will ever match the above class and class_mask.
> Changing them to zero makes it work according to Bert.
Ah, yeah, that would work, good catch. If you used the PCI_DEVICE()
macro, it would have also worked :)
thanks,
greg k-h
-
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]
[Stuff]
[Gimp]
[Yosemite News]
[MIPS Linux]
[ARM Linux]
[Linux Security]
[Linux RAID]
[Video 4 Linux]
[Linux for the blind]
[Linux Resources]
