[coreboot-gerrit] Change in coreboot[master]: soc/intel/common: Move CSE RW into new FMAP region to optimize boot ...

[Furquan Shaikh (Code Review)]

Furquan Shaikh has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/46312 )

Change subject: soc/intel/common:  Move CSE RW into new FMAP region to optimize boot time
......................................................................

Patch Set 11:

(5 comments)

https://review.coreboot.org/c/coreboot/+/46312/11/src/soc/intel/common/block... 
File src/soc/intel/common/block/cse/Kconfig:

https://review.coreboot.org/c/coreboot/+/46312/11/src/soc/intel/common/block... 
PS11, Line 35: Location
Name of CSE RW A region in FMAP

https://review.coreboot.org/c/coreboot/+/46312/11/src/soc/intel/common/block... 
PS11, Line 42: Location of CSE RW B in FMAP
Name of CSE RW A region in FMAP

https://review.coreboot.org/c/coreboot/+/46312/11/src/soc/intel/common/block... 
File src/soc/intel/common/block/cse/cse_lite.c:

https://review.coreboot.org/c/coreboot/+/46312/11/src/soc/intel/common/block... 
PS11, Line 504:  
nit: extra blank space not required.

https://review.coreboot.org/c/coreboot/+/46312/11/src/soc/intel/common/block... 
PS11, Line 506:  
nit: extra blank space not required.

https://review.coreboot.org/c/coreboot/+/46312/11/src/soc/intel/common/block... 
PS11, Line 797: SOC_INTEL_CSE_RW_UPDATE
This still needs to be defined in one of the preceding CLs.