[coreboot-gerrit] [S] Change in coreboot[main]: soc/mediatek/mt8196: Set mcupm mcufw_reserved region to non-cacheable

[Jarried Lin (Code Review)]

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

Jarried Lin has posted comments on this change by Jarried Lin. ( https://review.coreboot.org/c/coreboot/+/86159?usp=email )

Change subject: soc/mediatek/mt8196: Set mcupm mcufw_reserved region to non-cacheable
......................................................................

Patch Set 4:

(5 comments)

Patchset:

PS1:
> Also remove https://review.coreboot.org/c/coreboot/+/86041/5/src/soc/mediatek/mt8196/dvf.... […]

Done

Commit Message:

https://review.coreboot.org/c/coreboot/+/86159/comment/d2e7fbac_b91ed84d?usp... :
PS1, Line 7:
> mcufw_reserved region

Done

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

https://review.coreboot.org/c/coreboot/+/86159/comment/86d1d5a3_0ad2de8c?usp... :
PS1, Line 48: 	mtk_soc_mcufw_reserved();
> MT8173/MT8183/MT8186 do not have this region. […]

Done

File src/soc/mediatek/mt8196/mmu_cmops.c:

https://review.coreboot.org/c/coreboot/+/86159/comment/3b1371c2_e36e8f22?usp... :
PS1, Line 6: cachable
> cacheable

Done

https://review.coreboot.org/c/coreboot/+/86159/comment/c88667cc_2b5e11c9?usp... :
PS1, Line 10: NONSECURE
> The original SRAM region (where `mcufw_reserved` belongs) is `SECURE_CACHED_MEM`, so I think we shou […]

Done