[Qemu-devel] Re: [PATCH] ds1225y: Fix compiler errors in debug code
On Thu, Sep 30, 2010 at 7:30 PM, Stefan Weil w...@mail.berlios.de wrote: TARGET_FMT_lx is not allowed here, so use type casts to unsigned (which should be large enough to hold typical nvram addresses). The correct format is TARGET_FMT_plx.
[Qemu-devel] Re: [PATCH] ds1225y: Fix compiler errors in debug code
Am 30.09.2010 21:37, schrieb Blue Swirl: On Thu, Sep 30, 2010 at 7:30 PM, Stefan Weilw...@mail.berlios.de wrote: TARGET_FMT_lx is not allowed here, so use type casts to unsigned (which should be large enough to hold typical nvram addresses). The correct format is TARGET_FMT_plx. addr is typically less than 8192 (size of nvram). TARGET_FMT_plx displays addr using 16 hex characters when the target has 64 bit addresses. That's correct but does not look pretty. Do you prefer TARGET_FMT_plx nevertheless?
[Qemu-devel] Re: [PATCH] ds1225y: Fix compiler errors in debug code
On Thu, Sep 30, 2010 at 7:55 PM, Stefan Weil w...@mail.berlios.de wrote: Am 30.09.2010 21:37, schrieb Blue Swirl: On Thu, Sep 30, 2010 at 7:30 PM, Stefan Weilw...@mail.berlios.de wrote: TARGET_FMT_lx is not allowed here, so use type casts to unsigned (which should be large enough to hold typical nvram addresses). The correct format is TARGET_FMT_plx. addr is typically less than 8192 (size of nvram). TARGET_FMT_plx displays addr using 16 hex characters when the target has 64 bit addresses. That's correct but does not look pretty. Do you prefer TARGET_FMT_plx nevertheless? We could also introduce a new format which does not perform zero padding.
[Qemu-devel] Re: [PATCH] ds1225y: Fix compiler errors in debug code
Am 30.09.2010 22:09, schrieb Blue Swirl: On Thu, Sep 30, 2010 at 7:55 PM, Stefan Weil w...@mail.berlios.de wrote: Am 30.09.2010 21:37, schrieb Blue Swirl: On Thu, Sep 30, 2010 at 7:30 PM, Stefan Weilw...@mail.berlios.de wrote: TARGET_FMT_lx is not allowed here, so use type casts to unsigned (which should be large enough to hold typical nvram addresses). The correct format is TARGET_FMT_plx. addr is typically less than 8192 (size of nvram). TARGET_FMT_plx displays addr using 16 hex characters when the target has 64 bit addresses. That's correct but does not look pretty. Do you prefer TARGET_FMT_plx nevertheless? We could also introduce a new format which does not perform zero padding. Like PRIxTPA (TPA = Target Physical Address)? That would allow devices to print their addresses according to their needs with 1, 2, 4 or more characters. #if TARGET_PHYS_ADDR_BITS == 32 #define PRIxTPA PRIx32 #elif TARGET_PHYS_ADDR_BITS == 64 #define PRIxTPA PRIx64 #endif If that's ok for everybody, I could provide a patch for targphys.h and update the patch for ds1225y.c.