diff options
author | Kyösti Mälkki <kyosti.malkki@gmail.com> | 2023-04-29 07:42:45 +0300 |
---|---|---|
committer | Kyösti Mälkki <kyosti.malkki@gmail.com> | 2023-05-01 17:33:31 +0000 |
commit | 167ccc7e65b251c3b735f82ce0675a6f85e157a8 (patch) | |
tree | 718a9d19324029c5c424f5b4d71390d4963303fd /src/mainboard/intel | |
parent | 96581b3217203b0618062799b9a117bd7cc684ff (diff) | |
download | coreboot-167ccc7e65b251c3b735f82ce0675a6f85e157a8.tar.gz coreboot-167ccc7e65b251c3b735f82ce0675a6f85e157a8.tar.bz2 coreboot-167ccc7e65b251c3b735f82ce0675a6f85e157a8.zip |
mainboard/*: Drop USB power control bits in GNVS
There is no platform-level implementation for USB port power management
in various sleepstates. The mainboards changed here never evaluate the
set GNVS variables S3U0, S3U1, S5U0 and S5U1 in ASL or in their SMI
handlers.
Change-Id: Ia1bc5969804a7346caac4ae93336efd9f0240c87
Signed-off-by: Kyösti Mälkki <kyosti.malkki@gmail.com>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/74858
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Frans Hendriks <fhendriks@eltan.com>
Reviewed-by: Arthur Heymans <arthur@aheymans.xyz>
Reviewed-by: Erik van den Bogaert <ebogaert@eltan.com>
Diffstat (limited to 'src/mainboard/intel')
-rw-r--r-- | src/mainboard/intel/baskingridge/acpi_tables.c | 11 | ||||
-rw-r--r-- | src/mainboard/intel/dcp847ske/acpi_tables.c | 8 | ||||
-rw-r--r-- | src/mainboard/intel/wtm2/acpi_tables.c | 6 |
3 files changed, 0 insertions, 25 deletions
diff --git a/src/mainboard/intel/baskingridge/acpi_tables.c b/src/mainboard/intel/baskingridge/acpi_tables.c index 927f7bc77efa..675f732128fe 100644 --- a/src/mainboard/intel/baskingridge/acpi_tables.c +++ b/src/mainboard/intel/baskingridge/acpi_tables.c @@ -9,17 +9,6 @@ void mainboard_fill_gnvs(struct global_nvs *gnvs) { - /* Enable USB ports in S3 */ - gnvs->s3u0 = 1; - gnvs->s3u1 = 1; - - /* - * Enable Front USB ports in S5 by default - * to be consistent with back port behavior - */ - gnvs->s5u0 = 1; - gnvs->s5u1 = 1; - /* TPM Present */ gnvs->tpmp = 1; diff --git a/src/mainboard/intel/dcp847ske/acpi_tables.c b/src/mainboard/intel/dcp847ske/acpi_tables.c index b8d4421f9e38..64f3f04bd066 100644 --- a/src/mainboard/intel/dcp847ske/acpi_tables.c +++ b/src/mainboard/intel/dcp847ske/acpi_tables.c @@ -6,14 +6,6 @@ void mainboard_fill_gnvs(struct global_nvs *gnvs) { - /* Enable USB ports in S3 by default */ - gnvs->s3u0 = 1; - gnvs->s3u1 = 1; - - /* Enable USB ports in S5 by default */ - gnvs->s5u0 = 1; - gnvs->s5u1 = 1; - gnvs->tcrt = CRITICAL_TEMPERATURE; gnvs->tpsv = PASSIVE_TEMPERATURE; } diff --git a/src/mainboard/intel/wtm2/acpi_tables.c b/src/mainboard/intel/wtm2/acpi_tables.c index c6d4ce8a7bf1..d24238d42cc4 100644 --- a/src/mainboard/intel/wtm2/acpi_tables.c +++ b/src/mainboard/intel/wtm2/acpi_tables.c @@ -9,12 +9,6 @@ void mainboard_fill_gnvs(struct global_nvs *gnvs) { - /* Enable USB ports in S3 */ - gnvs->s3u0 = 1; - - /* Disable USB ports in S5 */ - gnvs->s5u0 = 0; - gnvs->tcrt = CRITICAL_TEMPERATURE; gnvs->tpsv = PASSIVE_TEMPERATURE; gnvs->tmax = MAX_TEMPERATURE; |