Thread (90 messages) 90 messages, 11 authors, 2021-03-02

Re: [PATCH v2 10/25] asm-generic/io.h: Add a non-posted variant of ioremap()

From: Hector Martin <hidden>
Date: 2021-02-15 16:48:35
Subsystem: sparc + ultrasparc (sparc/sparc64), the rest · Maintainers: "David S. Miller", Andreas Larsson, Linus Torvalds

On 16/02/2021 00.27, kernel test robot wrote:
config: mips-randconfig-r002-20210215 (attached as .config)
compiler: mips64-linux-gcc (GCC) 9.3.0
quoted
quoted
lib/devres.c:47:10: error: implicit declaration of function 'ioremap_np'; did you mean 'ioremap_uc'? [-Werror=implicit-function-declaration]
Well, today I learned that some architectures, like mips, only include
asm-generic/iomap.h, not asm-generic/io.h... and one, sparc64, includes
neither. That's unfortunate.

I added the missing bit to iomap.h, and a one-off patch to
arch/sparc/include/asm/io_64.h:
diff --git a/arch/sparc/include/asm/io_64.h b/arch/sparc/include/asm/io_64.h
index 9bb27e5c22f1..35470d275ddc 100644
--- a/arch/sparc/include/asm/io_64.h
+++ b/arch/sparc/include/asm/io_64.h
@@ -409,6 +409,7 @@ static inline void __iomem *ioremap(unsigned long offset, unsigned long size)
  #define ioremap_uc(X,Y)			ioremap((X),(Y))
  #define ioremap_wc(X,Y)			ioremap((X),(Y))
  #define ioremap_wt(X,Y)			ioremap((X),(Y))
+#define ioremap_np(X,Y)			ioremap((X),(Y))
  
  static inline void iounmap(volatile void __iomem *addr)
  {
David, is this okay? This series is headed for the soc tree, but I
wasn't expecting to have to touch arch-specific code for anything but
arm... you can see the rest of it here:

https://lore.kernel.org/linux-arm-kernel/YCqdi%2F5TSlbt0w%2F2@kroah.com/T/ (local)

I'll CC you on v3 if this works for you.

-- 
Hector Martin (marcan@marcan.st)
Public Key: https://mrcn.st/pub

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help