On 04/14/2013 09:14 PM, liguang wrote: > #ifdef DEBUG_DEBUGCON > - printf("debugcon: write addr=0x%04" HWADDR_PRIx " val=0x%02x\n", addr, > val); > + printf(" [debugcon: write addr=0x%04" HWADDR_PRIx " val=0x%02]\n", addr, > val);
I like the reformatting, but it looks like you accidentally clobbered the 'x' when you added the ']'. This is the cause of the compiler warnings in patch #3. Jesse Larrew Software Engineer, KVM Team IBM Linux Technology Center Phone: (512) 973-2052 (T/L: 363-2052) jlar...@linux.vnet.ibm.com