GNUBoot
/
coreboot-kgpe-d16
2
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

elog: handle ROM_SIZE differences from detected flash size 

The elog code calculates flash offsets and their equivalent
addresses in the memory address space. However, it assumes
the detected flash size is entirely mapped into the address
space. This can lead to incorrect calculations. Add code
to allow ROM_SIZE to be less than detected flash size. The
underlying assumption is that the first ROM_SIZE bytes are
programmed into the larger device.

Change-Id: Id848f136515289b40594b7d3762e26e3e55da62f
Signed-off-by: Aaron Durbin <adurbin@chromium.org>
Reviewed-on: https://gerrit.chromium.org/gerrit/60501
Reviewed-by: Duncan Laurie <dlaurie@chromium.org>
Reviewed-on: http://review.coreboot.org/4332
Tested-by: build bot (Jenkins)
Reviewed-by: Alexandru Gagniuc <mr.nuke.me@gmail.com>
This commit is contained in:
Aaron Durbin 2013-06-28 12:30:53 -07:00 committed by Alexandru Gagniuc
parent 8c0cb8ae3b
commit 10a070b447
1 changed files with 26 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

28
src/drivers/elog/elog.c
View File

@ -72,14 +72,33 @@ static u16 event_count;
static int elog_initialized; static int elog_initialized;
static struct spi_flash *elog_spi; static struct spi_flash *elog_spi;
static inline u32 get_rom_size(void)
{
u32 rom_size;
/* Assume the used space of the ROM image starts from 0. The
* physical size of the device may not be completely used. */
rom_size = elog_spi->size;
if (rom_size > CONFIG_ROM_SIZE)
rom_size = CONFIG_ROM_SIZE;
return rom_size;
}
/* /*
* Convert a memory mapped flash address into a flash offset * Convert a memory mapped flash address into a flash offset
*/ */
static inline u32 elog_flash_address_to_offset(u8 *address) static inline u32 elog_flash_address_to_offset(u8 *address)
{ {
u32 rom_size;
if (!elog_spi) if (!elog_spi)
return 0; return 0;
return (u32)address - ((u32)~0UL - elog_spi->size + 1);
rom_size = get_rom_size();
return (u32)address - ((u32)~0UL - rom_size + 1);
} }
/* /*
@ -87,9 +106,14 @@ static inline u32 elog_flash_address_to_offset(u8 *address)
*/ */
static inline u8* elog_flash_offset_to_address(u32 offset) static inline u8* elog_flash_offset_to_address(u32 offset)
{ {
u32 rom_size;
if (!elog_spi) if (!elog_spi)
return NULL; return NULL;
return (u8*)((u32)~0UL - elog_spi->size + 1 + offset);
rom_size = get_rom_size();
return (u8*)((u32)~0UL - rom_size + 1 + offset);
} }
/* /*