From 3cbf8d955f48fc39d5b3087934cbc5739ba07c5a Mon Sep 17 00:00:00 2001 From: Duncan Laurie Date: Thu, 28 Jan 2016 10:26:31 -0800 Subject: chromeos: Remove CONFIG_VBNV_SIZE variable The VBNV region size is determined by vboot and is not really configurable. Only the CMOS implementation defined this config variable so switch it to use VBNV_BLOCK_SIZE defined by vboot in vbnv_layout.h instead. This requires updating the broadwell/skylake cmos reset functions to use the right constant. BUG=chrome-os-partner:47915 BRANCH=glados TEST=manually tested on chell Change-Id: I45e3efc2a22efcb1470bbbefbdae4eda33fc6c96 Signed-off-by: Patrick Georgi Original-Commit-Id: e2b803ff3ac30ab22d65d1e62aca623730999a1d Original-Change-Id: I4896a1a5b7889d77ad00c4c8f285d184c4218e17 Original-Signed-off-by: Duncan Laurie Original-Reviewed-on: https://chromium-review.googlesource.com/324520 Original-Reviewed-by: Aaron Durbin Reviewed-on: https://review.coreboot.org/13598 Tested-by: build bot (Jenkins) Reviewed-by: Martin Roth --- src/lib/coreboot_table.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) (limited to 'src/lib') diff --git a/src/lib/coreboot_table.c b/src/lib/coreboot_table.c index 2226f5e1fd..86f22c9689 100644 --- a/src/lib/coreboot_table.c +++ b/src/lib/coreboot_table.c @@ -39,6 +39,7 @@ #endif #include #include +#include #endif #if CONFIG_ARCH_X86 #include @@ -191,7 +192,7 @@ static void lb_vbnv(struct lb_header *header) vbnv->tag = LB_TAG_VBNV; vbnv->size = sizeof(*vbnv); vbnv->range_start = CONFIG_VBNV_OFFSET + 14; - vbnv->range_size = CONFIG_VBNV_SIZE; + vbnv->range_size = VBNV_BLOCK_SIZE; #endif } -- cgit v1.2.3