mb/google/brya/var/agah: Remove variant_finalize
The EEs and I misunderstood, and apparently the vfio-pci kernel driver
will turn off the dGPU when it sees it is unused, so coreboot should
leave the dGPU on so the kernel driver can save state before it shuts it
down.
TEST=Tested by ODM
Signed-off-by: Tim Wawrzynczak <twawrzynczak@chromium.org>
Change-Id: I30b5dead7a5302f3385ddcaecfbf134c3bb68779
Reviewed-on: https://review.coreboot.org/c/coreboot/+/65181
Reviewed-by: Robert Zieba <robertzieba@google.com>
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Eric Lai <eric_lai@quanta.corp-partner.google.com>
diff --git a/src/mainboard/google/brya/variants/agah/variant.c b/src/mainboard/google/brya/variants/agah/variant.c
index b1ab79f..cc938928 100644
--- a/src/mainboard/google/brya/variants/agah/variant.c
+++ b/src/mainboard/google/brya/variants/agah/variant.c
@@ -138,23 +138,6 @@
dgpu_power_sequence_on();
}
-void variant_finalize(void)
-{
- if (acpi_is_wakeup_s3() || !gpu_powered_on)
- return;
-
- /*
- * Because the dGPU is used here in a way similar to "hybrid graphics"
- * modes, it is powered down here. The DRIVERS_GFX_NVIDIA_SAVE_BARS
- * option is selected for agah, so the BARs will be saved to ACPI memory
- * during its finalize routine. Thus, it is powered down here, as the
- * proper resources have already been allocated.
- */
- dgpu_power_sequence_off();
-
- printk(BIOS_INFO, "GPU power sequenced off.\n");
-}
-
/* Save PCI BARs to the ACPI copy of the "saved PCI config space" */
void variant_fill_ssdt(const struct device *unused)
{