On Wed, 11 Jul 2007, Atsushi Nemoto wrote:
> CKSEG1ADDR() returns unsigned int value on 32bit kernel. Cast it to
> unsigned long to get rid of this warning:
>
> include2/asm/io.h:215:12: warning: cast adds address space to expression
> (<asn:2>)
>
> Signed-off-by: Atsushi Nemoto <anemo@mba.ocn.ne.jp>
> ---
> diff --git a/include/asm-mips/io.h b/include/asm-mips/io.h
> index 12bcc1f..7ba9289 100644
> --- a/include/asm-mips/io.h
> +++ b/include/asm-mips/io.h
> @@ -212,7 +212,8 @@ static inline void __iomem * __ioremap_mode(phys_t
> offset, unsigned long size,
> */
> if (__IS_LOW512(phys_addr) && __IS_LOW512(last_addr) &&
> flags == _CACHE_UNCACHED)
> - return (void __iomem *)CKSEG1ADDR(phys_addr);
> + return (void __iomem *)
> + (unsigned long)CKSEG1ADDR(phys_addr);
> }
It looks like a bug in sparse. The result of CKSEG1ADDR() has the same
size as the pointer. Perhaps we could append 'L' to the expansion of
KSEG1 et al, but that should not really matter.
But -- I have just checked two example calls to this function, one with a
32-bit configuration and another one with a 64-bit one and sparse did not
complain. The cpp expansions of the expression in question are:
return (void *)((((int)(int)(phys_addr)) & 0x1fffffff) | 0xa0000000);
and:
return (void *)((((long int)(int)(phys_addr)) & 0x1fffffff) |
0xffffffffa0000000L);
respectively, so your cast is definitely redundant in these cases. What
sort of configuration are you using? What's the preprocessor output for
the problematic case?
Maciej
|