Ricardo Quesada has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/56986 )
Change subject: elog: define RW_ELOG constant ......................................................................
elog: define RW_ELOG constant
This CL indroduces the ELOG_RW_REGION_NAME. This constant replaced the hardcoded "RW_ELOG" value. This constant will be used also by elogtool (see CL in the commit chain).
BUG=b:172210863
Change-Id: Ie8d31204e65fd67d52b0f8ced7b8c1ffdcf5b539 Signed-off-by: Ricardo Quesada ricardoq@google.com --- M src/commonlib/bsd/include/commonlib/bsd/elog.h M src/drivers/elog/elog.c 2 files changed, 2 insertions(+), 1 deletion(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/86/56986/1
diff --git a/src/commonlib/bsd/include/commonlib/bsd/elog.h b/src/commonlib/bsd/include/commonlib/bsd/elog.h index 2ac15f0..d428b5d 100644 --- a/src/commonlib/bsd/include/commonlib/bsd/elog.h +++ b/src/commonlib/bsd/include/commonlib/bsd/elog.h @@ -18,6 +18,7 @@ /* ELOG related constants */ #define ELOG_SIGNATURE 0x474f4c45 /* 'ELOG' */ #define ELOG_VERSION 1 +#define ELOG_RW_REGION_NAME "RW_ELOG"
/* SMBIOS event log header */ struct event_header { diff --git a/src/drivers/elog/elog.c b/src/drivers/elog/elog.c index 4b51504..69567c0 100644 --- a/src/drivers/elog/elog.c +++ b/src/drivers/elog/elog.c @@ -619,7 +619,7 @@ elog_debug("%s()\n", __func__);
/* Find the ELOG base and size in FMAP */ - if (fmap_locate_area_as_rdev_rw("RW_ELOG", rdev) < 0) { + if (fmap_locate_area_as_rdev_rw(ELOG_RW_REGION_NAME, rdev) < 0) { printk(BIOS_WARNING, "ELOG: Unable to find RW_ELOG in FMAP\n"); return -1; }