Hung-Te Lin has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/47786 )
Change subject: soc/mediatek/mt8192: Init SSPM ......................................................................
Patch Set 17:
(1 comment)
https://review.coreboot.org/c/coreboot/+/47786/17/src/soc/mediatek/mt8192/so... File src/soc/mediatek/mt8192/soc.c:
https://review.coreboot.org/c/coreboot/+/47786/17/src/soc/mediatek/mt8192/so... PS17, Line 19: sspm_init(); a minor question: how do we decide if the various mcu init should be in mainboard.c or soc.c?