Attention is currently required from: Ana Carolina Cabral.
Hello Felix Held, build bot (Jenkins),
I'd like you to reexamine a change. Please visit
https://review.coreboot.org/c/coreboot/+/85493?usp=email
to look at the new patch set (#16).
The following approvals got outdated and were removed:
Verified-1 by build bot (Jenkins)
Change subject: mb/amd/birman_plus: Kconfig
......................................................................
mb/amd/birman_plus: Kconfig
Rectify board configuration flags based on the schematics Doc. 105-D99700-00C
and User Guide #58168 (NDA).
Change-Id: Ia310ea616006479b9a052afb99d08df6a11431f4
Signed-off-by: Ana Carolina Cabral <ana.cabral(a)amd.com>
---
M src/mainboard/amd/birman_plus/Kconfig
M src/mainboard/amd/birman_plus/ec.c
M src/mainboard/amd/birman_plus/port_descriptors_glinda.c
3 files changed, 38 insertions(+), 49 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/93/85493/16
--
To view, visit https://review.coreboot.org/c/coreboot/+/85493?usp=email
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings?usp=email
Gerrit-MessageType: newpatchset
Gerrit-Project: coreboot
Gerrit-Branch: main
Gerrit-Change-Id: Ia310ea616006479b9a052afb99d08df6a11431f4
Gerrit-Change-Number: 85493
Gerrit-PatchSet: 16
Gerrit-Owner: Ana Carolina Cabral
Gerrit-Reviewer: Felix Held <felix-coreboot(a)felixheld.de>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-CC: Martin Roth <martin.roth(a)amd.corp-partner.google.com>
Gerrit-CC: Nick Kochlowski <nickkochlowski(a)gmail.com>
Gerrit-CC: Paul Menzel <paulepanter(a)mailbox.org>
Gerrit-Attention: Ana Carolina Cabral
Attention is currently required from: Ana Carolina Cabral.
Hello build bot (Jenkins),
I'd like you to reexamine a change. Please visit
https://review.coreboot.org/c/coreboot/+/86084?usp=email
to look at the new patch set (#2).
The following approvals got outdated and were removed:
Verified-1 by build bot (Jenkins)
Change subject: mb/amd/birman_plus/ec: register bits
......................................................................
mb/amd/birman_plus/ec: register bits
Rectify wrong EC module ECRAM registers bits
based on PI source code 1.0.0.1b
Change-Id: I1a13d99a55a4aa02a5cb0e67ffa4ed555f91a471
Signed-off-by: Ana Carolina Cabral <ana.cabral(a)amd.com>
---
M src/mainboard/amd/birman_plus/ec.c
1 file changed, 31 insertions(+), 37 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/84/86084/2
--
To view, visit https://review.coreboot.org/c/coreboot/+/86084?usp=email
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings?usp=email
Gerrit-MessageType: newpatchset
Gerrit-Project: coreboot
Gerrit-Branch: main
Gerrit-Change-Id: I1a13d99a55a4aa02a5cb0e67ffa4ed555f91a471
Gerrit-Change-Number: 86084
Gerrit-PatchSet: 2
Gerrit-Owner: Ana Carolina Cabral
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-Attention: Ana Carolina Cabral
Attention is currently required from: Ana Carolina Cabral.
Hello Felix Held, build bot (Jenkins),
I'd like you to reexamine a change. Please visit
https://review.coreboot.org/c/coreboot/+/85493?usp=email
to look at the new patch set (#15).
The following approvals got outdated and were removed:
Verified-1 by build bot (Jenkins)
Change subject: mb/amd/birman_plus: Kconfig
......................................................................
mb/amd/birman_plus: Kconfig
Rectify board configuration flags based on the schematics Doc. 105-D99700-00C
and User Guide #58168 (NDA).
Change-Id: Ia310ea616006479b9a052afb99d08df6a11431f4
Signed-off-by: Ana Carolina Cabral <ana.cabral(a)amd.com>
---
M src/mainboard/amd/birman_plus/Kconfig
M src/mainboard/amd/birman_plus/ec.c
M src/mainboard/amd/birman_plus/port_descriptors_glinda.c
3 files changed, 38 insertions(+), 44 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/93/85493/15
--
To view, visit https://review.coreboot.org/c/coreboot/+/85493?usp=email
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings?usp=email
Gerrit-MessageType: newpatchset
Gerrit-Project: coreboot
Gerrit-Branch: main
Gerrit-Change-Id: Ia310ea616006479b9a052afb99d08df6a11431f4
Gerrit-Change-Number: 85493
Gerrit-PatchSet: 15
Gerrit-Owner: Ana Carolina Cabral
Gerrit-Reviewer: Felix Held <felix-coreboot(a)felixheld.de>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-CC: Martin Roth <martin.roth(a)amd.corp-partner.google.com>
Gerrit-CC: Nick Kochlowski <nickkochlowski(a)gmail.com>
Gerrit-CC: Paul Menzel <paulepanter(a)mailbox.org>
Gerrit-Attention: Ana Carolina Cabral
Yidi Lin has submitted this change. ( https://review.coreboot.org/c/coreboot/+/86069?usp=email )
Change subject: soc/mediatek/mt8196: Fix issue with incomplete modem disable
......................................................................
soc/mediatek/mt8196: Fix issue with incomplete modem disable
If the modem is not completely disabled, it will cause issues with
suspend to RAM. Update the condition check in MD1_PWR_STA and increased
the MAX_RETRY_COUNT from 200 to 4000 to make sure that the modem has
sufficient time to completely disable before proceeding.
TEST=Measure the power and ensure that the DRAM enters self-refresh
mode.
BUG=b:377628718
Change-Id: I6e915d26e5b3caee36f4726bc2fc1c53cfc17bfc
Signed-off-by: Jarried Lin <jarried.lin(a)mediatek.corp-partner.google.com>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/86069
Tested-by: build bot (Jenkins) <no-reply(a)coreboot.org>
Reviewed-by: Yidi Lin <yidilin(a)google.com>
Reviewed-by: Yu-Ping Wu <yupingso(a)google.com>
---
M src/soc/mediatek/mt8196/modem_power_ctrl.c
1 file changed, 2 insertions(+), 2 deletions(-)
Approvals:
build bot (Jenkins): Verified
Yidi Lin: Looks good to me, approved
Yu-Ping Wu: Looks good to me, approved
diff --git a/src/soc/mediatek/mt8196/modem_power_ctrl.c b/src/soc/mediatek/mt8196/modem_power_ctrl.c
index 0fc4653..f849efd 100644
--- a/src/soc/mediatek/mt8196/modem_power_ctrl.c
+++ b/src/soc/mediatek/mt8196/modem_power_ctrl.c
@@ -10,7 +10,7 @@
#include <soc/spmi.h>
#define SPMI_SLAVE_4_750MV 0x78
-#define MAX_RETRY_COUNT 200
+#define MAX_RETRY_COUNT 4000
#define VSRAM_VOSEL 0x24d
#define VMODEM_VOSEL 0x24e
#define MD1_PWR_STA_MASK BIT(0)
@@ -74,7 +74,7 @@
clrbits32(&mtk_spm->md1_pwr_con, PWR_ON);
if (!retry(MAX_RETRY_COUNT,
- read32(&mtk_spm->pwr_status) & MD1_PWR_STA_MASK)) {
+ (read32(&mtk_spm->pwr_status) & MD1_PWR_STA_MASK) == 0)) {
printk(BIOS_ERR, "Check MD1_PWR_STA failed\n");
}
--
To view, visit https://review.coreboot.org/c/coreboot/+/86069?usp=email
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings?usp=email
Gerrit-MessageType: merged
Gerrit-Project: coreboot
Gerrit-Branch: main
Gerrit-Change-Id: I6e915d26e5b3caee36f4726bc2fc1c53cfc17bfc
Gerrit-Change-Number: 86069
Gerrit-PatchSet: 2
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-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Attention is currently required from: Ana Carolina Cabral.
Hello Felix Held, build bot (Jenkins),
I'd like you to reexamine a change. Please visit
https://review.coreboot.org/c/coreboot/+/85493?usp=email
to look at the new patch set (#14).
The following approvals got outdated and were removed:
Verified-1 by build bot (Jenkins)
Change subject: mb/amd/birman_plus: Kconfig
......................................................................
mb/amd/birman_plus: Kconfig
Rectify board configuration flags based on the schematics Doc. 105-D99700-00C
and User Guide #58168 (NDA).
Change-Id: Ia310ea616006479b9a052afb99d08df6a11431f4
Signed-off-by: Ana Carolina Cabral <ana.cabral(a)amd.com>
---
M src/mainboard/amd/birman_plus/Kconfig
M src/mainboard/amd/birman_plus/ec.c
M src/mainboard/amd/birman_plus/port_descriptors_glinda.c
3 files changed, 71 insertions(+), 90 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/93/85493/14
--
To view, visit https://review.coreboot.org/c/coreboot/+/85493?usp=email
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings?usp=email
Gerrit-MessageType: newpatchset
Gerrit-Project: coreboot
Gerrit-Branch: main
Gerrit-Change-Id: Ia310ea616006479b9a052afb99d08df6a11431f4
Gerrit-Change-Number: 85493
Gerrit-PatchSet: 14
Gerrit-Owner: Ana Carolina Cabral
Gerrit-Reviewer: Felix Held <felix-coreboot(a)felixheld.de>
Gerrit-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-CC: Martin Roth <martin.roth(a)amd.corp-partner.google.com>
Gerrit-CC: Nick Kochlowski <nickkochlowski(a)gmail.com>
Gerrit-CC: Paul Menzel <paulepanter(a)mailbox.org>
Gerrit-Attention: Ana Carolina Cabral
Attention is currently required from: Dinesh Gehlot, Dtrain Hsu, Eric Lai, Jayvik Desai, John Su, Kapil Porwal, Nick Vaccaro, Subrata Banik.
Hello Dinesh Gehlot, Dtrain Hsu, Eric Lai, Jayvik Desai, Kapil Porwal, Nick Vaccaro, Subrata Banik, build bot (Jenkins),
I'd like you to reexamine a change. Please visit
https://review.coreboot.org/c/coreboot/+/86079?usp=email
to look at the new patch set (#2).
The following approvals got outdated and were removed:
Code-Review+2 by Dtrain Hsu, Verified-1 by build bot (Jenkins)
Change subject: mb/brya/var/uldrenite: Remove location setting in HDA verb tables
......................................................................
mb/brya/var/uldrenite: Remove location setting in HDA verb tables
Update the HDA verb table to remove the location setting
for uldrenite.
BUG=b:374203133
TEST=emerge-nissa coreboot
Change-Id: I05767ac80c2e3d609f944d0f669fcb343c1991ef
Signed-off-by: John Su <john_su(a)compal.corp-partner.google.com>
---
M src/mainboard/google/brya/variants/uldrenite/hda_verb.c
1 file changed, 14 insertions(+), 13 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/79/86079/2
--
To view, visit https://review.coreboot.org/c/coreboot/+/86079?usp=email
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings?usp=email
Gerrit-MessageType: newpatchset
Gerrit-Project: coreboot
Gerrit-Branch: main
Gerrit-Change-Id: I05767ac80c2e3d609f944d0f669fcb343c1991ef
Gerrit-Change-Number: 86079
Gerrit-PatchSet: 2
Gerrit-Owner: John Su <john_su(a)compal.corp-partner.google.com>
Gerrit-Reviewer: Dinesh Gehlot <digehlot(a)google.com>
Gerrit-Reviewer: Dtrain Hsu <dtrain_hsu(a)compal.corp-partner.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-Reviewer: build bot (Jenkins) <no-reply(a)coreboot.org>
Gerrit-CC: Frank Wu <frank_wu(a)compal.corp-partner.google.com>
Gerrit-CC: Jamie Chen <jamie_chen(a)compal.corp-partner.google.com>
Gerrit-CC: Paul Menzel <paulepanter(a)mailbox.org>
Gerrit-Attention: Jayvik Desai <jayvik(a)google.com>
Gerrit-Attention: John Su <john_su(a)compal.corp-partner.google.com>
Gerrit-Attention: Eric Lai <ericllai(a)google.com>
Gerrit-Attention: Dtrain Hsu <dtrain_hsu(a)compal.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>