On Mon, 2013-02-25 at 14:53 -0700, Shuah Khan wrote: > On Mon, 2013-02-25 at 14:23 -0700, Bjorn Helgaas wrote: > > On Mon, Feb 25, 2013 at 9:37 AM, Shuah Khan <shuah.khan@xxxxxx> wrote: > > > On Wed, 2013-02-20 at 18:19 -0700, Bjorn Helgaas wrote: > > >> On Mon, Feb 11, 2013 at 4:00 PM, Shuah Khan <shuah.khan@xxxxxx> wrote: > > > > It's not nice and consistent, but it does follow the simple rule of > > "don't expose things to user-space unnecessarily." We might want to > > add a comment to keep somebody from cleaning it up later. > > ok. Will resend patches adding the new defines to linux/pci.h and > renaming PCI_BUS() to PCI_BUS_NR() or PCI_BUS_NUM() like you suggested. > > Thanks, > -- Shuah > Bjorn/Joerg, I added PCI_BUS_NUM() amd PCI_DEVID() to linux/pci.h. Please note that changing PCI_BUS() to PCI_BUS_NUM() required additional changes to AMD IOMMU source files and aer driver. Essentially in addition to removing local PCI_BUS() define, PCI_BUS() usages are changed to PCI_BUS_NUM(). I am resending the patches. Thanks, -- Shuah -- To unsubscribe from this list: send the line "unsubscribe linux-pci" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html