Hung-Te Lin has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/36456 )
Change subject: soc/mediatek/mt8183: Disable DRAM DVFS in recovery mode ......................................................................
Patch Set 1:
(2 comments)
https://review.coreboot.org/c/coreboot/+/36456/1//COMMIT_MSG Commit Message:
https://review.coreboot.org/c/coreboot/+/36456/1//COMMIT_MSG@9 PS1, Line 9: In order to decrease the boot time, DRAM DVFS will not be supported in : recovery mode. Currently in recovery mode we will always run full-k with DVFS (which implies tripling memory training time for multiple frequencies) and that takes a lot of time.
However, in recovery mode the system should be running only the recovery programs with minimal services. DVFS should be not needed.
In order to improve stability and system boot time, we want to disable DVFS training in recovery mode.
https://review.coreboot.org/c/coreboot/+/36456/1/src/soc/mediatek/mt8183/mem... File src/soc/mediatek/mt8183/memory.c:
https://review.coreboot.org/c/coreboot/+/36456/1/src/soc/mediatek/mt8183/mem... PS1, Line 169: DRAM DVFS is not supported in recovery mode We can support that, so this should be 'DRAM DVFS is disabled in recovery mode.'