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

[Sridhar Siricilla (Code Review)]

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

Change subject: src/soc/intel/common/block/cse: Add helper functions to CSE lib
......................................................................

Patch Set 14:

(1 comment)

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

https://review.coreboot.org/c/coreboot/+/35226/14/src/soc/intel/common/block... 
PS14, Line 76: me_hfs
> which status register is this ? STS1 ? if yes then we already have the definitions in soc directory. […]

Yes, this is as same as STS1. But, the STS1 structure used different field names. If we have to generalize this part, we can move all me related stuff to common. Most of the ME functions are dumping and decoding STSx registers,  and have been implemented same differently across socs.Can we do this change as part of separate task?