On Thu, 17 Jan 2013 13:47:50 +0100
Holger Brunck <[email protected]> wrote:

> +int piggy_present(void)
> +{
> +     struct km_bec_fpga *base =
> +             (struct km_bec_fpga *)CONFIG_SYS_KMBEC_FPGA_BASE;
> +
> +     return in_8(&base->bprth) & PIGGY_PRESENT;
> +}

this change produces two new sparse warnings:

km83xx.c:137:22: warning: incorrect type in argument 1 (different address 
spaces)
km83xx.c:137:22:    expected unsigned char const volatile [noderef] <asn:2>*addr
km83xx.c:137:22:    got unsigned char *<noident>
km83xx.c:132:5: warning: symbol 'piggy_present' was not declared. Should it be 
static?

so make the function static, and add __iomem annotation to the
assignment of 'base'.

Kim

_______________________________________________
U-Boot mailing list
[email protected]
http://lists.denx.de/mailman/listinfo/u-boot

Reply via email to