[coreboot-gerrit] [XL] Change in coreboot[main]: soc/meidatek/mt8196: Add SPM loader

[Ccong Chen (Code Review)]

Attention is currently required from: Alicja Michalska, Hung-Te Lin, Jarried Lin, Wenzhen Yu, Yidi Lin, Yu-Ping Wu.

Ccong Chen has posted comments on this change by Ccong Chen. ( https://review.coreboot.org/c/coreboot/+/85599?usp=email )

Change subject: soc/meidatek/mt8196: Add SPM loader
......................................................................

Patch Set 9:

(3 comments)

File src/soc/mediatek/common/Kconfig:

https://review.coreboot.org/c/coreboot/+/85599/comment/95164730_b1d1862c?usp... :
PS8, Line 122: version2
>

Done

File src/soc/mediatek/common/spm.c:

https://review.coreboot.org/c/coreboot/+/85599/comment/50561a0b_ee979b98?usp... :
PS8, Line 30: 	ptr = (uintptr_t)pcm->buf + SPM_SYSTEM_BASE_OFFSET;
> Define SPM_SYSTEM_BASE_OFFSET as 0 in the header file. There's no need to change this. […]

Done

File src/soc/mediatek/mt8196/include/soc/spm.h:

https://review.coreboot.org/c/coreboot/+/85599/comment/48009ce4_09961ad7?usp... :
PS8, Line 66: 0x40000000
> This should be 0 for mt8196, right?

Done