[coreboot-gerrit] Change in coreboot[master]: soc/intel/common/basecode: Implement CSE update flow

[Sridhar Siricilla (Code Review)]

Sridhar Siricilla has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/35403 )

Change subject: soc/intel/common/basecode: Implement CSE update flow
......................................................................

Patch Set 96:

(4 comments)

https://review.coreboot.org/c/coreboot/+/35403/95//COMMIT_MSG 
Commit Message:

https://review.coreboot.org/c/coreboot/+/35403/95//COMMIT_MSG@9 
PS95, Line 9: have been
> Nit: Present tense: are

Ack

https://review.coreboot.org/c/coreboot/+/35403/95//COMMIT_MSG@13 
PS95, Line 13: version
> versions

Ack

https://review.coreboot.org/c/coreboot/+/35403/95//COMMIT_MSG@15 
PS95, Line 15: * Set the CSE's next boot partition to RO using SET_BOOT_PARTITION
             : 	     HECI command.
             : 	* Send global reset command to reset the system.
> In the commit message (and comments), please use spaces for indenting the items, and align the wrapp […]

Ack

https://review.coreboot.org/c/coreboot/+/35403/95/Documentation/soc/intel/cs... 
File Documentation/soc/intel/cse_fw_update/cse_fw_update.md:

https://review.coreboot.org/c/coreboot/+/35403/95/Documentation/soc/intel/cs... 
PS95, Line 11: ChromiumOS Systems
> Chromium OS systems

Done