[coreboot-gerrit] [XS] Change in coreboot[main]: security/vboot: Include new gbb flag to enforce CSE sync

[Dinesh Gehlot (Code Review)]

Attention is currently required from: Julius Werner, Subrata Banik, Yu-Ping Wu.

Dinesh Gehlot has posted comments on this change by Dinesh Gehlot. ( https://review.coreboot.org/c/coreboot/+/83685?usp=email )

Change subject: security/vboot: Include new gbb flag to enforce CSE sync
......................................................................

Patch Set 2:

(6 comments)

Commit Message:

https://review.coreboot.org/c/coreboot/+/83685/comment/8e669c19_dc98da71?usp... :
PS1, Line 7: cse
> CSE

Acknowledged

https://review.coreboot.org/c/coreboot/+/83685/comment/e73b9e30_3da50ec4?usp... :
PS1, Line 7: src
> security/vboot

Acknowledged

https://review.coreboot.org/c/coreboot/+/83685/comment/01f0d937_84d982f8?usp... :
PS1, Line 12:
> > > to make sure this is test only GBB […]

Acknowledged

File src/security/vboot/Kconfig:

https://review.coreboot.org/c/coreboot/+/83685/comment/f6726aed_fc0e3f78?usp... :
PS1, Line 454: CSME
> > I actually don't know the difference between CSE and CSME (and ME). […]

Acknowledged

https://review.coreboot.org/c/coreboot/+/83685/comment/951449f6_c53e8af5?usp... :
PS1, Line 455: 	bool "Enforce CSE Sync"
> WDYT? […]

Acknowledged

https://review.coreboot.org/c/coreboot/+/83685/comment/7c66795f_cd44d776?usp... :
PS1, Line 455: Sync
> nit: `sync`

Acknowledged