build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/48071 )
Change subject: TEST-ONLY: soc/mediatek/mt8192: add dram full scan ......................................................................
Patch Set 1:
(2 comments)
https://review.coreboot.org/c/coreboot/+/48071/1/src/soc/mediatek/common/mem... File src/soc/mediatek/common/memory_test.c:
https://review.coreboot.org/c/coreboot/+/48071/1/src/soc/mediatek/common/mem... PS1, Line 6: #define LOGH printk(BIOS_ERR, "%s@%d\n", __func__, __LINE__); macros should not use a trailing semicolon
https://review.coreboot.org/c/coreboot/+/48071/1/src/soc/mediatek/common/mem... PS1, Line 118: trailing whitespace