Re: [PATCH v2] PCI: Remove unused includes and superfluous struct declaration

From: Rob Herring
Date: Tue Sep 03 2019 - 13:20:16 EST


On Tue, Sep 03, 2019 at 01:30:59PM +0200, Krzysztof Wilczynski wrote:
> Remove <linux/pci.h> and <linux/msi.h> from being included
> directly as part of the include/linux/of_pci.h, and remove
> superfluous declaration of struct of_phandle_args.
>
> Move users of include <linux/of_pci.h> to include <linux/pci.h>
> and <linux/msi.h> directly rather than rely on both being
> included transitively through <linux/of_pci.h>.
>
> Signed-off-by: Krzysztof Wilczynski <kw@xxxxxxxxx>
> ---
> drivers/iommu/of_iommu.c | 2 ++
> drivers/irqchip/irq-gic-v2m.c | 1 +
> drivers/irqchip/irq-gic-v3-its-pci-msi.c | 1 +
> drivers/pci/controller/dwc/pcie-designware-host.c | 1 +
> drivers/pci/controller/pci-aardvark.c | 1 +
> drivers/pci/controller/pci-thunder-pem.c | 1 +
> drivers/pci/pci.c | 1 +
> drivers/pci/probe.c | 1 +
> include/linux/of_pci.h | 5 ++---
> 9 files changed, 11 insertions(+), 3 deletions(-)

Reviewed-by: Rob Herring <robh@xxxxxxxxxx>