Nicolas Boichat has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/33569
Change subject: mediatek/mt8183: Set small core _normal mode_ regulator value ......................................................................
mediatek/mt8183: Set small core _normal mode_ regulator value
dcb2eef582 ("mediatek/mt8183: Set processor voltage to 1.05v") incorrectly sets the VPROC12 (small core) regulator sleep mode voltage value (0x140a) to 1.05V, instead of the normal mode value (0x1426), causing the regulator to be stuck at 1.05V.
The big core register is correct, let's not touch that.
BUG=b:80501386,b:135404378 BRANCH=none Test=Boots correctly on Kukui, see that regulator is not stuck anymore: grep vproc12 /sys/kernel/debug/regulator/regulator_summary
Change-Id: I2cd40db2cec70a878b4b3b895e83cd648f1bc25c Signed-off-by: Nicolas Boichat drinkcat@chromium.org --- M src/soc/mediatek/mt8183/mt6358.c 1 file changed, 1 insertion(+), 1 deletion(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/69/33569/1
diff --git a/src/soc/mediatek/mt8183/mt6358.c b/src/soc/mediatek/mt8183/mt6358.c index 7054243..774f636 100644 --- a/src/soc/mediatek/mt8183/mt6358.c +++ b/src/soc/mediatek/mt8183/mt6358.c @@ -497,7 +497,7 @@
/* Vproc11/Vproc12 to 1.05V */ {0x13a6, 0x58, 0x7F, 0}, - {0x140a, 0x58, 0x7F, 0}, + {0x1426, 0x58, 0x7F, 0}, };
static struct pmic_setting lp_setting[] = {
Nicolas Boichat has abandoned this change. ( https://review.coreboot.org/c/coreboot/+/33569 )
Change subject: mediatek/mt8183: Set small core _normal mode_ regulator value ......................................................................
Abandoned
https://review.coreboot.org/c/coreboot/+/33168