Tristan Hsieh has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/32396
Change subject: mediatek: Set processor voltage to 1.05v ......................................................................
mediatek: Set processor voltage to 1.05v
The maximum CPU frequency is 1417MHz with current processor voltage (0.8v). Set processor voltage to 1.05v for higher CPU frequency.
BUG=b:80501386 BRANCH=none Test=Boots correctly on Kukui
Change-Id: I24ecdac2c85d3f012d9235449c0d727d727dc185 Signed-off-by: Tristan Shieh tristan.shieh@mediatek.com --- M src/soc/mediatek/mt8183/mt6358.c 1 file changed, 4 insertions(+), 0 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/96/32396/1
diff --git a/src/soc/mediatek/mt8183/mt6358.c b/src/soc/mediatek/mt8183/mt6358.c index 3dc9fe2..321e5cf 100644 --- a/src/soc/mediatek/mt8183/mt6358.c +++ b/src/soc/mediatek/mt8183/mt6358.c @@ -493,6 +493,10 @@ /* [2:1]: RG_LDO_VSRAM_PROC12_TRACK_ON_CTRL */ /* [2:2]: RG_LDO_VSRAM_PROC12_TRACK_VPROC12_ON_CTRL */ {0x1B6C, 0x6, 0x6, 0}, + + /* Vproc11/Vproc12 to 1.05V */ + {0x13a6, 0x58, 0x7F, 0}, + {0x140a, 0x58, 0x7F, 0}, };
static struct pmic_setting lp_setting[] = {
You-Cheng Syu has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/32396 )
Change subject: mediatek: Set processor voltage to 1.05v ......................................................................
Patch Set 1: Code-Review+1
You-Cheng Syu has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/32396 )
Change subject: mediatek: Set processor voltage to 1.05v ......................................................................
Patch Set 1:
(1 comment)
https://review.coreboot.org/#/c/32396/1//COMMIT_MSG Commit Message:
https://review.coreboot.org/#/c/32396/1//COMMIT_MSG@7 PS1, Line 7: mediatek mediatek/mt8183
Hung-Te Lin has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/32396 )
Change subject: mediatek: Set processor voltage to 1.05v ......................................................................
Patch Set 1: Code-Review+2
Hello Julius Werner, You-Cheng Syu, Hung-Te Lin, build bot (Jenkins),
I'd like you to reexamine a change. Please visit
https://review.coreboot.org/c/coreboot/+/32396
to look at the new patch set (#2).
Change subject: mediatek/mt8183: Set processor voltage to 1.05v ......................................................................
mediatek/mt8183: Set processor voltage to 1.05v
The maximum CPU frequency is 1417MHz with current processor voltage (0.8v). Set processor voltage to 1.05v for higher CPU frequency.
BUG=b:80501386 BRANCH=none Test=Boots correctly on Kukui
Change-Id: I24ecdac2c85d3f012d9235449c0d727d727dc185 Signed-off-by: Tristan Shieh tristan.shieh@mediatek.com --- M src/soc/mediatek/mt8183/mt6358.c 1 file changed, 4 insertions(+), 0 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/96/32396/2
Tristan Hsieh has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/32396 )
Change subject: mediatek/mt8183: Set processor voltage to 1.05v ......................................................................
Patch Set 2:
(1 comment)
https://review.coreboot.org/#/c/32396/1//COMMIT_MSG Commit Message:
https://review.coreboot.org/#/c/32396/1//COMMIT_MSG@7 PS1, Line 7: mediatek
mediatek/mt8183
Done
Patrick Georgi has submitted this change and it was merged. ( https://review.coreboot.org/c/coreboot/+/32396 )
Change subject: mediatek/mt8183: Set processor voltage to 1.05v ......................................................................
mediatek/mt8183: Set processor voltage to 1.05v
The maximum CPU frequency is 1417MHz with current processor voltage (0.8v). Set processor voltage to 1.05v for higher CPU frequency.
BUG=b:80501386 BRANCH=none Test=Boots correctly on Kukui
Change-Id: I24ecdac2c85d3f012d9235449c0d727d727dc185 Signed-off-by: Tristan Shieh tristan.shieh@mediatek.com Reviewed-on: https://review.coreboot.org/c/coreboot/+/32396 Tested-by: build bot (Jenkins) no-reply@coreboot.org Reviewed-by: You-Cheng Syu youcheng@google.com Reviewed-by: Hung-Te Lin hungte@chromium.org --- M src/soc/mediatek/mt8183/mt6358.c 1 file changed, 4 insertions(+), 0 deletions(-)
Approvals: build bot (Jenkins): Verified Hung-Te Lin: Looks good to me, approved You-Cheng Syu: Looks good to me, but someone else must approve
diff --git a/src/soc/mediatek/mt8183/mt6358.c b/src/soc/mediatek/mt8183/mt6358.c index 5803271..8162e3a 100644 --- a/src/soc/mediatek/mt8183/mt6358.c +++ b/src/soc/mediatek/mt8183/mt6358.c @@ -493,6 +493,10 @@ /* [2:1]: RG_LDO_VSRAM_PROC12_TRACK_ON_CTRL */ /* [2:2]: RG_LDO_VSRAM_PROC12_TRACK_VPROC12_ON_CTRL */ {0x1B6C, 0x6, 0x6, 0}, + + /* Vproc11/Vproc12 to 1.05V */ + {0x13a6, 0x58, 0x7F, 0}, + {0x140a, 0x58, 0x7F, 0}, };
static struct pmic_setting lp_setting[] = {
-
Hung-Te Lin (Code Review) -
Patrick Georgi (Code Review)
-
Tristan Hsieh (Code Review)
-
You-Cheng Syu (Code Review)