diff mbox

[TRIVIAL] PCI: Change pci_host_common_probe visibility

Message ID 11bd9bd5-5d96-bd7c-24c7-5f262a46085c@sigmadesigns.com (mailing list archive)
State New, archived
Delegated to: Bjorn Helgaas
Headers show

Commit Message

Marc Gonzalez March 28, 2017, 3:29 p.m. UTC
pci_host_common_probe is defined when CONFIG_PCI_HOST_COMMON=y
therefore the function declaration should match that.

drivers/pci/host/pcie-tango.c:300:9: error:
	implicit declaration of function 'pci_host_common_probe'

Signed-off-by: Marc Gonzalez <marc_gonzalez@sigmadesigns.com>
---
 include/linux/pci-ecam.h | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Marc Gonzalez April 10, 2017, 12:26 p.m. UTC | #1
On 28/03/2017 17:29, Marc Gonzalez wrote:

> pci_host_common_probe is defined when CONFIG_PCI_HOST_COMMON=y
> therefore the function declaration should match that.
> 
> drivers/pci/host/pcie-tango.c:300:9: error:
> 	implicit declaration of function 'pci_host_common_probe'
> 
> Signed-off-by: Marc Gonzalez <marc_gonzalez@sigmadesigns.com>
> ---
>  include/linux/pci-ecam.h | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/include/linux/pci-ecam.h b/include/linux/pci-ecam.h
> index 33c4750781a0..e746803df566 100644
> --- a/include/linux/pci-ecam.h
> +++ b/include/linux/pci-ecam.h
> @@ -60,7 +60,7 @@ void __iomem *pci_ecam_map_bus(struct pci_bus *bus, unsigned int devfn,
>  /* default ECAM ops */
>  extern struct pci_ecam_ops pci_generic_ecam_ops;
>  
> -#ifdef CONFIG_PCI_HOST_GENERIC
> +#ifdef CONFIG_PCI_HOST_COMMON
>  /* for DT-based PCI controllers that support ECAM */
>  int pci_host_common_probe(struct platform_device *pdev,
>  			  struct pci_ecam_ops *ops);
> 

Hello Bjorn,

What is the status of this patch?

Arnd submitted an alternative patch earlier.

https://patchwork.kernel.org/patch/9637045/
https://patchwork.kernel.org/patch/9649993/

Arnd's solution is to make the declaration always visible.
My solution is to make the declaration visible only when
CONFIG_PCI_HOST_COMMON is defined.

The difference boils down to getting an error
at link-time VS at compile-time.

Regards.
Bjorn Helgaas April 18, 2017, 7:26 p.m. UTC | #2
On Mon, Apr 10, 2017 at 02:26:42PM +0200, Marc Gonzalez wrote:
> On 28/03/2017 17:29, Marc Gonzalez wrote:
> 
> > pci_host_common_probe is defined when CONFIG_PCI_HOST_COMMON=y
> > therefore the function declaration should match that.
> > 
> > drivers/pci/host/pcie-tango.c:300:9: error:
> > 	implicit declaration of function 'pci_host_common_probe'
> > 
> > Signed-off-by: Marc Gonzalez <marc_gonzalez@sigmadesigns.com>
> > ---
> >  include/linux/pci-ecam.h | 2 +-
> >  1 file changed, 1 insertion(+), 1 deletion(-)
> > 
> > diff --git a/include/linux/pci-ecam.h b/include/linux/pci-ecam.h
> > index 33c4750781a0..e746803df566 100644
> > --- a/include/linux/pci-ecam.h
> > +++ b/include/linux/pci-ecam.h
> > @@ -60,7 +60,7 @@ void __iomem *pci_ecam_map_bus(struct pci_bus *bus, unsigned int devfn,
> >  /* default ECAM ops */
> >  extern struct pci_ecam_ops pci_generic_ecam_ops;
> >  
> > -#ifdef CONFIG_PCI_HOST_GENERIC
> > +#ifdef CONFIG_PCI_HOST_COMMON
> >  /* for DT-based PCI controllers that support ECAM */
> >  int pci_host_common_probe(struct platform_device *pdev,
> >  			  struct pci_ecam_ops *ops);
> > 
> 
> Hello Bjorn,
> 
> What is the status of this patch?
> 
> Arnd submitted an alternative patch earlier.
> 
> https://patchwork.kernel.org/patch/9637045/
> https://patchwork.kernel.org/patch/9649993/
> 
> Arnd's solution is to make the declaration always visible.
> My solution is to make the declaration visible only when
> CONFIG_PCI_HOST_COMMON is defined.
> 
> The difference boils down to getting an error
> at link-time VS at compile-time.

I applied this to pci/misc for v4.12, thanks!

Arnd's patch included:

  Fixes: 4e64dbe226e7 ("PCI: generic: Expose pci_host_common_probe() for use by other drivers")

which didn't look quite right to me.  I was originally hoping to merge
it for v4.11, but I didn't think I had quite the right justification
for that.

In any case, I like the idea of making the declaration visible in
exactly the same case the definition is visible.

Bjorn
diff mbox

Patch

diff --git a/include/linux/pci-ecam.h b/include/linux/pci-ecam.h
index 33c4750781a0..e746803df566 100644
--- a/include/linux/pci-ecam.h
+++ b/include/linux/pci-ecam.h
@@ -60,7 +60,7 @@  void __iomem *pci_ecam_map_bus(struct pci_bus *bus, unsigned int devfn,
 /* default ECAM ops */
 extern struct pci_ecam_ops pci_generic_ecam_ops;
 
-#ifdef CONFIG_PCI_HOST_GENERIC
+#ifdef CONFIG_PCI_HOST_COMMON
 /* for DT-based PCI controllers that support ECAM */
 int pci_host_common_probe(struct platform_device *pdev,
 			  struct pci_ecam_ops *ops);