[coreboot-gerrit] Change in coreboot[master]: soc/mediatek/mt8183: Add ddr geometry to support 6GB, 8GB DDR bootup

[Yu-Ping Wu (Code Review)]

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

Change subject: soc/mediatek/mt8183: Add ddr geometry to support 6GB, 8GB DDR bootup
......................................................................

Patch Set 10:

(2 comments)

https://review.coreboot.org/c/coreboot/+/41949/8/src/soc/mediatek/mt8183/emi... 
File src/soc/mediatek/mt8183/emi.c:

https://review.coreboot.org/c/coreboot/+/41949/8/src/soc/mediatek/mt8183/emi... 
PS8, Line 393: 		dramc_err("density err!\n");
> Done

Forgot to change this?

https://review.coreboot.org/c/coreboot/+/41949/10/src/soc/mediatek/mt8183/in... 
File src/soc/mediatek/mt8183/include/soc/dramc_param.h:

PS10: 
Do we really need to modify this file? Could we move enum DRAMC_PARAM_GEOMETRY_TYPE to emi.h?