
18 Jan
2013
18 Jan
'13
1:23 a.m.
On Thu, 17 Jan 2013 13:47:50 +0100 Holger Brunck holger.brunck@keymile.com 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