[coreboot-gerrit] [L] Change in coreboot[master]: soc/intel/cmd/block/cse: Store CSE version in CMOS memory

[Sridhar Siricilla (Code Review)]

Attention is currently required from: Subrata Banik, Dinesh Gehlot, Paul Menzel, Kapil Porwal.

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

Change subject: soc/intel/cmd/block/cse: Store CSE version in CMOS memory
......................................................................

Patch Set 20: Code-Review+1

(1 comment)

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

https://review.coreboot.org/c/coreboot/+/74995/comment/923018c2_f22c4e3a 
PS9, Line 1233: write current ISH version to CBMEM memory.
> Yes, The ISH version must be stored in both CMOS and CBMEM. […]
> The CBMEM version of ISH is used by the exposed API get_ish_version() to provide the ISH version to its users.

Can not get_ish_version() read the ISH version from CMOS and share with it's users?
I mean can't you avoid relying on CBMEM here?