[coreboot-gerrit] Change in coreboot[master]: soc/intel/common: Update CSE sub partition update

[Tim Wawrzynczak (Code Review)]

Attention is currently required from: Subrata Banik, Paul Menzel, Sridhar Siricilla, Kane Chen.
Tim Wawrzynczak has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/63169 )

Change subject: soc/intel/common: Update CSE sub partition update
......................................................................

Patch Set 5: Code-Review+1

(2 comments)

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

https://review.coreboot.org/c/coreboot/+/63169/comment/6a95cad6_8ff1a322 
PS2, Line 786: uint32_t size;
             : 	uint32_t start_offset;
             : 	uint32_t end_offset;
> Gentle ping.

cse_get_bp_entry_range takes uint32_t* as parameters

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

https://review.coreboot.org/c/coreboot/+/63169/comment/bdb2d505_43bb7206 
PS5, Line 955: static const char * const cse_regions[] = {"RO", "RW"};
Since you are also reusing this ("RO" anyway) on line 793, does it make sense to move this declaration to file scope ?