Attention is currently required from: Hung-Te Lin, Yu-Ping Wu.
Yidi Lin has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/79990?usp=email )
Change subject: soc/mediatek: Add EARLY_INIT_MMU kconfig option ......................................................................
Patch Set 3:
(3 comments)
File src/soc/mediatek/common/Kconfig:
https://review.coreboot.org/c/coreboot/+/79990/comment/11678f00_ff20e757 : PS1, Line 94: EARLY_INIT_MMU
How about `EARLY_MMU_INIT`?
Done
https://review.coreboot.org/c/coreboot/+/79990/comment/b67c25c1_5aa3b561 : PS1, Line 98: When this opiton is enabled, `mtk_mmu_init()` will be done in
`'opiton' may be misspelled - perhaps 'option'?` […]
Done
File src/soc/mediatek/mt8188/Makefile.inc:
https://review.coreboot.org/c/coreboot/+/79990/comment/0e009b9f_d078b226 : PS1, Line 13: bootblock-$(CONFIG_EARLY_INIT_MMU) += ../common/bootblock.c
This reads a bit weird to me, as common/bootblock.c may not be directly related to `mtk_mmu_init`. […]
Done