Re: [PATCH 0/4] PCI / DPC: Few improvements

[Mika Westerberg <mika.westerberg@linux.intel.com>]

On Tue, Jun 21, 2016 at 01:57:20PM -0500, Bjorn Helgaas wrote:
> On Mon, Jun 06, 2016 at 04:06:04PM +0300, Mika Westerberg wrote:
> > Hi,
> > 
> > This series includes few improvements for the PCIe DPC driver. First we
> > introduce module_pci_port_service() which allows reducing of boilerblate
> > code used in the DPC driver. In addition we convert the driver to use
> > managed resource functions (devm) to get rid of even more boilerblate.
> > 
> > I do not have hardware supporting DPC so these are only compile tested.
> > 
> > Mika Westerberg (4):
> >   PCI / PCIe: Introduce module_pci_port_service()
> >   PCI / DPC: Convert to use module_pcie_port_service()
> >   PCI / DPC: Convert the driver to use devm_* functions
> >   PCI / DPC: Fix whitespace in struct dpc_dev
> > 
> >  drivers/pci/pcie/pcie-dpc.c | 30 +++++++-----------------------
> >  include/linux/pcieport_if.h | 13 +++++++++++++
> >  2 files changed, 20 insertions(+), 23 deletions(-)
> 
> I applied the last two, with Keith's acks, to pci/dpc for v4.8.
> 
> I didn't apply the first two because I don't think these port drivers
> should be modules.  I think DPC and thunderbolt are the only candidates
> right now, and I think it's a stretch to think about either one as a
> module.

Understood. Thanks.