[coreboot-gerrit] [S] Change in coreboot[master]: src/soc/intel/common/block/cse/: Store CSE RW FW version during cse_f...

[Julius Werner (Code Review)]

Attention is currently required from: Dinesh Gehlot.

Julius Werner has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/76395?usp=email )

Change subject: src/soc/intel/common/block/cse/: Store CSE RW FW version during cse_fw_sync
......................................................................

Patch Set 1:

(1 comment)

File src/soc/intel/common/block/cse/cse_lite.c:

https://review.coreboot.org/c/coreboot/+/76395/comment/a2edc9d9_8ef85f61 :
PS1, Line 211: 	version = cbmem_add(CBMEM_ID_CSE_PARTITION_VERSION, sizeof(*version));
FWIW, this patch series may be a good idea as an independent improvement in general, but it's not really needed just for this (transferring information from romstage to romstage, as opposed to from actual earlier stages). You could easily just store the value as a global variable and then move it into CBMEM with an init hook).