[coreboot-gerrit] Change in coreboot[master]: 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: mediatek/mt8183: Use cached calibration result for faster bootup
......................................................................

Patch Set 29:

(2 comments)

https://review.coreboot.org/c/coreboot/+/35164/12/src/soc/mediatek/mt8183/dr... 
File src/soc/mediatek/mt8183/dramc_pi_calibration_api.c:

https://review.coreboot.org/c/coreboot/+/35164/12/src/soc/mediatek/mt8183/dr... 
PS12, Line 911: 		dramc_dbg("[bypass Gating]\n");
> @huayang Why adding "[" and "]" in this debug message?

I think that's okay for debug messages. After all, this file already contains this kinds of logs.

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

https://review.coreboot.org/c/coreboot/+/35164/12/src/soc/mediatek/mt8183/in... 
PS12, Line 26: sdram_params
> I think a better style is to isolate the part of serialization, for example […]

Done