[coreboot-gerrit] Change in coreboot[master]: soc/mediatek/mt8183: Run DRAM full calibration

[Yu-Ping Wu (Code Review)]

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

Change subject: soc/mediatek/mt8183: Run DRAM full calibration
......................................................................

Patch Set 50:

(2 comments)

https://review.coreboot.org/c/coreboot/+/35110/47/src/mainboard/google/kukui... 
File src/mainboard/google/kukui/Makefile.inc:

https://review.coreboot.org/c/coreboot/+/35110/47/src/mainboard/google/kukui... 
PS47, Line 35: DRAM_CBFS := $(CONFIG_CBFS_PREFIX)/dram
> That may be a good idea. In the meantime I think we should make it a Kconfig. […]

Let's add the config in another CL.

https://review.coreboot.org/c/coreboot/+/35110/47/src/mainboard/google/kukui... 
File src/mainboard/google/kukui/romstage.c:

https://review.coreboot.org/c/coreboot/+/35110/47/src/mainboard/google/kukui... 
PS47, Line 44: assert(length <= CALIBRATION_REGION_SIZE);
> Can we replace this with Static_assert?

Done