diff options
| author | Edward O'Callaghan <eocallaghan@alterapraxis.com> | 2014-11-11 16:02:05 +1100 |
|---|---|---|
| committer | Edward O'Callaghan <eocallaghan@alterapraxis.com> | 2014-11-11 14:42:18 +0100 |
| commit | a955fefd146bedf1913dc81c5af66581bcb668a4 (patch) | |
| tree | c20034b0e9d7339837291baa120f8c2f65eac908 | |
| parent | e90080253b827f8dec7113ff80e723b9fa1b503c (diff) | |
| download | coreboot-a955fefd146bedf1913dc81c5af66581bcb668a4.tar.xz | |
drivers/intel/gma/intel_dp.c: Fix printf type-specifier
'%02hx' is unsigned short, where as the argument is typed as
uint8_t and so '%02hhx' is actually correct here.
Found-by: Clang
Change-Id: I40c48dcecf12845f4708e511236184908e90fb56
Signed-off-by: Edward O'Callaghan <eocallaghan@alterapraxis.com>
Reviewed-on: http://review.coreboot.org/7428
Reviewed-by: Paul Menzel <paulepanter@users.sourceforge.net>
Reviewed-by: Marc Jones <marc.jones@se-eng.com>
Tested-by: build bot (Jenkins)
| -rw-r--r-- | src/drivers/intel/gma/intel_dp.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/drivers/intel/gma/intel_dp.c b/src/drivers/intel/gma/intel_dp.c index 7aca28925d..dc2a957cdd 100644 --- a/src/drivers/intel/gma/intel_dp.c +++ b/src/drivers/intel/gma/intel_dp.c @@ -1533,7 +1533,7 @@ intel_dp_get_dpcd(struct intel_dp *intel_dp) return 0; } - printk(BIOS_SPEW, "DPCD: %02hx%02hx%02hx%02hx%02hx%02hx%02hx%02hx\n", + printk(BIOS_SPEW, "DPCD: %02hhx%02hhx%02hhx%02hhx%02hhx%02hhx%02hhx%02hhx\n", intel_dp->dpcd[0], intel_dp->dpcd[1], intel_dp->dpcd[2], intel_dp->dpcd[3], intel_dp->dpcd[4], intel_dp->dpcd[5], intel_dp->dpcd[6], intel_dp->dpcd[7]); |