diff options
| author | Michael Brown | 2015-08-27 11:08:00 +0200 |
|---|---|---|
| committer | Michael Brown | 2015-08-27 16:40:44 +0200 |
| commit | 9501eaf68d79a7baa2d71567481ccb0f43e56253 (patch) | |
| tree | 5cf2705ee089b65dfc93b9a4f5f5c3bf3a10cf65 /src/drivers/net/efi/snp.c | |
| parent | [efi] Mark EFI debug transcription functions as __attribute__ (( pure )) (diff) | |
| download | ipxe-9501eaf68d79a7baa2d71567481ccb0f43e56253.tar.gz ipxe-9501eaf68d79a7baa2d71567481ccb0f43e56253.tar.xz ipxe-9501eaf68d79a7baa2d71567481ccb0f43e56253.zip | |
[efi] Remove raw EFI_HANDLE values from debug messages
The raw EFI_HANDLE value is almost never useful to know, and simply
adds noise to the already verbose debug messages. Improve the
legibility of debug messages by using only the name generated by
efi_handle_name().
Signed-off-by: Michael Brown <mcb30@ipxe.org>
Diffstat (limited to 'src/drivers/net/efi/snp.c')
| -rw-r--r-- | src/drivers/net/efi/snp.c | 24 |
1 files changed, 12 insertions, 12 deletions
diff --git a/src/drivers/net/efi/snp.c b/src/drivers/net/efi/snp.c index acfcfba9..fbd60690 100644 --- a/src/drivers/net/efi/snp.c +++ b/src/drivers/net/efi/snp.c @@ -48,8 +48,8 @@ static int snp_supported ( EFI_HANDLE device ) { /* Check that this is not a device we are providing ourselves */ if ( find_snpdev ( device ) != NULL ) { - DBGCP ( device, "SNP %p %s is provided by this binary\n", - device, efi_handle_name ( device ) ); + DBGCP ( device, "SNP %s is provided by this binary\n", + efi_handle_name ( device ) ); return -ENOTTY; } @@ -58,12 +58,12 @@ static int snp_supported ( EFI_HANDLE device ) { &efi_simple_network_protocol_guid, NULL, efi_image_handle, device, EFI_OPEN_PROTOCOL_TEST_PROTOCOL))!=0){ - DBGCP ( device, "SNP %p %s is not an SNP device\n", - device, efi_handle_name ( device ) ); + DBGCP ( device, "SNP %s is not an SNP device\n", + efi_handle_name ( device ) ); return -EEFI ( efirc ); } - DBGC ( device, "SNP %p %s is an SNP device\n", - device, efi_handle_name ( device ) ); + DBGC ( device, "SNP %s is an SNP device\n", + efi_handle_name ( device ) ); return 0; } @@ -80,8 +80,8 @@ static int nii_supported ( EFI_HANDLE device ) { /* Check that this is not a device we are providing ourselves */ if ( find_snpdev ( device ) != NULL ) { - DBGCP ( device, "NII %p %s is provided by this binary\n", - device, efi_handle_name ( device ) ); + DBGCP ( device, "NII %s is provided by this binary\n", + efi_handle_name ( device ) ); return -ENOTTY; } @@ -90,12 +90,12 @@ static int nii_supported ( EFI_HANDLE device ) { &efi_nii31_protocol_guid, NULL, efi_image_handle, device, EFI_OPEN_PROTOCOL_TEST_PROTOCOL))!=0){ - DBGCP ( device, "NII %p %s is not an NII device\n", - device, efi_handle_name ( device ) ); + DBGCP ( device, "NII %s is not an NII device\n", + efi_handle_name ( device ) ); return -EEFI ( efirc ); } - DBGC ( device, "NII %p %s is an NII device\n", - device, efi_handle_name ( device ) ); + DBGC ( device, "NII %s is an NII device\n", + efi_handle_name ( device ) ); return 0; } |