[coreboot-gerrit] Change in ...coreboot[master]: vboot: remove VBOOT_EC_SOFTWARE_SYNC Kconfig option

[Julius Werner (Code Review)]

Julius Werner has submitted this change and it was merged. ( https://review.coreboot.org/c/coreboot/+/31926 )

Change subject: vboot: remove VBOOT_EC_SOFTWARE_SYNC Kconfig option
......................................................................

vboot: remove VBOOT_EC_SOFTWARE_SYNC Kconfig option

This option is duplicated in depthcharge:
https://crrev.com/c/1524811

BUG=b:124141368, b:124192753, b:128737909
TEST=Build and deploy to eve
TEST=util/lint/checkpatch.pl -g origin/master..HEAD
TEST=util/abuild/abuild -B -e -y -c 50 -p none -x
TEST=make clean && make test-abuild
CQ-DEPEND=CL:1524811
BRANCH=none

Change-Id: Id8c207ec4ad5a476e24eee1ceb9e40f24d55e725
Signed-off-by: Joel Kitching 
Reviewed-on: https://review.coreboot.org/c/coreboot/+/31926
Reviewed-by: Simon Glass 
Reviewed-by: Furquan Shaikh 
Tested-by: build bot (Jenkins) 
---
M src/security/vboot/Kconfig
M src/security/vboot/vboot_handoff.c
2 files changed, 0 insertions(+), 11 deletions(-)

Approvals:
  build bot (Jenkins): Verified
  Furquan Shaikh: Looks good to me, approved
  Simon Glass: Looks good to me, approved
  Joel Kitching: Looks good to me, but someone else must approve

diff --git a/src/security/vboot/Kconfig b/src/security/vboot/Kconfig
index 004113b..42e7b19 100644
--- a/src/security/vboot/Kconfig
+++ b/src/security/vboot/Kconfig
@@ -191,15 +191,6 @@
    bool
    default n

-config VBOOT_EC_SOFTWARE_SYNC
-	bool "Enable EC software sync"
-	default y if EC_GOOGLE_CHROMEEC
-	default n
-	help
-	  EC software sync is a mechanism where the AP helps the EC verify its
-	  firmware similar to how vboot verifies the main system firmware. This
-	  option selects whether vboot should support EC software sync.
-
 config VBOOT_PHYSICAL_DEV_SWITCH
    bool
    default n
diff --git a/src/security/vboot/vboot_handoff.c b/src/security/vboot/vboot_handoff.c
index f009440..cbfedf5 100644
--- a/src/security/vboot/vboot_handoff.c
+++ b/src/security/vboot/vboot_handoff.c
@@ -81,8 +81,6 @@
    /* TODO: Set these in depthcharge */
    if (!CONFIG(VBOOT_PHYSICAL_DEV_SWITCH))
    	vb_sd->flags |= VBSD_HONOR_VIRT_DEV_SWITCH;
-	if (CONFIG(VBOOT_EC_SOFTWARE_SYNC))
-		vb_sd->flags |= VBSD_EC_SOFTWARE_SYNC;
    if (!CONFIG(VBOOT_PHYSICAL_REC_SWITCH))
    	vb_sd->flags |= VBSD_BOOT_REC_SWITCH_VIRTUAL;
    if (CONFIG(VBOOT_OPROM_MATTERS)) {