[coreboot-gerrit] Change in coreboot[master]: mediatek/mt8192: add spmfw loader

[Roger Lu (Code Review)]

Roger Lu has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/46389 )

Change subject: mediatek/mt8192: add spmfw loader
......................................................................

Patch Set 16:

(1 comment)

https://review.coreboot.org/c/coreboot/+/46389/15/src/soc/mediatek/mt8192/sp... 
File src/soc/mediatek/mt8192/spm.c:

https://review.coreboot.org/c/coreboot/+/46389/15/src/soc/mediatek/mt8192/sp... 
PS15, Line 951: 	/* kick IM to fetch (only toggle IM_KICK) */
> Wait, as part of my experiments, I did print mtk_spm->md32pcm_dma0_rlct _before_ this, and the DMA i […]

Oh, it means spm clk has been set on before Line 953 and, in this case, SPM DMA will start after Line 945 is set.