Sorry for the late reply; maybe this has already been queued up somewhere, so this might be moot.
I usually capitalize the "PCI" and first letter of the subject, like: PCI: Add DT based ARM Versatile PCI host driver And I try to ask for MAINTAINER updates since these drivers are under drivers/pci, but I can't maintain them all myself. On Tue, Dec 30, 2014 at 01:28:33PM -0600, Rob Herring wrote: > From: Rob Herring <[email protected]> > > This converts the Versatile PCI host code to a platform driver using > the commom DT parsing and setup. The driver uses only an empty ARM > pci_sys_data struct and does not use pci_common_init_dev init function. > The old host code will be removed in a subsequent commit when Versatile > is completely converted to DT. > > I've tested this on QEMU with the sym53c8xx driver in both i/o and > memory mapped modes. > > Signed-off-by: Rob Herring <[email protected]> > Cc: Bjorn Helgaas <[email protected]> > Cc: [email protected] > Cc: Russell King <[email protected]> > Cc: Linus Walleij <[email protected]> > Cc: Peter Maydell <[email protected]> Looks fine to me. Acked-by: Bjorn Helgaas <[email protected]> > +++ b/drivers/pci/host/pci-versatile.c > ... > + /* > + * We need to discover the PCI core first to configure itself > + * before the main PCI probing is performed Unusual to have two spaces between the "*" and the comments here. > +MODULE_LICENSE("GPLv2"); This needs to be "GPL v2" per license_is_gpl_compatible(). -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
