[coreboot-gerrit] Change in coreboot[master]: src/soc/intel/common/cse: Add helper functions to support CSE FW update

[Paul Menzel (Code Review)]

Paul Menzel has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/36786 )

Change subject: src/soc/intel/common/cse: Add helper functions to support CSE FW update
......................................................................

Patch Set 16:

(2 comments)

https://review.coreboot.org/c/coreboot/+/36786/16/src/soc/intel/common/block... 
File src/soc/intel/common/block/cse/cse.c:

https://review.coreboot.org/c/coreboot/+/36786/16/src/soc/intel/common/block... 
PS16, Line 317: uint8_t
Why this data type? Please use CB_SUCCESS and friends.

https://review.coreboot.org/c/coreboot/+/36786/16/src/soc/intel/common/block... 
PS16, Line 325: 	}
As you have the stopwatch, please print out the time it took as a debug or spew level message.