[coreboot-gerrit] Change in coreboot[master]: mediatek/mt8183: add scp voltage initialization

[Julius Werner (Code Review)]

Julius Werner has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/34205 )

Change subject: mediatek/mt8183: add scp voltage initialization
......................................................................

Patch Set 5:

(2 comments)

Remember you have to always resolve comments before you can merge now.

https://review.coreboot.org/c/coreboot/+/34205/5//COMMIT_MSG 
Commit Message:

https://review.coreboot.org/c/coreboot/+/34205/5//COMMIT_MSG@10 
PS5, Line 10:
> Hi Paul, […]

Ack

https://review.coreboot.org/c/coreboot/+/34205/4/src/soc/mediatek/mt8183/mt6... 
File src/soc/mediatek/mt8183/mt6358.c:

https://review.coreboot.org/c/coreboot/+/34205/4/src/soc/mediatek/mt8183/mt6... 
PS4, Line 762: }
> Can you use a "static struct pmic_setting scp_setting" for this, instead? This should save a bit of  […]

Done