On Wed, Oct 12, 2016 at 11:04:40AM -0500, Bjorn Helgaas wrote:
> On Wed, Oct 12, 2016 at 08:28:55AM -0500, Bjorn Helgaas wrote:
> > - Add local "dev" pointers to reduce repetition of things like
> > "&pdev->dev".
> >
> > - Remove platform drvdata because it appears unused (we called
> > platform_set_drvdata() but not platform_get_drvdata()).
> >
> > - Use generic DesignWare accessors when possible.
> >
> > - Add register accessors to encapsulate regmap usage.
> >
> > - 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 patch because it was a lot of churn for
> > questionable benefit:
> > PCI: artpec6: Name private struct pointer "artpec6" consistently
> >
> > ---
> >
> > Bjorn Helgaas (7):
> > PCI: artpec6: Add local struct device pointers
> > PCI: artpec6: Remove unused platform data
> > PCI: artpec6: Add register accessors
> > PCI: artpec6: Use generic DesignWare accessors
> > PCI: artpec6: Remove unnecessary artpec6_pcie_link_up()
> > PCI: artpec6: Pass device-specific struct to internal functions
> > PCI: artpec6: Add resource name comments
> >
> >
> > drivers/pci/host/pcie-artpec6.c | 115 ++++++++++++++++++++-------------------
> > 1 file changed, 59 insertions(+), 56 deletions(-)
>
> I applied these to pci/host-artpec for v4.9. I hope to ask Linus to
> pull them tomorrow, so if you see any issues, let me know soon.
Looks good, although I haven't been able run a test on real hardware since I'm
currently at ELCE in Berlin... :-)
/^JN - Jesper Nilsson
--
Jesper Nilsson -- jesper.nilsson@xxxxxxxx
--
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
