diff options
| author | Kyösti Mälkki <kyosti.malkki@gmail.com> | 2014-06-30 13:48:18 +0300 |
|---|---|---|
| committer | Kyösti Mälkki <kyosti.malkki@gmail.com> | 2014-06-30 17:55:36 +0200 |
| commit | 38a8fb0c1891bbe3b53965e04c1c7ebcc00654da (patch) | |
| tree | 877bd916925d48a10a2561ed8728166ed98f0326 | |
| parent | 599cda82283294944883cb17e8c2a43d96f9d2e9 (diff) | |
x86 MTRR: Drop unused return value
It was never well-defined what value this function should return.
Change-Id: If84aff86e0b556591d7ad557842910a2dfcd3b46
Signed-off-by: Kyösti Mälkki <kyosti.malkki@gmail.com>
Reviewed-on: http://review.coreboot.org/6166
Tested-by: build bot (Jenkins)
Reviewed-by: Patrick Georgi <patrick@georgi-clan.de>
Reviewed-by: Edward O'Callaghan <eocallaghan@alterapraxis.com>
| -rw-r--r-- | src/cpu/x86/mtrr/mtrr.c | 3 | ||||
| -rw-r--r-- | src/include/cpu/x86/mtrr.h | 2 |
2 files changed, 2 insertions, 3 deletions
diff --git a/src/cpu/x86/mtrr/mtrr.c b/src/cpu/x86/mtrr/mtrr.c index 8826535528..b19b853196 100644 --- a/src/cpu/x86/mtrr/mtrr.c +++ b/src/cpu/x86/mtrr/mtrr.c @@ -772,7 +772,7 @@ void x86_setup_mtrrs(void) x86_setup_var_mtrrs(address_size, 1); } -int x86_mtrr_check(void) +void x86_mtrr_check(void) { /* Only Pentium Pro and later have MTRR */ msr_t msr; @@ -795,5 +795,4 @@ int x86_mtrr_check(void) printk(BIOS_DEBUG, "\n"); post_code(0x93); - return ((int) msr.lo); } diff --git a/src/include/cpu/x86/mtrr.h b/src/include/cpu/x86/mtrr.h index 2a440723b8..95cabf1247 100644 --- a/src/include/cpu/x86/mtrr.h +++ b/src/include/cpu/x86/mtrr.h @@ -66,7 +66,7 @@ void enable_fixed_mtrr(void); void x86_setup_fixed_mtrrs(void); /* Set up fixed MTRRs but do not enable them. */ void x86_setup_fixed_mtrrs_no_enable(void); -int x86_mtrr_check(void); +void x86_mtrr_check(void); #endif #if !defined(__ASSEMBLER__) && defined(__PRE_RAM__) && !defined(__ROMCC__) |