Re: [PATCH 2/2] powerpc: Fix build without CONFIG_PCI

On Fri, 2011-08-05 at 17:37 +1000, Stephen Rothwell wrote:

Hi Ben,

On Fri, 05 Aug 2011 16:04:07 +1000 Benjamin Herrenschmidt [off-list ref] wrote:

quoted

Commit fea80311a939a746533a6d7e7c3183729d6a3faf
"iomap: make IOPORT/PCI mapping functions conditional"

Broke powerpc build without CONFIG_PCI as we would still define
pci_iomap(), which overlaps with the new empty inline in the headers.

If we are using a static inline implementation for !CONFIG_PCI, then we
should probably not EXPORT it in that case either ...

Good point, interesting that my build test didn't catch it ... I'll
add another patch on top before I ask Linus to pull.

Cheers,
Ben.

quoted

@@ -143,6 +144,7 @@ void pci_iounmap(struct pci_dev *dev, void __iomem *addr)
 		return;
 	iounmap(addr);
 }
+#endif /* CONFIG_PCI */
 
 EXPORT_SYMBOL(pci_iomap);
 EXPORT_SYMBOL(pci_iounmap);