[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 20:

(1 comment)

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

https://review.coreboot.org/c/coreboot/+/34988/17/src/soc/mediatek/mt8183/dr... 
PS17, Line 23: impedance
> can we also pack the impedance to the shared variable? […]

I'll take a look and try to pass it as a function argument. If it's really too hard as @huayang said, I'll pack it with other shared variables.