[coreboot-gerrit] Change in coreboot[master]: soc/mediatek/mt8192: Do dram full calibration

[Hung-Te Lin (Code Review)]

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

Change subject: soc/mediatek/mt8192: Do dram full calibration
......................................................................

Patch Set 18: Code-Review+2

(2 comments)

https://review.coreboot.org/c/coreboot/+/44570/11/src/soc/mediatek/mt8192/Ma... 
File src/soc/mediatek/mt8192/Makefile.inc:

https://review.coreboot.org/c/coreboot/+/44570/11/src/soc/mediatek/mt8192/Ma... 
PS11, Line 41: dram
> The name "dram" won't modify, i think it can also stay here?

Given that we currently don't have plan to change this, should be fine to keep it in current shape until we've moved the whole calibration to a shared module.

https://review.coreboot.org/c/coreboot/+/44570/11/src/soc/mediatek/mt8192/me... 
File src/soc/mediatek/mt8192/memory.c:

https://review.coreboot.org/c/coreboot/+/44570/11/src/soc/mediatek/mt8192/me... 
PS11, Line 149: DRAM-K
> stands for: dram calibration.

Ack