[coreboot-gerrit] Change in coreboot[master]: soc/mediatek/mt8183: Use cached calibration result for faster bootup

[Yu-Ping Wu (Code Review)]

Yu-Ping Wu has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/35164 )

Change subject: soc/mediatek/mt8183: Use cached calibration result for faster bootup
......................................................................

Patch Set 35:

(1 comment)

https://review.coreboot.org/c/coreboot/+/35164/35/src/soc/mediatek/mt8183/in... 
File src/soc/mediatek/mt8183/include/soc/emi.h:

https://review.coreboot.org/c/coreboot/+/35164/35/src/soc/mediatek/mt8183/in... 
PS35, Line 24: DRAMC_PARAM_SOURCE_SDRAM_CONFIG = 0,
             : 	DRAMC_PARAM_SOURCE_FLASH,
> What about adding a DRAMC_PARAM_SOURCE_INVALID = 0 ? […]

I've thought about it, but then we would have to deal with the _invalid_ case every time.

What do you think we should do when encountering _invalid_ source?