diff options
| author | Angel Pons <th3fanbus@gmail.com> | 2021-09-08 12:06:53 +0200 |
|---|---|---|
| committer | Felix Held <felix-coreboot@felixheld.de> | 2021-09-09 14:37:42 +0000 |
| commit | e9b71d91c2ea2956efb39a782bcffb5d280009b0 (patch) | |
| tree | 80952a14df54e46fc487366a969555818bc0951f /src | |
| parent | 467eb569c05e67c7e723466fa7c8e46a2046c07a (diff) | |
soc/intel/broadwell: Set FADT `duty_offset` to 0
From ACPI specification, version 6.2 Errata A:
A `duty_width` value of 0 indicates that processor duty cycle is not
supported and the processor continuously runs at its base frequency.
Because Broadwell sets `duty_width` to 0, processor duty cycle is not
supported, and the value of `duty_offset` is ignored. For consistency
with Lynx Point, set `duty_offset` to 0.
Change-Id: I68cb85ec32a6cceda0cea29d76df6c6219b78a40
Signed-off-by: Angel Pons <th3fanbus@gmail.com>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/57493
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Nico Huber <nico.h@gmx.de>
Reviewed-by: Paul Menzel <paulepanter@mailbox.org>
Diffstat (limited to 'src')
| -rw-r--r-- | src/soc/intel/broadwell/pch/fadt.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/soc/intel/broadwell/pch/fadt.c b/src/soc/intel/broadwell/pch/fadt.c index 91924668cc..f13601dff0 100644 --- a/src/soc/intel/broadwell/pch/fadt.c +++ b/src/soc/intel/broadwell/pch/fadt.c @@ -32,7 +32,7 @@ void acpi_fill_fadt(acpi_fadt_t *fadt) /* P_LVLx not used */ fadt->p_lvl2_lat = 101; fadt->p_lvl3_lat = 1001; - fadt->duty_offset = 1; + fadt->duty_offset = 0; fadt->duty_width = 0; fadt->day_alrm = 0xd; fadt->mon_alrm = 0x00; |