[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 947: 		write32(&mtk_spm->pcm_con1, read32(&mtk_spm->pcm_con1) |
              : 			SPM_REGWR_CFG_KEY | RG_IM_SLAVE_LSB);
> 2=> Oh, it is hardware detail design. Need to consult with SPM designer.

Replied by SPM Designer:

There is a define file for SPM core to identify which part of SPM SRAM is used as Program Memory or Data Memory. After the DMA completed moving code to SPM SRAM, SPM core reset will be released and start to run code.