diff options
| author | Angel Pons <th3fanbus@gmail.com> | 2021-01-04 17:18:10 +0100 |
|---|---|---|
| committer | Patrick Georgi <pgeorgi@google.com> | 2021-01-08 08:18:13 +0000 |
| commit | b6dff82aab6638e5914dce09fdef05ce8da87190 (patch) | |
| tree | 2352e3fc3581c16b2e6cf538b05a8e10bed443fd /src/cpu/intel | |
| parent | 04b2bb61fde8246cc9c3c52d785963bb2744acbc (diff) | |
| download | coreboot-b6dff82aab6638e5914dce09fdef05ce8da87190.tar.gz coreboot-b6dff82aab6638e5914dce09fdef05ce8da87190.tar.bz2 coreboot-b6dff82aab6638e5914dce09fdef05ce8da87190.zip | |
cpu/intel/model_206ax: Always return a package from _CST
The _CST method is supposed to return a package. If a mainboard used
zero for all ACPI C-states, the generated _CST would return nothing,
which is invalid. Instead, return a package with no C-state entries.
This change is a no-op, since all mainboards have at least one valid
ACPI C-state. This is what `acpigen_write_CST_package()` does, too.
Change-Id: I1f531e168683ed108a8d6d03dee6f5415fd15587
Signed-off-by: Angel Pons <th3fanbus@gmail.com>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/49092
Reviewed-by: Patrick Georgi <pgeorgi@google.com>
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Diffstat (limited to 'src/cpu/intel')
| -rw-r--r-- | src/cpu/intel/model_206ax/acpi.c | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/src/cpu/intel/model_206ax/acpi.c b/src/cpu/intel/model_206ax/acpi.c index b53e505a5c98..beb2fbc57927 100644 --- a/src/cpu/intel/model_206ax/acpi.c +++ b/src/cpu/intel/model_206ax/acpi.c @@ -31,8 +31,6 @@ static void generate_cstate_entries(acpi_cstate_t *cstates, ++cstate_count; if (c3 > 0) ++cstate_count; - if (!cstate_count) - return; acpigen_write_package(cstate_count + 1); acpigen_write_byte(cstate_count); |