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

[Nicolas Boichat (Code Review)]

Nicolas Boichat 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) */
> Oh, it means spm clk has been set on before Line 953 and, in this case, SPM DMA will start after Lin […]

Right, so that change seems useless (but does not seem to make any difference, nevertheless...)