summaryrefslogtreecommitdiff
path: root/src/mainboard
diff options
context:
space:
mode:
authorTarun Tuli <taruntuli@google.com>2023-05-09 13:00:26 +0000
committerFelix Held <felix-coreboot@felixheld.de>2023-05-11 16:48:44 +0000
commit3e304e5257d30920101e82510a17abe20b581d68 (patch)
tree6abec9cd97befbbbc7e4158775a0f97cf784587d /src/mainboard
parent7ad8b0987ac6f3438b209ce2c6d3bd16a5c93f21 (diff)
mb/google/brya/variant/hades: Reduce PEXVDD shutoff delay for Hades
For the sequenced controlled shutdown path, there's a 10ms delay after the PEXVDD rail is disabled to permit discharge needed on Agah/Proxima. This can be dropped to 3ms for Hades designs Proto0 and forward. Once Agah board is dropped, "if CONFIG" can be cleaned up/removed. BUG=b:271167335 TEST=builds Signed-off-by: Tarun Tuli <taruntuli@google.com> Change-Id: I8a0d62ec76caff861adce2d6c0ba2d4e4064affa Reviewed-on: https://review.coreboot.org/c/coreboot/+/75051 Reviewed-by: Eric Lai <eric_lai@quanta.corp-partner.google.com> Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Diffstat (limited to 'src/mainboard')
-rw-r--r--src/mainboard/google/brya/acpi/power.asl8
-rw-r--r--src/mainboard/google/brya/variants/hades/variant.c2
2 files changed, 9 insertions, 1 deletions
diff --git a/src/mainboard/google/brya/acpi/power.asl b/src/mainboard/google/brya/acpi/power.asl
index 186b10977f..171b406178 100644
--- a/src/mainboard/google/brya/acpi/power.asl
+++ b/src/mainboard/google/brya/acpi/power.asl
@@ -131,7 +131,11 @@ Method (GC6I, 0, Serialized)
/* Ramp down PEXVDD */
CTXS (GPIO_PEXVDD_PWR_EN)
GPPL (GPIO_PEXVDD_PG, 0, 20)
+#if CONFIG(BOARD_GOOGLE_AGAH)
Sleep (10)
+#else
+ Sleep (3)
+#endif
/* Deassert EN_PPVAR_GPU_NVVDD */
CTXS (GPIO_NVVDD_PWR_EN)
@@ -272,7 +276,11 @@ Method (PGOF, 0, Serialized)
/* Ramp down PEXVDD and let rail discharge to <10% */
CTXS (GPIO_PEXVDD_PWR_EN)
GPPL (GPIO_PEXVDD_PG, 0, 20)
+#if CONFIG(BOARD_GOOGLE_AGAH)
Sleep (10)
+#else
+ Sleep (3)
+#endif
/* Ramp down NVVDD and let rail discharge to <10% */
CTXS (GPIO_NVVDD_PWR_EN)
diff --git a/src/mainboard/google/brya/variants/hades/variant.c b/src/mainboard/google/brya/variants/hades/variant.c
index 38ff79371e..5cdfabbb7c 100644
--- a/src/mainboard/google/brya/variants/hades/variant.c
+++ b/src/mainboard/google/brya/variants/hades/variant.c
@@ -61,7 +61,7 @@ static struct power_rail_sequence gpu_on_seq[] = {
/* In GCOFF entry order (i.e., power-off order) */
static struct power_rail_sequence gpu_off_seq[] = {
{ "FBVDD", FBVDD_PWR_EN, false, FBVDD_PG, 0,},
- { "PEXVDD", PEXVDD_PWR_EN, false, PEXVDD_PG, 10,},
+ { "PEXVDD", PEXVDD_PWR_EN, false, PEXVDD_PG, 3,},
{ "NVVDD+MSVDD", NVVDD_PWR_EN, false, NVVDD_PG, 2,},
{ "GPU 3.3V", GPU_3V3_PWR_EN, false, GPU_3V3_PG, 4,},
{ "GPU 1.8V", GPU_1V8_PWR_EN, false, GPU_1V8_PG, 0,},