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

[Hung-Te Lin (Code Review)]

Hung-Te Lin has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/34988 )

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

Patch Set 2:

(1 comment)

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

https://review.coreboot.org/c/coreboot/+/34988/2/src/soc/mediatek/mt8183/dra... 
PS2, Line 23: impedance
If the only goal of this is to allow passing data between dramc_sw_impedance_cal and dramc_sw_impedance_save_reg, I wonder if it is possible to pass that as a struct to make the data flow more explicit?

struct dram_impedance {
    u32 data[2][4];
 }