On Mon, Mar 11, 2013 at 11:23 PM, Andrew Cooks <acooks@xxxxxxxxx> wrote: > On Fri, Mar 8, 2013 at 12:24 AM, Bjorn Helgaas <bhelgaas@xxxxxxxxxx> wrote: >> On Thu, Mar 7, 2013 at 7:28 AM, <yxlraid@xxxxxxxxx> wrote: >>> From: Xiangliang Yu <yuxiangl@xxxxxxxxxxx> >>> >>> Define PCI_VENDOR_MARVELL_ID_EXT macro for 0x1b4b vendor ID >> >> "PCI_VENDOR_MARVELL_ID_EXT" is not actually the symbol you defined below. >> >> Per the comment at the top of pci_ids.h, we don't add things here >> unless they are shared between multiple drivers. >> >> That said, the 0x1b4b vendor ID is used in mv_init.c, mvumi.h, >> mvumi.c, and ahci.c. If you want to unify all those so they all use >> same symbol, and define that symbol here in pci_ids.h, that would be >> fine with me. >> > > This symbol will also be useful for the quirk that creates iommu > context entries for Marvell devices that use incorrect dma source > tags. I'll update and resend that patch when this change has been > merged. Andrew - A new series that includes the 0x1b4b vendor ID change was posted last night so you may want to follow that: https://lkml.org/lkml/2013/3/21/12 > > Thanks > Andrew > -- > 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 -- 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