On Wed, Oct 12, 2016 at 08:25:11AM -0500, Bjorn Helgaas wrote: > - Remove platform drvdata because it appears unused (we called > platform_set_drvdata() but not platform_get_drvdata()). > > - Remove redundant struct members. > > - Use generic DesignWare accessors when possible. > > - Pass device-specific struct to internal functions for consistency. > > Nothing here should change the behavior of the driver. > > Changes from v1: > I dropped the following patches because they were a lot of churn for > questionable benefit: > PCI: armada: Name private struct pointer "armada8k" consistently > PCI: armada: Add register accessors > > --- > > Bjorn Helgaas (6): > PCI: armada: Remove unused platform data > PCI: armada: Add local base pointer > PCI: armada: Remove redundant struct armada8k_pcie.base > PCI: armada: Use generic DesignWare accessors > PCI: armada: Pass device-specific struct to internal functions > PCI: armada: Reorder struct armada8k_pcie > > > drivers/pci/host/pcie-armada8k.c | 78 ++++++++++++++++++-------------------- I applied these to pci/host-armada for v4.9. I hope to ask Linus to pull them tomorrow, so if you see any issues, let me know soon. -- 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