[coreboot-gerrit] Change in coreboot[master]: vboot: remove extraneous vboot_recovery_mode_memory_retrain

[Julius Werner (Code Review)]

Julius Werner 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 5:

(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) {
> Here's the original CL. […]

Ohhh... so it's literally just supposed to be a recovery boot that reboots automatically. I think I never quite understood how that was meant. It's probably wrong to have it on Kukui here then, too.

I thought we had a requirement to allow the system to automatically trigger a retrain from software, but I guess we don't? Or I guess you could always use flashrom to erase training data manually, too.