[coreboot-gerrit] Change in coreboot[master]: mediatek/mt8183: Implement the dramc init setting

[Yu-Ping Wu (Code Review)]

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

Change subject: mediatek/mt8183: Implement the dramc init setting
......................................................................

Patch Set 19:

(3 comments)

https://review.coreboot.org/c/coreboot/+/34988/17/src/soc/mediatek/mt8183/dr... 
File src/soc/mediatek/mt8183/dramc_init_setting.c:

https://review.coreboot.org/c/coreboot/+/34988/17/src/soc/mediatek/mt8183/dr... 
PS17, Line 47: 0x2,
> does this come from DDR3200?  […]

Done

https://review.coreboot.org/c/coreboot/+/34988/17/src/soc/mediatek/mt8183/dr... 
PS17, Line 49: u32 sdm_pcw = 0, delta = 0;
> no need to initialize them - simply let switch() do that?

Done

https://review.coreboot.org/c/coreboot/+/34988/17/src/soc/mediatek/mt8183/dr... 
PS17, Line 76: }
> since freq_group is u8, I think we do need a  […]

Done