[coreboot-gerrit] [S] Change in coreboot[main]: soc/intel/cmn/cse: Use CSE RW partition version directly for CBFS entry

[Subrata Banik (Code Review)]

Attention is currently required from: Arthur Heymans, Kapil Porwal, Nick Vaccaro, Werner Zeh.

Subrata Banik has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/80923?usp=email )

Change subject: soc/intel/cmn/cse: Use CSE RW partition version directly for CBFS entry
......................................................................

Patch Set 2:

(1 comment)

File src/soc/intel/common/block/cse/Makefile.mk:

https://review.coreboot.org/c/coreboot/+/80923/comment/febd3f23_3ca099f1 :
PS1, Line 101: $(eval $(word $(index),$(VERSIONS)) := $(shell printf "%d" 0x$(shell echo $(shell echo $(shell xxd -p $(TEMP_DATA)) | cut -c3-4)$(shell echo $(shell xxd -p $(TEMP_DATA)) | cut -c1-2))) )
> > Does this still support version number parts that contains more than two digits?
> > Intel tends to use build numbers (last version entry) that are 4 digits long.
>
> This is as per the ME specification where CSE major/minor/hotfix/build all are 2 byte long.

may be https://winraid.level1techs.com/t/intel-cs-me-cs-txe-cs-sps-gsc-pmc-pchc-phy... link has some details about CSE version control for last several generations.