[OpenBIOS] [commit] r949 - in trunk/openbios-devel/include/arch: amd64 ia64 ppc sparc32 sparc64 x86

[Andreas Färber]

Am 08.11.2010 um 22:17 schrieb Blue Swirl:

> On Mon, Nov 8, 2010 at 9:12 PM, repository service  
> <svn at 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