Hi Ben, On Fri, 05 Aug 2011 16:04:07 +1000 Benjamin Herrenschmidt wrote: > > 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 ... > @@ -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); -- Stephen Rothwell