Yu-Ping Wu has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/46389 )
Change subject: mediatek/mt8192: add spmfw loader ......................................................................
Patch Set 29:
(3 comments)
There're still unresolved comments in PS4 and PS17.
https://review.coreboot.org/c/coreboot/+/46389/17/src/soc/mediatek/mt8192/sp... File src/soc/mediatek/mt8192/spm.c:
https://review.coreboot.org/c/coreboot/+/46389/17/src/soc/mediatek/mt8192/sp... PS17, Line 945: write32(&mtk_spm->md32pcm_dma0_start, MD32PCM_DMA0_START_VAL);
Wait SPM designer response.
Update?
https://review.coreboot.org/c/coreboot/+/46389/26/src/soc/mediatek/mt8192/sp... File src/soc/mediatek/mt8192/spm.c:
https://review.coreboot.org/c/coreboot/+/46389/26/src/soc/mediatek/mt8192/sp... PS26, Line 566: write32(&mtk_spm->pcm_con1, SPM_REGWR_CFG_KEY | : (read32(&mtk_spm->pcm_con1) & ~RG_PCM_TIMER_EN_LSB));
Use SET32_BITFIELDS?
Forgot this?
https://review.coreboot.org/c/coreboot/+/46389/29/src/soc/mediatek/mt8192/sp... File src/soc/mediatek/mt8192/spm.c:
https://review.coreboot.org/c/coreboot/+/46389/29/src/soc/mediatek/mt8192/sp... PS29, Line 421: con1 = read32(&mtk_spm->pcm_con1) & RG_PCM_WDT_WAKE_LSB; : write32(&mtk_spm->pcm_con1, : con1 | SPM_REGWR_CFG_KEY | REG_EVENT_LOCK_EN_LSB | : REG_SPM_SRAM_ISOINT_B_LSB | RG_AHBMIF_APBEN_LSB | : REG_MD32_APB_INTERNAL_EN_LSB); Use clrsetbits32?