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

[Huayang Duan (Code Review)]

Huayang Duan has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/34988 )

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

Patch Set 17:

(3 comments)

https://review.coreboot.org/c/coreboot/+/34988/8//COMMIT_MSG 
Commit Message:

https://review.coreboot.org/c/coreboot/+/34988/8//COMMIT_MSG@10 
PS8, Line 10: hard init-sequence to support more frequency calibration.
> those code was from mtk internal DRAM full calibration source code

Done

https://review.coreboot.org/c/coreboot/+/34988/8//COMMIT_MSG@14 
PS8, Line 14: stress test pass
> /usr/sbin/memtester 500M

Done

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

https://review.coreboot.org/c/coreboot/+/34988/14/src/soc/mediatek/mt8183/dr... 
PS14, Line 1353: if (freq_group == LP4X_DDR1600)
               : 		dramc_setting_DDR1600();
               : 	else if (freq_group == LP4X_DDR2400)
               : 		dramc_setting_DDR2400();
               : 	else if (freq_group == LP4X_DDR3600)
               : 		dramc_setting_DDR3600();
> @huayang I changed to 'switch' statement. Please check https://review.coreboot. […]

Done