Patrick Georgi has submitted this change. ( https://review.coreboot.org/c/coreboot/+/39024 )
Change subject: vboot: remove rogue vboot_struct.h include ......................................................................
vboot: remove rogue vboot_struct.h include
As part of vboot1 deprecation, remove an unused vboot_struct.h include. coreboot is now free of vboot1 data structure use. One vboot_api.h include remains as part of security/vboot/ec_sync.c.
BUG=b:124141368 TEST=make clean && make test-abuild BRANCH=none
Change-Id: I042d692aa252f8f859d4005455eb6a2eabc24a87 Signed-off-by: Joel Kitching kitching@google.com Reviewed-on: https://review.coreboot.org/c/coreboot/+/39024 Tested-by: build bot (Jenkins) no-reply@coreboot.org Reviewed-by: Aaron Durbin adurbin@chromium.org Reviewed-by: Julius Werner jwerner@chromium.org --- M src/security/vboot/ec_sync.c M src/vendorcode/google/chromeos/gnvs.c 2 files changed, 1 insertion(+), 2 deletions(-)
Approvals: build bot (Jenkins): Verified Aaron Durbin: Looks good to me, approved Julius Werner: Looks good to me, approved
diff --git a/src/security/vboot/ec_sync.c b/src/security/vboot/ec_sync.c index f4f9c23..e7b64b2 100644 --- a/src/security/vboot/ec_sync.c +++ b/src/security/vboot/ec_sync.c @@ -23,7 +23,7 @@ #include <timer.h> #include <timestamp.h> #include <vb2_api.h> -#include <vboot_api.h> +#include <vboot_api.h> /* for VbExDisplayScreen() and VbScreenData */
#define _EC_FILENAME(select, suffix) \ (select == VB_SELECT_FIRMWARE_READONLY ? "ecro" suffix : "ecrw" suffix) diff --git a/src/vendorcode/google/chromeos/gnvs.c b/src/vendorcode/google/chromeos/gnvs.c index 8115455..04680c0 100644 --- a/src/vendorcode/google/chromeos/gnvs.c +++ b/src/vendorcode/google/chromeos/gnvs.c @@ -22,7 +22,6 @@ #include <fmap.h> #include <security/vboot/vbnv.h> #include <security/vboot/vboot_common.h> -#include <vboot_struct.h>
#include "chromeos.h" #include "gnvs.h"