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 (#17).
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/17
--
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: 17
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 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 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
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
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