Re: [PATCH v2 0/6] PCI: armada: Cleanups

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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



[Index of Archives]     [DMA Engine]     [Linux Coverity]     [Linux USB]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [Greybus]

  Powered by Linux