Attention is currently required from: Hung-Te Lin.
Hello Hung-Te Lin, build bot (Jenkins),
I'd like you to reexamine a change. Please visit
https://review.coreboot.org/c/coreboot/+/59245
to look at the new patch set (#3).
Change subject: soc/mediatek/mt8186: Add support for PMIC MT6366
......................................................................
soc/mediatek/mt8186: Add support for PMIC MT6366
Add basic support for VCORE/VDRAM1/VDDQ of MT6366.
TEST=build pass
BUG=b:202871018
Signed-off-by: James Lo <james.lo(a)mediatek.corp-partner.google.com>
Change-Id: I22e30421560a32f4a9e15899e8150376b1414494
---
M src/mainboard/google/corsola/romstage.c
M src/soc/mediatek/common/include/soc/pmic_wrap_common.h
M src/soc/mediatek/common/pmic_wrap.c
M src/soc/mediatek/mt8186/Makefile.inc
M src/soc/mediatek/mt8186/include/soc/addressmap.h
A src/soc/mediatek/mt8186/include/soc/mt6366.h
A src/soc/mediatek/mt8186/include/soc/pmic_wrap.h
A src/soc/mediatek/mt8186/mt6366.c
A src/soc/mediatek/mt8186/pmic_wrap.c
9 files changed, 1,555 insertions(+), 4 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/45/59245/3
--
To view, visit https://review.coreboot.org/c/coreboot/+/59245
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I22e30421560a32f4a9e15899e8150376b1414494
Gerrit-Change-Number: 59245
Gerrit-PatchSet: 3
Gerrit-Owner: Rex-BC Chen <rex-bc.chen(a)mediatek.corp-partner.google.com>
Gerrit-Reviewer: Hung-Te Lin <hungte(a)chromium.org>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-CC: Yu-Ping Wu <yupingso(a)google.com>
Gerrit-Attention: Hung-Te Lin <hungte(a)chromium.org>
Gerrit-MessageType: newpatchset
Attention is currently required from: Hung-Te Lin, Rex-BC Chen.
Yu-Ping Wu has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/59252 )
Change subject: soc/mediatek/mt8186: add GIC pre-initialization function
......................................................................
Patch Set 2:
(2 comments)
Commit Message:
https://review.coreboot.org/c/coreboot/+/59252/comment/7fc5595d_24a9485b
PS2, Line 9: (
One space before "(".
https://review.coreboot.org/c/coreboot/+/59252/comment/2fcda448_f21ffbcf
PS2, Line 9: GIC(generic interrupt controller) defines architectural
: requirements for handling all interrupt sources and common
: interrupt controller programming interface.
: GIC needs to be pre-initialized on MT8186, so we add this
: initialize function.
Please reflow the text to match the line length limit (72 chars).
--
To view, visit https://review.coreboot.org/c/coreboot/+/59252
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I6bf439d0d9e1ca7130a69b9006b957afca8b133c
Gerrit-Change-Number: 59252
Gerrit-PatchSet: 2
Gerrit-Owner: Rex-BC Chen <rex-bc.chen(a)mediatek.corp-partner.google.com>
Gerrit-Reviewer: Hung-Te Lin <hungte(a)chromium.org>
Gerrit-Reviewer: Yu-Ping Wu <yupingso(a)google.com>
Gerrit-CC: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-Attention: Hung-Te Lin <hungte(a)chromium.org>
Gerrit-Attention: Rex-BC Chen <rex-bc.chen(a)mediatek.corp-partner.google.com>
Gerrit-Comment-Date: Fri, 12 Nov 2021 08:39:45 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Gerrit-MessageType: comment
Attention is currently required from: Hung-Te Lin.
Hello Hung-Te Lin,
I'd like you to reexamine a change. Please visit
https://review.coreboot.org/c/coreboot/+/59254
to look at the new patch set (#2).
Change subject: soc/mediatek/mt8186: Add support for regulator VMCH and VMC
......................................................................
soc/mediatek/mt8186: Add support for regulator VMCH and VMC
Add support for VMCH and VMC of MT6366.
TEST=measure voltage 3.3V for VMCH and VMC
BUG=b:202871018
Signed-off-by: Rex-BC Chen <rex-bc.chen(a)mediatek.com>
Change-Id: Id8d98b6d827abd4713ee5c216941a9621422c7eb
---
M src/soc/mediatek/mt8186/include/soc/mt6366.h
M src/soc/mediatek/mt8186/mt6366.c
2 files changed, 124 insertions(+), 0 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/54/59254/2
--
To view, visit https://review.coreboot.org/c/coreboot/+/59254
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: Id8d98b6d827abd4713ee5c216941a9621422c7eb
Gerrit-Change-Number: 59254
Gerrit-PatchSet: 2
Gerrit-Owner: Rex-BC Chen <rex-bc.chen(a)mediatek.corp-partner.google.com>
Gerrit-Reviewer: Hung-Te Lin <hungte(a)chromium.org>
Gerrit-CC: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-Attention: Hung-Te Lin <hungte(a)chromium.org>
Gerrit-MessageType: newpatchset
Attention is currently required from: Hung-Te Lin.
Hello Hung-Te Lin, build bot (Jenkins),
I'd like you to reexamine a change. Please visit
https://review.coreboot.org/c/coreboot/+/59245
to look at the new patch set (#2).
Change subject: soc/mediatek/mt8186: Add support for PMIC MT6366
......................................................................
soc/mediatek/mt8186: Add support for PMIC MT6366
Add basic support for VCORE/VDRAM1/VDDQ of MT6366.
TEST=build pass
BUG=b:202871018
Signed-off-by: James Lo <james.lo(a)mediatek.corp-partner.google.com>
Change-Id: I22e30421560a32f4a9e15899e8150376b1414494
---
M src/mainboard/google/corsola/romstage.c
M src/soc/mediatek/common/include/soc/pmic_wrap_common.h
M src/soc/mediatek/common/pmic_wrap.c
M src/soc/mediatek/mt8186/Makefile.inc
M src/soc/mediatek/mt8186/include/soc/addressmap.h
A src/soc/mediatek/mt8186/include/soc/mt6366.h
A src/soc/mediatek/mt8186/include/soc/pmic_wrap.h
A src/soc/mediatek/mt8186/mt6366.c
A src/soc/mediatek/mt8186/pmic_wrap.c
9 files changed, 1,556 insertions(+), 4 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/45/59245/2
--
To view, visit https://review.coreboot.org/c/coreboot/+/59245
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I22e30421560a32f4a9e15899e8150376b1414494
Gerrit-Change-Number: 59245
Gerrit-PatchSet: 2
Gerrit-Owner: Rex-BC Chen <rex-bc.chen(a)mediatek.corp-partner.google.com>
Gerrit-Reviewer: Hung-Te Lin <hungte(a)chromium.org>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-CC: Yu-Ping Wu <yupingso(a)google.com>
Gerrit-Attention: Hung-Te Lin <hungte(a)chromium.org>
Gerrit-MessageType: newpatchset