On Thu, 26 Mar 2009 14:05:31 +1100 Stephen Rothwell <sfr@xxxxxxxxxxxxxxxx> wrote: > Hi Jesse, > > On Wed, 25 Mar 2009 08:55:50 -0700 Jesse Barnes > <jbarnes@xxxxxxxxxxxxxxxx> wrote: > > > > Ok fix pushed. Yeah it does seem weird. This file clearly uses PCI > > defines... > > Thanks. > > Actually all that file required was a "forward" declaration of "struct > pci_dev". It was not accessing the pci_dev in any way, just passing > pointers to them on to other routines. I guess it got such a > declaration indirectly. > > /me likes to understand ... Looked liked it used the PCI MSI capability ID offset though? In that case it would really want at least pci_regs.h, but pci.h seems a little more complete in this case. -- Jesse Barnes, Intel Open Source Technology Center -- To unsubscribe from this list: send the line "unsubscribe linux-next" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html