On Mon, 29 May 2017, Logan Gunthorpe wrote: > Move the CONFIG_PCI device so that ioremap and iounmap are always > available. This looks safe as there's nothing PCI specific in the > implementation of these functions. > > I have designs to use these functions in scatterlist.c where they'd likely > never be called without CONFIG_PCI set, but this is needed to compile > such changes. > > Signed-off-by: Logan Gunthorpe <logang@xxxxxxxxxxxx> > Cc: Martin Schwidefsky <schwidefsky@xxxxxxxxxx> > Cc: Heiko Carstens <heiko.carstens@xxxxxxxxxx> > cc: Sebastian Ott <sebott@xxxxxxxxxxxxxxxxxx> > Cc: Al Viro <viro@xxxxxxxxxxxxxxxxxx> > --- > > v2 of this patch is changed to simply move the CONFIG_PCI #ifdef > to expose only a single set of functions as suggested by Heiko > Carstens. > > arch/s390/include/asm/io.h | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/arch/s390/include/asm/io.h b/arch/s390/include/asm/io.h > index 437e9af96688..904e4b3af95d 100644 > --- a/arch/s390/include/asm/io.h > +++ b/arch/s390/include/asm/io.h > @@ -25,8 +25,6 @@ void unxlate_dev_mem_ptr(phys_addr_t phys, void *addr); > > #define IO_SPACE_LIMIT 0 > > -#ifdef CONFIG_PCI > - > #define ioremap_nocache(addr, size) ioremap(addr, size) > #define ioremap_wc ioremap_nocache > #define ioremap_wt ioremap_nocache > @@ -49,6 +47,8 @@ static inline void ioport_unmap(void __iomem *p) > { > } > > +#ifdef CONFIG_PCI > + > /* > * s390 needs a private implementation of pci_iomap since ioremap with its > * offset parameter isn't sufficient. That's because BAR spaces are not > -- > 2.11.0 > > Applied. The patch will go upstream via Martin's tree. Thanks! Sebastian -- To unsubscribe from this list: send the line "unsubscribe linux-s390" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html