18 Jan
2013
18 Jan
'13
9:07 a.m.
On 01/18/2013 01:23 AM, Kim Phillips wrote:
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'.
ok will do. Just out of curiosity which compiler throws these warnings? I use powerpc-gcc (GCC) 4.7.2 and don't see them.
Anyway I send a v2 for this patch.
Regards Holger