[coreboot-gerrit] Change in coreboot[master]: mrc_cache: Update mrc_cache data in romstage

[Shelley Chen (Code Review)]

Shelley Chen has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/44196 )

Change subject: mrc_cache: Update mrc_cache data in romstage
......................................................................

Patch Set 24:

(2 comments)

https://review.coreboot.org/c/coreboot/+/44196/22/src/drivers/mrc_cache/mrc_... 
File src/drivers/mrc_cache/mrc_cache.c:

https://review.coreboot.org/c/coreboot/+/44196/22/src/drivers/mrc_cache/mrc_... 
PS22, Line 120:
> nit: irrelevant change.

This was not aligned before and now it's aligned.

https://review.coreboot.org/c/coreboot/+/44196/24/src/drivers/spi/Kconfig 
File src/drivers/spi/Kconfig:

https://review.coreboot.org/c/coreboot/+/44196/24/src/drivers/spi/Kconfig@45 
PS24, Line 45: BOOT_DEVICE_SPI_FLASH_NO_EARLY_WRITES
> Addition of this Kconfig should be its own separate CL along with selecting it for platforms that ne […]

Do you just want a CL with the config and the platforms selecting it in a CL prior to the romstage writeback CL (basically a no-op)?  Is it weird for platforms to be selecting a config that doesn't do anything?