[coreboot-gerrit] Patch set updated for coreboot: 45de2cb elog: Hide elog_flash_offset_to_address() from SMM

Patrick Georgi (pgeorgi@google.com) gerrit at coreboot.org
Fri Apr 10 10:21:44 CEST 2015


Patrick Georgi (pgeorgi at google.com) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/9450

-gerrit

commit 45de2cba61e506e5f9cb3cd27a30f3c5251b5f5f
Author: Furquan Shaikh <furquan at google.com>
Date:   Wed Nov 12 16:19:37 2014 -0800

    elog: Hide elog_flash_offset_to_address() from SMM
    
    Change-Id: Iaef9d4755f07ca03ca823831c3272183b5d6aed1
    Signed-off-by: Patrick Georgi <pgeorgi at chromium.org>
    Original-Commit-Id: 7c3db3d5c8e00b6c273ae240da137062597749aa
    Original-Change-Id: I5e38966fe06aa3302a7c1b536f5ffd8bb22d4947
    Original-Signed-off-by: Furquan Shaikh <furquan at google.com>
    Original-Reviewed-on: https://chromium-review.googlesource.com/229413
    Original-Reviewed-by: Aaron Durbin <adurbin at chromium.org>
    Original-Commit-Queue: Furquan Shaikh <furquan at chromium.org>
    Original-Tested-by: Furquan Shaikh <furquan at chromium.org>
---
 src/drivers/elog/elog.c | 33 ++++++++++++++++++---------------
 1 file changed, 18 insertions(+), 15 deletions(-)

diff --git a/src/drivers/elog/elog.c b/src/drivers/elog/elog.c
index c96a923..f6b0006 100644
--- a/src/drivers/elog/elog.c
+++ b/src/drivers/elog/elog.c
@@ -113,21 +113,6 @@ static inline u32 elog_flash_address_to_offset(u8 *address)
 }
 
 /*
- * Convert a flash offset into a memory mapped flash address
- */
-static inline u8* elog_flash_offset_to_address(u32 offset)
-{
-	u32 rom_size;
-
-	if (!elog_spi)
-		return NULL;
-
-	rom_size = get_rom_size();
-
-	return (u8*)((u32)~0UL - rom_size + 1 + offset);
-}
-
-/*
  * Pointer to an event log header in the event data area
  */
 static inline struct event_header*
@@ -452,6 +437,23 @@ static int elog_shrink(void)
 }
 
 #ifndef __SMM__
+#if IS_ENABLED(CONFIG_ARCH_X86)
+
+/*
+ * Convert a flash offset into a memory mapped flash address
+ */
+static inline u8 *elog_flash_offset_to_address(u32 offset)
+{
+	u32 rom_size;
+
+	if (!elog_spi)
+		return NULL;
+
+	rom_size = get_rom_size();
+
+	return (u8 *)((u32)~0UL - rom_size + 1 + offset);
+}
+
 /*
  * Fill out SMBIOS Type 15 table entry so the
  * event log can be discovered at runtime.
@@ -492,6 +494,7 @@ int elog_smbios_write_type15(unsigned long *current, int handle)
 	return len;
 }
 #endif
+#endif
 
 /*
  * Clear the entire event log



More information about the coreboot-gerrit mailing list