From 4265d5265dfc7bd823bcf5c8c5dded8d06189f69 Mon Sep 17 00:00:00 2001 From: Sean Rhodes Date: Tue, 2 May 2023 20:54:40 +0100 Subject: soc/intel/common: Correct the check for ramtop length The `ramtop_table` is 10 bytes long, so adjust the check to account for this. Also, adjust the wording to make it clear what is required to fix it, should the error be shown. Signed-off-by: Sean Rhodes Change-Id: If2898c4bb22abb1779035aadc08f32898e9a096b Reviewed-on: https://review.coreboot.org/c/coreboot/+/74919 Tested-by: build bot (Jenkins) Reviewed-by: Lean Sheng Tan Reviewed-by: Subrata Banik --- src/soc/intel/common/basecode/ramtop/ramtop.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) (limited to 'src/soc') diff --git a/src/soc/intel/common/basecode/ramtop/ramtop.c b/src/soc/intel/common/basecode/ramtop/ramtop.c index bf85fa8efc..f6aa4eadd4 100644 --- a/src/soc/intel/common/basecode/ramtop/ramtop.c +++ b/src/soc/intel/common/basecode/ramtop/ramtop.c @@ -23,9 +23,9 @@ #error "The `ramtop` CMOS entry needs to be byte aligned, check your cmos.layout." #endif // CMOS_VSTART_ramtop % 8 != 0 -#if CMOS_VLEN_ramtop != 12 -#error "CMOS length for RAMTOP_CMOS bytes are not correct, check your cmos.layout" -#endif +#if CMOS_VLEN_ramtop != (10 * 8) +#error "The `ramtop` CMOS entry needs to be 10 bytes long, check your cmos.layout." +#endif // CMOS_VLEN_ramtop != (10 * 8) #else #define CMOS_VSTART_ramtop 800 -- cgit v1.2.3