[PATCH 01/24] ARM: shmobile: use __iomem pointers for MMIO

Arnd Bergmann arnd at arndb.de
Tue Sep 18 04:31:06 EDT 2012


On Tuesday 18 September 2012, Simon Horman wrote:
> On Fri, Sep 14, 2012 at 11:34:29PM +0200, Arnd Bergmann wrote:
> > ARM is moving to stricter checks on readl/write functions,
> > so we need to use the correct types everywhere.
> > 
> > This patch is a bit ugly for shmobile, which is the only platform
> > that just uses integer literals all over the place, but I can't
> > see a better way to do this.
> > 
> > Cc: Magnus Damm <magnus.damm at gmail.com>
> > Cc: Kuninori Morimoto <kuninori.morimoto.gx at renesas.com>
> > Cc: Simon Horman <horms at verge.net.au>
> > Cc: Paul Mundt <lethal at linux-sh.org>
> > Cc: linux-sh at vger.kernel.org
> > Signed-off-by: Arnd Bergmann <arnd at arndb.de>
> 
> Wow, thats a lot of chrun, but if we must
> 
> Acked-by: Simon Horman <horms at verge.net.au>
> 

I agree it's not nice to have to do this, but this is largely because
of shmobile doing things differently from all other ARM platforms, on
which the respective patches tend to clean up things and reduce the
number of type casts.

The only alternative I can see is for shmobile to introduce its own
mach/io.h file with the relaxed type checking, but that would only
defer the problem until the point where you want shmobile to be part
of the common multiplatform kernel binary.

	Arnd




More information about the linux-arm-kernel mailing list