Matt DeVillier has submitted this change. ( https://review.coreboot.org/c/coreboot/+/86493?usp=email )
Change subject: mb/google/brya/var/agah: Remove the AGAH DPTF OEM variant ......................................................................
mb/google/brya/var/agah: Remove the AGAH DPTF OEM variant
Because the AGAH EC code is based on monitoring adapter current to choose the corresponding DPTF OEM variable table, but not every project follows this design. Based on the comment below, the AGAH EC code was removed in 2023, so remove the AGAH DPTF OEM variant, allowing each OEM to adjust in EC ASL accordingly.
BUG=b:394177292 BRANCH=None TEST=None
Change-Id: I2929eaa65a518b06f32e33cc31ae4a01bcfb77e8 Signed-off-by: John Su john_su@compal.corp-partner.google.com Reviewed-on: https://review.coreboot.org/c/coreboot/+/86493 Reviewed-by: Dtrain Hsu dtrain_hsu@compal.corp-partner.google.com Tested-by: build bot (Jenkins) no-reply@coreboot.org --- M src/ec/google/chromeec/acpi/ec.asl 1 file changed, 0 insertions(+), 8 deletions(-)
Approvals: build bot (Jenkins): Verified Dtrain Hsu: Looks good to me, approved
diff --git a/src/ec/google/chromeec/acpi/ec.asl b/src/ec/google/chromeec/acpi/ec.asl index 436207c..8e241fc 100644 --- a/src/ec/google/chromeec/acpi/ec.asl +++ b/src/ec/google/chromeec/acpi/ec.asl @@ -379,14 +379,6 @@ Printf ("EC: THROTTLE START") _TZ.THRT (1) #endif - -#ifdef DPTF_ENABLE_OEM_VARIABLES - Local0 = ToInteger(EOVD) & EC_OEM_VARIABLE_DATA_MASK - _SB.DPTF.ODUP(0, Local0) - Local0 = _SB.DPTF.ODGT(0) - _SB.DPTF.ODVP() - Notify (_SB.DPTF, INT3400_ODVP_CHANGED) -#endif }
// Throttle Stop