Furquan Shaikh has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/39343 )
Change subject: vboot: remove extraneous vboot_recovery_mode_memory_retrain ......................................................................
Patch Set 6:
(1 comment)
https://review.coreboot.org/c/coreboot/+/39343/5/src/soc/mediatek/mt8183/mem... File src/soc/mediatek/mt8183/memory.c:
https://review.coreboot.org/c/coreboot/+/39343/5/src/soc/mediatek/mt8183/mem... PS5, Line 179: vboot_check_recovery_request() == VB2_RECOVERY_TRAIN_AND_REBOOT) {
http://cs/chromeos_public/src/platform/factory/py/test/pytests/mrc_cache. […]
That test ensures recovery MRC cache gets generated on setting appropriate recovery reason. It was done to ensure that we do not miss this during firmware qualification.