Am 08.11.2010 um 22:17 schrieb Blue Swirl:
On Mon, Nov 8, 2010 at 9:12 PM, repository service svn@openbios.org wrote:
Author: afaerber Date: Mon Nov 8 22:12:36 2010 New Revision: 949 URL: http://tracker.coreboot.org/trac/openbios/changeset/949
Log: Introduce FMT_plx for phys_addr_t
Define a zero-padded format string to be used for phys_addr_t arguments. Introduce PRIx{32,64} macros as needed.
v2:
- Also define PRIx32 for sparc32 and sparc64, requested by Blue.
What I meant was that all PRIx* macros should be defined for all architectures for consistency.
So in particular you mean PRIx16 and PRIx8? ppc, sparc32 and sparc64 are in fact consistent now in defining both PRIx32 and PRIx64.
x86, amd64 and ia64 use the system headers, so we shouldn't define any such macro there, just like we don't do in the bootstrap code path.
Andreas