summaryrefslogtreecommitdiff
path: root/src/soc/intel/skylake
diff options
context:
space:
mode:
authorAaron Durbin <adurbin@chromium.org>2017-07-14 15:42:35 -0600
committerAaron Durbin <adurbin@chromium.org>2017-07-17 14:59:56 +0000
commit2c628f1652fb0ca28a1fc6c43225475362b50fc7 (patch)
treee4750f006c876f749f9ac20ee6769b91a42d1467 /src/soc/intel/skylake
parent3c874e9d04b7a8ae2af5fef39abb925766e85e9c (diff)
soc/intel/skylake: remove top_of_32bit_ram() declaration
It should never be globally exposed. Remove the global symbol and make it static. Change-Id: I3b85f3bbf6a73d480cdefdcdec26e137e3a3f75f Signed-off-by: Aaron Durbin <adurbin@chromium.org> Reviewed-on: https://review.coreboot.org/20584 Tested-by: build bot (Jenkins) <no-reply@coreboot.org> Reviewed-by: Furquan Shaikh <furquan@google.com> Reviewed-by: Subrata Banik <subrata.banik@intel.com>
Diffstat (limited to 'src/soc/intel/skylake')
-rw-r--r--src/soc/intel/skylake/include/soc/systemagent.h3
-rw-r--r--src/soc/intel/skylake/memmap.c2
2 files changed, 1 insertions, 4 deletions
diff --git a/src/soc/intel/skylake/include/soc/systemagent.h b/src/soc/intel/skylake/include/soc/systemagent.h
index 96e6c6a81b..d8192a3e75 100644
--- a/src/soc/intel/skylake/include/soc/systemagent.h
+++ b/src/soc/intel/skylake/include/soc/systemagent.h
@@ -42,7 +42,4 @@
#define MCH_DDR_POWER_LIMIT_LO 0x58e0
#define MCH_DDR_POWER_LIMIT_HI 0x58e4
-/* Top of 32bit usable memory */
-u32 top_of_32bit_ram(void);
-
#endif
diff --git a/src/soc/intel/skylake/memmap.c b/src/soc/intel/skylake/memmap.c
index 17dbc320c6..43faabb011 100644
--- a/src/soc/intel/skylake/memmap.c
+++ b/src/soc/intel/skylake/memmap.c
@@ -137,7 +137,7 @@ int smm_subregion(int sub, void **start, size_t *size)
* other words, the memory map is in a fixed order no matter what.
*/
-u32 top_of_32bit_ram(void)
+static u32 top_of_32bit_ram(void)
{
msr_t prmrr_base;
u32 top_of_ram;