[coreboot-gerrit] Change in coreboot[master]: soc/mediatek/mt8192: Update initial settings of dramc

[Hung-Te Lin (Code Review)]

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

Change subject: soc/mediatek/mt8192: Update initial settings of dramc
......................................................................

Patch Set 39:

(1 comment)

https://review.coreboot.org/c/coreboot/+/44708/39/src/soc/mediatek/mt8192/dr... 
File src/soc/mediatek/mt8192/dramc_pi_calibration_api.c:

https://review.coreboot.org/c/coreboot/+/44708/39/src/soc/mediatek/mt8192/dr... 
PS39, Line 182: memcpy(result, &cali->impedance.result, sizeof(result));
should we make sure impedance.result has exactly same size as result? (if yes please add assert)

And can you clarify why not just make a pointer to cali->impedance.result?