From 3e304e5257d30920101e82510a17abe20b581d68 Mon Sep 17 00:00:00 2001 From: Tarun Tuli Date: Tue, 9 May 2023 13:00:26 +0000 Subject: 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 Change-Id: I8a0d62ec76caff861adce2d6c0ba2d4e4064affa Reviewed-on: https://review.coreboot.org/c/coreboot/+/75051 Reviewed-by: Eric Lai Tested-by: build bot (Jenkins) --- src/mainboard/google/brya/acpi/power.asl | 8 ++++++++ src/mainboard/google/brya/variants/hades/variant.c | 2 +- 2 files changed, 9 insertions(+), 1 deletion(-) (limited to 'src/mainboard/google/brya') 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,}, -- cgit v1.2.3