Attention is currently required from: Hung-Te Lin, Jarried Lin.
Yidi Lin has posted comments on this change by Jarried Lin. ( https://review.coreboot.org/c/coreboot/+/86353?usp=email )
Change subject: soc/mediatek/mt8196: Fix issue of RTC recovery failed
......................................................................
Patch Set 1:
(2 comments)
Commit Message:
https://review.coreboot.org/c/coreboot/+/86353/comment/19254050_7dc32cbe?us… :
PS1, Line 16: rtc will boot normally
RTC boots normally.
https://review.coreboot.org/c/coreboot/+/86353/comment/dfc42d74_70de3ab6?us… :
PS1, Line 16: charge
charger ?
--
To view, visit https://review.coreboot.org/c/coreboot/+/86353?usp=email
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings?usp=email
Gerrit-MessageType: comment
Gerrit-Project: coreboot
Gerrit-Branch: main
Gerrit-Change-Id: Iea44f13af030f24c02993dd43a35a9d8b4f72179
Gerrit-Change-Number: 86353
Gerrit-PatchSet: 1
Gerrit-Owner: Jarried Lin <jarried.lin(a)mediatek.com>
Gerrit-Reviewer: Hung-Te Lin <hungte(a)chromium.org>
Gerrit-Reviewer: Yidi Lin <yidilin(a)google.com>
Gerrit-Reviewer: Yu-Ping Wu <yupingso(a)google.com>
Gerrit-Attention: Hung-Te Lin <hungte(a)chromium.org>
Gerrit-Attention: Jarried Lin <jarried.lin(a)mediatek.com>
Gerrit-Comment-Date: Tue, 11 Feb 2025 08:03:01 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Attention is currently required from: David Wu, Dinesh Gehlot, Eric Lai, Jayvik Desai, Kapil Porwal, Nick Vaccaro, Subrata Banik.
Ivy Jian has posted comments on this change by Ivy Jian. ( https://review.coreboot.org/c/coreboot/+/86351?usp=email )
Change subject: mb/google/nissa/var/dirks: Update GPIO table
......................................................................
Set Ready For Review
--
To view, visit https://review.coreboot.org/c/coreboot/+/86351?usp=email
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings?usp=email
Gerrit-MessageType: comment
Gerrit-Project: coreboot
Gerrit-Branch: main
Gerrit-Change-Id: Iefc6aefeae0a3a05ba04fb2718c6c3a3058de5f8
Gerrit-Change-Number: 86351
Gerrit-PatchSet: 2
Gerrit-Owner: Ivy Jian <ivy.jian(a)quanta.corp-partner.google.com>
Gerrit-Reviewer: David Wu <david_wu(a)quanta.corp-partner.google.com>
Gerrit-Reviewer: Dinesh Gehlot <digehlot(a)google.com>
Gerrit-Reviewer: Eric Lai <ericllai(a)google.com>
Gerrit-Reviewer: Jayvik Desai <jayvik(a)google.com>
Gerrit-Reviewer: Kapil Porwal <kapilporwal(a)google.com>
Gerrit-Reviewer: Nick Vaccaro <nvaccaro(a)chromium.org>
Gerrit-Reviewer: Subrata Banik <subratabanik(a)google.com>
Gerrit-Attention: Jayvik Desai <jayvik(a)google.com>
Gerrit-Attention: Eric Lai <ericllai(a)google.com>
Gerrit-Attention: David Wu <david_wu(a)quanta.corp-partner.google.com>
Gerrit-Attention: Subrata Banik <subratabanik(a)google.com>
Gerrit-Attention: Kapil Porwal <kapilporwal(a)google.com>
Gerrit-Attention: Dinesh Gehlot <digehlot(a)google.com>
Gerrit-Attention: Nick Vaccaro <nvaccaro(a)chromium.org>
Gerrit-Comment-Date: Tue, 11 Feb 2025 08:00:42 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: No
Attention is currently required from: Hung-Te Lin, Yu-Ping Wu.
Yidi Lin has posted comments on this change by Yu-Ping Wu. ( https://review.coreboot.org/c/coreboot/+/86352?usp=email )
Change subject: soc/mediatek/mt8196: Add validity check for PI_IMG
......................................................................
Patch Set 1: Code-Review+2
--
To view, visit https://review.coreboot.org/c/coreboot/+/86352?usp=email
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings?usp=email
Gerrit-MessageType: comment
Gerrit-Project: coreboot
Gerrit-Branch: main
Gerrit-Change-Id: I7b8085c1229c1a7a8cad904e166471ff8bda5cfb
Gerrit-Change-Number: 86352
Gerrit-PatchSet: 1
Gerrit-Owner: Yu-Ping Wu <yupingso(a)google.com>
Gerrit-Reviewer: Hung-Te Lin <hungte(a)chromium.org>
Gerrit-Reviewer: Yidi Lin <yidilin(a)google.com>
Gerrit-Attention: Hung-Te Lin <hungte(a)chromium.org>
Gerrit-Attention: Yu-Ping Wu <yupingso(a)google.com>
Gerrit-Comment-Date: Tue, 11 Feb 2025 07:59:57 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Attention is currently required from: Hung-Te Lin, Jarried Lin, Yidi Lin.
Yu-Ping Wu has posted comments on this change by Jarried Lin. ( https://review.coreboot.org/c/coreboot/+/86353?usp=email )
Change subject: soc/mediatek/mt8196: Fix issue of RTC recovery failed
......................................................................
Patch Set 1:
(1 comment)
Commit Message:
https://review.coreboot.org/c/coreboot/+/86353/comment/ad36fdf4_683c8d84?us… :
PS1, Line 7: issue of RTC recovery failed
RTC recovery by disabling external XTAL
--
To view, visit https://review.coreboot.org/c/coreboot/+/86353?usp=email
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings?usp=email
Gerrit-MessageType: comment
Gerrit-Project: coreboot
Gerrit-Branch: main
Gerrit-Change-Id: Iea44f13af030f24c02993dd43a35a9d8b4f72179
Gerrit-Change-Number: 86353
Gerrit-PatchSet: 1
Gerrit-Owner: Jarried Lin <jarried.lin(a)mediatek.com>
Gerrit-Reviewer: Hung-Te Lin <hungte(a)chromium.org>
Gerrit-Reviewer: Yidi Lin <yidilin(a)google.com>
Gerrit-Reviewer: Yu-Ping Wu <yupingso(a)google.com>
Gerrit-Attention: Hung-Te Lin <hungte(a)chromium.org>
Gerrit-Attention: Jarried Lin <jarried.lin(a)mediatek.com>
Gerrit-Attention: Yidi Lin <yidilin(a)google.com>
Gerrit-Comment-Date: Tue, 11 Feb 2025 07:51:15 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Jarried Lin has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/86353?usp=email )
Change subject: soc/mediatek/mt8196: Fix issue of RTC recovery failed
......................................................................
soc/mediatek/mt8196: Fix issue of RTC recovery failed
Config pmic register 0x50c bit0 which decide whether use external xtal,
this bit of mt6685 should be set to 1, which means without external
xtal.
BRANCH=rauru
BUG=b:395485005
TEST=emerge-rauru coreboot chromeos-bootimage, remove battery and
charge, then insert battery and charge, rtc will boot normally
Signed-off-by: Shunxi Zhang <ot_shunxi.zhang(a)mediatek.com>
Change-Id: Iea44f13af030f24c02993dd43a35a9d8b4f72179
---
M src/soc/mediatek/mt8196/mt6685_rtc.c
1 file changed, 4 insertions(+), 0 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/53/86353/1
diff --git a/src/soc/mediatek/mt8196/mt6685_rtc.c b/src/soc/mediatek/mt8196/mt6685_rtc.c
index 4564a31..4e63f10 100644
--- a/src/soc/mediatek/mt8196/mt6685_rtc.c
+++ b/src/soc/mediatek/mt8196/mt6685_rtc.c
@@ -345,6 +345,10 @@
{
printk(BIOS_INFO, "%s: enter\n", __func__);
+ config_interface(SCK_TOP_XTAL_SEL_ADDR, 1, SCK_TOP_XTAL_SEL_MASK,
+ SCK_TOP_XTAL_SEL_SHIFT);
+ udelay(100);
+
if (!retry(RECOVERY_RETRY_COUNT,
rtc_frequency_meter_check() &&
rtc_init_after_recovery())) {
--
To view, visit https://review.coreboot.org/c/coreboot/+/86353?usp=email
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings?usp=email
Gerrit-MessageType: newchange
Gerrit-Project: coreboot
Gerrit-Branch: main
Gerrit-Change-Id: Iea44f13af030f24c02993dd43a35a9d8b4f72179
Gerrit-Change-Number: 86353
Gerrit-PatchSet: 1
Gerrit-Owner: Jarried Lin <jarried.lin(a)mediatek.com>