diff options
| author | Angel Pons <th3fanbus@gmail.com> | 2020-07-13 01:41:00 +0200 |
|---|---|---|
| committer | Angel Pons <th3fanbus@gmail.com> | 2020-07-20 13:22:44 +0000 |
| commit | 3eb8dbaee2eac62438b6c5391c09979bcaed32b0 (patch) | |
| tree | e80d10e66432d7450c12c7598be70e7cd8d60604 /src/soc/intel/denverton_ns/acpi.c | |
| parent | b74975e40364ce3b66bda54167ca20cdf6cbda35 (diff) | |
| download | coreboot-3eb8dbaee2eac62438b6c5391c09979bcaed32b0.tar.gz coreboot-3eb8dbaee2eac62438b6c5391c09979bcaed32b0.tar.bz2 coreboot-3eb8dbaee2eac62438b6c5391c09979bcaed32b0.zip | |
src: Drop useless cache flush settings in FADT
They are ignored if the ACPI_FADT_WBINVD flag is set, which is required
on current ACPI versions and only maintained for ACPI 1.0 compatibility.
Tested on Asus P8Z77-V LX2 with Linux 5.7.6 and Windows 10 at the end of
the patch train, both operating systems are able to boot successfully.
Change-Id: Ief1219542ba71d18153b64180e0ff60bd1e7687b
Signed-off-by: Angel Pons <th3fanbus@gmail.com>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/43390
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: HAOUAS Elyes <ehaouas@noos.fr>
Diffstat (limited to 'src/soc/intel/denverton_ns/acpi.c')
| -rw-r--r-- | src/soc/intel/denverton_ns/acpi.c | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/src/soc/intel/denverton_ns/acpi.c b/src/soc/intel/denverton_ns/acpi.c index 99782b7ff015..f4204da5efb5 100644 --- a/src/soc/intel/denverton_ns/acpi.c +++ b/src/soc/intel/denverton_ns/acpi.c @@ -129,8 +129,6 @@ void soc_fill_fadt(acpi_fadt_t *fadt) fadt->p_lvl2_lat = ACPI_FADT_C2_NOT_SUPPORTED; fadt->p_lvl3_lat = ACPI_FADT_C3_NOT_SUPPORTED; - fadt->flush_size = 0; /* set to 0 if WBINVD is 1 in flags */ - fadt->flush_stride = 0; /* set to 0 if WBINVD is 1 in flags */ fadt->duty_offset = 1; fadt->duty_width = 0; |