Re: [Qemu-devel] [PATCH v2 0/4] Define and use TARGET_PRI*PHYS format specifier macros for target_phys_addr_t

[Peter Maydell]

Ping?

-- PMM

On 25 June 2012 16:52, Peter Maydell <address@hidden> wrote:
> This patchset defines a set of macros for use in printf format strings
> for handling target_phys_addr_t values, as suggested by Andreas Färber
> (http://lists.gnu.org/archive/html/qemu-devel/2012-01/msg01547.html)
>
> We can then use them in some places which were previously either
> making assumptions about the width of target_phys_addr_t or had
> an ifdef to handle the 32 and 64 bit cases separately. Patches 2-4
> cover the format-string related parts of Anthony's original "make
> target_phys_addr_t 64 bits in all cases" patch:
> http://lists.gnu.org/archive/html/qemu-devel/2012-01/msg01523.html
>
> Changes v1->v2:
>  * rename macros to TARGET_PRI*PHYS, to be (a) a bit less opaque
>    about what they're for and (b) not in the namespace reserved
>    by POSIX
>
> Peter Maydell (4):
>   targphys.h: Define TARGET_PRI*PHYS format specifier macros
>   hw/omap.h: Use TARGET_PRIxPHYS to define OMAP_FMT_plx
>   hw/sh_serial: Use TARGET_PRIxPHYS rather than %x for physaddr
>   monitor: Use TARGET_PRI*PHYS to avoid TARGET_PHYS_ADDR_BITS ifdef
>
>  hw/omap.h      |    8 +-------
>  hw/sh_serial.c |    6 ++++--
>  monitor.c      |   29 ++++-------------------------
>  targphys.h     |   16 ++++++++++++++++
>  4 files changed, 25 insertions(+), 34 deletions(-)
>
>