diff options
| author | Ricardo Quesada <ricardoq@google.com> | 2021-08-16 14:48:36 -0700 |
|---|---|---|
| committer | Felix Held <felix-coreboot@felixheld.de> | 2021-08-26 18:50:56 +0000 |
| commit | 2e0b5c40073a38080e7ee5ebc7e67cbf5d1f475a (patch) | |
| tree | 8c3ab23a41ef0408e986c7c92b0b1f2f8f958c6a | |
| parent | e929a75fbe51eb7bcc48d18f6bfbda11b39a8373 (diff) | |
elog: Define constant for RW region name
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>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/56986
Reviewed-by: Furquan Shaikh <furquan@google.com>
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
| -rw-r--r-- | src/commonlib/bsd/include/commonlib/bsd/elog.h | 1 | ||||
| -rw-r--r-- | src/drivers/elog/elog.c | 2 |
2 files changed, 2 insertions, 1 deletions
diff --git a/src/commonlib/bsd/include/commonlib/bsd/elog.h b/src/commonlib/bsd/include/commonlib/bsd/elog.h index 2ac15f0a4f..d428b5d4d6 100644 --- a/src/commonlib/bsd/include/commonlib/bsd/elog.h +++ b/src/commonlib/bsd/include/commonlib/bsd/elog.h @@ -18,6 +18,7 @@ struct elog_header { /* 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 4b51504d6a..69567c0f4d 100644 --- a/src/drivers/elog/elog.c +++ b/src/drivers/elog/elog.c @@ -619,7 +619,7 @@ static int elog_find_flash(void) 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; } |