Attention is currently required from: Rex-BC Chen, Yu-Ping Wu. Hung-Te Lin has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/63109 )
Change subject: soc/mediatek: Add a configurate "USE_CBMEM_DRAM_INFO" ......................................................................
Patch Set 4:
(1 comment)
Commit Message:
https://review.coreboot.org/c/coreboot/+/63109/comment/56153f58_24e9cdf7 PS4, Line 9: There is a assertion error for platforms which dram_info are not : implemented in blob, Try to not only mention "assertion error" because people can't tell what's causing that. Also not mentioning blob because we should focus on what (memory init) instead of how (blob impl).
Try something like this:
The memory initialization reference code didn't support returning DRAM information in the old platforms, for example MT8192 and MT8195. So we have to add a new configuration USE_CBMEM_DRAM_INFO to make sure the common code will try to get DRAM information on new platforms supporting that.
-
Hung-Te Lin (Code Review)