On Fri, Jun 28, 2019 at 09:01:40AM +0200, Geert Uytterhoeven wrote: > This is due to include/asm-generic/io.h and include/asm-generic/iomap.h > using different prototypes, right? > > include/asm-generic/io.h:static inline u8 ioread8(const volatile void > __iomem *addr) > include/asm-generic/io.h:static inline u16 ioread16(const volatile > void __iomem *addr) > include/asm-generic/io.h:static inline u32 ioread32(const volatile > void __iomem *addr) > include/asm-generic/io.h:static inline u64 ioread64(const volatile > void __iomem *addr) > > include/asm-generic/iomap.h:extern unsigned int ioread8(void __iomem *); > include/asm-generic/iomap.h:extern unsigned int ioread16(void __iomem *); > include/asm-generic/iomap.h:extern unsigned int ioread32(void __iomem *); > include/asm-generic/iomap.h:extern u64 ioread64(void __iomem *); > > Wouldn't it be better to fix include/asm-generic/iomap.h and lib/iomap.c > instead?
Oh, I didn't even notice we had this declared by different files..
