summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorKyösti Mälkki <kyosti.malkki@gmail.com>2014-06-30 13:48:18 +0300
committerKyösti Mälkki <kyosti.malkki@gmail.com>2014-06-30 17:55:36 +0200
commit38a8fb0c1891bbe3b53965e04c1c7ebcc00654da (patch)
tree877bd916925d48a10a2561ed8728166ed98f0326
parent599cda82283294944883cb17e8c2a43d96f9d2e9 (diff)
downloadcoreboot-38a8fb0c1891bbe3b53965e04c1c7ebcc00654da.tar.gz
coreboot-38a8fb0c1891bbe3b53965e04c1c7ebcc00654da.tar.bz2
coreboot-38a8fb0c1891bbe3b53965e04c1c7ebcc00654da.zip
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.c3
-rw-r--r--src/include/cpu/x86/mtrr.h2
2 files changed, 2 insertions, 3 deletions
diff --git a/src/cpu/x86/mtrr/mtrr.c b/src/cpu/x86/mtrr/mtrr.c
index 8826535528e5..b19b8531961b 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 2a440723b854..95cabf124705 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__)