[PATCH 01/14] ARM: LPC32XX: Initial architecture header files

Russell King - ARM Linux linux at arm.linux.org.uk
Tue Feb 9 04:59:34 EST 2010


On Tue, Feb 09, 2010 at 10:31:29AM +0100, Uwe Kleine-König wrote:
> > +#define io_p2v(x)	((void __iomem *) (unsigned long) IO_ADDRESS(x))
> Is this cast to unsigned long needed?  AFAIK IO_ADDRESS(x) has
> type unsigned for x in { 0x0 ... 0xffffffff } (provided that int uses a
> 32 bit 2s-complement representation).  If unsigned long is really
> needed, maybe put it into the IO_ADDRESS macro?

int -> void __iomem * = sparse warning
unsigned long -> void __iomem * = no sparse warning



More information about the linux-arm-kernel mailing list