From 7f4221c1787eb52874a99ed736f0fd692c71e335 Mon Sep 17 00:00:00 2001 From: Furquan Shaikh Date: Wed, 12 Nov 2014 16:19:37 -0800 Subject: [PATCH] elog: Hide elog_flash_offset_to_address() from SMM Change-Id: Iaef9d4755f07ca03ca823831c3272183b5d6aed1 Signed-off-by: Patrick Georgi Original-Commit-Id: 7c3db3d5c8e00b6c273ae240da137062597749aa Original-Change-Id: I5e38966fe06aa3302a7c1b536f5ffd8bb22d4947 Original-Signed-off-by: Furquan Shaikh Original-Reviewed-on: https://chromium-review.googlesource.com/229413 Original-Reviewed-by: Aaron Durbin Original-Commit-Queue: Furquan Shaikh Original-Tested-by: Furquan Shaikh Reviewed-on: http://review.coreboot.org/9450 Tested-by: build bot (Jenkins) Reviewed-by: Stefan Reinauer --- 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 c96a923b93..f6b0006db8 100644 --- a/src/drivers/elog/elog.c +++ b/src/drivers/elog/elog.c @@ -112,21 +112,6 @@ static inline u32 elog_flash_address_to_offset(u8 *address) #endif } -/* - * 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 */ @@ -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