diff options
| author | Michael Brown | 2008-11-19 04:45:44 +0100 |
|---|---|---|
| committer | Michael Brown | 2008-11-19 20:22:49 +0100 |
| commit | 3f85626fa95cb356a655d3538db6f05231f07003 (patch) | |
| tree | e460ab7fbb1525c43bdbab1018a70f57c8d252a7 /src/interface/efi/efi_timer.c | |
| parent | [i386] Move iSCSI and AoE boot code to arch/i386/interface/pcbios (diff) | |
| download | ipxe-3f85626fa95cb356a655d3538db6f05231f07003.tar.gz ipxe-3f85626fa95cb356a655d3538db6f05231f07003.tar.xz ipxe-3f85626fa95cb356a655d3538db6f05231f07003.zip | |
[efi] Add efi_strerror()
EFI_STATUS is defined as an INTN, which maps to UINT32 (i.e. unsigned
int) on i386 and UINT64 (i.e. unsigned long) on x86_64. This would
require a cast each time the error status is printed.
Add efi_strerror() to avoid this ickiness and simultaneously enable
prettier reporting of EFI status codes.
Diffstat (limited to 'src/interface/efi/efi_timer.c')
| -rw-r--r-- | src/interface/efi/efi_timer.c | 7 |
1 files changed, 4 insertions, 3 deletions
diff --git a/src/interface/efi/efi_timer.c b/src/interface/efi/efi_timer.c index 66211770..d1ba43af 100644 --- a/src/interface/efi/efi_timer.c +++ b/src/interface/efi/efi_timer.c @@ -53,8 +53,8 @@ static void efi_udelay ( unsigned long usecs ) { EFI_STATUS efirc; if ( ( efirc = bs->Stall ( usecs ) ) != 0 ) { - DBG ( "EFI could not delay for %ldus: %x\n", - usecs, efirc ); + DBG ( "EFI could not delay for %ldus: %s\n", + usecs, efi_strerror ( efirc ) ); /* Probably screwed */ } } @@ -71,7 +71,8 @@ static unsigned long efi_currticks ( void ) { /* Read CPU timer 0 (TSC) */ if ( ( efirc = cpu_arch->GetTimerValue ( cpu_arch, 0, &time, NULL ) ) != 0 ) { - DBG ( "EFI could not read CPU timer: %x\n", efirc ); + DBG ( "EFI could not read CPU timer: %s\n", + efi_strerror ( efirc ) ); /* Probably screwed */ return -1UL; } |