[+cc Lorenzo]
On Sun, Nov 26, 2017 at 01:02:09AM +0530, Manikanta Maddireddy wrote:
> pci_flags variable is used in inline functions in 'pci.h', Tegra PCIe
> driver use one of these functions pci_add_flags() and includes 'pci.h'.
> Export pci_flags to allow Tegra PCIe host controller driver to be
> compiled as loadable kernel module.
Here's the usage in tegra_pcie_probe():
pci_add_flags(PCI_REASSIGN_ALL_RSRC | PCI_REASSIGN_ALL_BUS);
We've probably had this discussion before, but I don't know why Tegra
needs PCI_REASSIGN_ALL_RSRC and PCI_REASSIGN_ALL_BUS.
I would prefer to drop this usage of pci_add_flags() if possible. It
seems to be just an arm/powerpc thing and I'm not convinced it's
really necessary.
> Signed-off-by: Manikanta Maddireddy <mmaddireddy@xxxxxxxxxx>
> ---
> V2:
> * commit message update
>
> drivers/pci/setup-bus.c | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/drivers/pci/setup-bus.c b/drivers/pci/setup-bus.c
> index b1ad466199ad..3567e1c4e340 100644
> --- a/drivers/pci/setup-bus.c
> +++ b/drivers/pci/setup-bus.c
> @@ -29,6 +29,7 @@
> #include "pci.h"
>
> unsigned int pci_flags;
> +EXPORT_SYMBOL(pci_flags);
>
> struct pci_dev_resource {
> struct list_head list;
> --
> 2.1.4
>
