build bot (Jenkins) has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/41949 )
Change subject: soc/mediatek/mt8183: Support 6GB, 8GB DDR bootup ......................................................................
Patch Set 6:
(2 comments)
https://review.coreboot.org/c/coreboot/+/41949/6/src/soc/mediatek/mt8183/dra... File src/soc/mediatek/mt8183/dramc_pi_calibration_api.c:
https://review.coreboot.org/c/coreboot/+/41949/6/src/soc/mediatek/mt8183/dra... PS6, Line 2694: void get_dram_infor_after_cal(u8* density_result) "foo* bar" should be "foo *bar"
https://review.coreboot.org/c/coreboot/+/41949/6/src/soc/mediatek/mt8183/inc... File src/soc/mediatek/mt8183/include/soc/dramc_pi_api.h:
https://review.coreboot.org/c/coreboot/+/41949/6/src/soc/mediatek/mt8183/inc... PS6, Line 110: void get_dram_infor_after_cal(u8* density_result); "foo* bar" should be "foo *bar"