Shelley Chen has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/35803 )
Change subject: mb/google/hatch: Preserve MRC training data across FW update ......................................................................
Patch Set 2:
(2 comments)
https://review.coreboot.org/c/coreboot/+/35803/1//COMMIT_MSG Commit Message:
https://review.coreboot.org/c/coreboot/+/35803/1//COMMIT_MSG@7 PS1, Line 7: Preserve MRC training data across FW update
I'm not sure why this would be desired over retraining. […]
The reason we're doing this is because after we do a FW update, it takes awhile to boot to kernel (like 15 seconds) due to memory training and people are getting confused if the device is actually booting or not and try to hard reset it at some point by holding the power button down.
https://review.coreboot.org/c/coreboot/+/35803/1/src/mainboard/google/hatch/... File src/mainboard/google/hatch/chromeos.fmd:
https://review.coreboot.org/c/coreboot/+/35803/1/src/mainboard/google/hatch/... PS1, Line 22: RW_PRESERVE
We can just add PRESERVE property to UNIFIED_MRC_CACHE. […]
Done