Julius Werner has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/36251 )
Change subject: soc/mediatek/mt8183: Force retraining memory if requested ......................................................................
Patch Set 1: Code-Review+2
(2 comments)
https://review.coreboot.org/c/coreboot/+/36251/1//COMMIT_MSG Commit Message:
https://review.coreboot.org/c/coreboot/+/36251/1//COMMIT_MSG@14 PS1, Line 14: On x86 (MRC cache) this is primarily used for ensuring RO calibration I don't actually see any other coreboot code referencing this reason. Where is that implemented on x86? Is this actually used anywhere?
https://review.coreboot.org/c/coreboot/+/36251/1/src/soc/mediatek/mt8183/mem... File src/soc/mediatek/mt8183/memory.c:
https://review.coreboot.org/c/coreboot/+/36251/1/src/soc/mediatek/mt8183/mem... PS1, Line 173: vboot_check_recovery_request() == VB2_RECOVERY_TRAIN_AND_REBOOT) { When it says TRAIN_AND_REBOOT, should it actually reboot afterwards? What are the expected semantics for this?