Attention is currently required from: Hung-Te Lin, Xixi Chen. Yu-Ping Wu has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/62065 )
Change subject: src/mediatek/mt8186: Get dram size from CBMEM_ID_MEMINFO ......................................................................
Patch Set 14:
(5 comments)
Commit Message:
https://review.coreboot.org/c/coreboot/+/62065/comment/d36a3d65_e38aafa4 PS14, Line 10: should update from calculate it from the mem chip info stored in
https://review.coreboot.org/c/coreboot/+/62065/comment/947f3959_8395b2e5 PS14, Line 13: TEST=Build pass on Kingler Did you see the correct "dram size: ..." log?
File src/soc/mediatek/mt8186/emi.c:
https://review.coreboot.org/c/coreboot/+/62065/comment/2d034a75_e2fd2bfa PS14, Line 16: p channel
https://review.coreboot.org/c/coreboot/+/62065/comment/7c7c7da2_05d7e98c PS14, Line 17: uint64_t size_t
https://review.coreboot.org/c/coreboot/+/62065/comment/b9a6b76f_44268625 PS14, Line 20: != NULL Remove