[coreboot-gerrit] Change in coreboot[master]: soc/intel/common/block/cse: Move me_read_config32() to common code

[Subrata Banik (Code Review)]

Subrata Banik has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/35225 )

Change subject: soc/intel/common/block/cse: Move me_read_config32() to common code
......................................................................

Patch Set 18:

(3 comments)

https://review.coreboot.org/c/coreboot/+/35225/18/src/soc/intel/skylake/me.c 
File src/soc/intel/skylake/me.c:

https://review.coreboot.org/c/coreboot/+/35225/18/src/soc/intel/skylake/me.c... 
PS18, Line 245: 
remove this extra line ?

https://review.coreboot.org/c/coreboot/+/35225/18/src/soc/intel/skylake/me.c... 
PS18, Line 491: return status;
goto ret; just to make consistency if you wish ?

https://review.coreboot.org/c/coreboot/+/35225/18/src/soc/intel/skylake/me.c... 
PS18, Line 495: 
same here ?