From e9b71d91c2ea2956efb39a782bcffb5d280009b0 Mon Sep 17 00:00:00 2001 From: Angel Pons Date: Wed, 8 Sep 2021 12:06:53 +0200 Subject: [PATCH] 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 Reviewed-on: https://review.coreboot.org/c/coreboot/+/57493 Tested-by: build bot (Jenkins) Reviewed-by: Nico Huber Reviewed-by: Paul Menzel --- src/soc/intel/broadwell/pch/fadt.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) 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;