Subrata Banik has submitted this change. ( https://review.coreboot.org/c/coreboot/+/83415?usp=email )
Change subject: soc/intel/cmn/cse: Correct CMOS error message for CSE partition firmware ......................................................................
soc/intel/cmn/cse: Correct CMOS error message for CSE partition firmware
The CMOS entry for CSE partition firmware was incorrectly labeled as `ramtop` and `partition firmware` in the error messages.
This patch corrects the messages to accurately refer to `CSE partition firmware`.
Additionally, the alignment and size check comments are updated to reflect this change.
Change-Id: Ib3a7fb88f52c4d0c47d828bcd1c4649e62d19654 Signed-off-by: Subrata Banik subratabanik@google.com Reviewed-on: https://review.coreboot.org/c/coreboot/+/83415 Tested-by: build bot (Jenkins) no-reply@coreboot.org Reviewed-by: Dinesh Gehlot digehlot@google.com --- M src/soc/intel/common/block/cse/cse_lite_cmos.c 1 file changed, 2 insertions(+), 2 deletions(-)
Approvals: build bot (Jenkins): Verified Dinesh Gehlot: Looks good to me, approved
diff --git a/src/soc/intel/common/block/cse/cse_lite_cmos.c b/src/soc/intel/common/block/cse/cse_lite_cmos.c index 5c26c01..bcd9ac0 100644 --- a/src/soc/intel/common/block/cse/cse_lite_cmos.c +++ b/src/soc/intel/common/block/cse/cse_lite_cmos.c @@ -16,11 +16,11 @@ # include "option_table.h"
#ifndef CMOS_VSTART_partition_fw -#error "The `ramtop` CMOS entry is missing, please add it to your cmos.layout." +#error "The `CSE partition firmware` CMOS entry is missing, please add it to your cmos.layout." #endif
#if CMOS_VSTART_partition_fw % 8 != 0 -#error "The `partition firmware` CMOS entry needs to be byte aligned, check your cmos.layout." +#error "The `CSE partition firmware` CMOS entry needs to be byte aligned, check your cmos.layout." #endif // CMOS_VSTART_partition_fw % 8 != 0
#if CMOS_VLEN_partition_fw != (32 * 8)