diff options
| author | Felix Held <felix-coreboot@felixheld.de> | 2021-03-30 01:54:45 +0200 |
|---|---|---|
| committer | Martin Roth <martinroth@google.com> | 2021-03-31 21:06:33 +0000 |
| commit | 3920bc9626a4800d8ad9e63f2e92e5e1ba89bc2a (patch) | |
| tree | faaaca5284bc839f144caea0330bbb42f15d6cc8 | |
| parent | 3e343559e230d8690f3a7aecb0897822fae06aa7 (diff) | |
| download | coreboot-3920bc9626a4800d8ad9e63f2e92e5e1ba89bc2a.tar.xz | |
soc/amd/picasso/acpi: pass correct enum to acpigen_write_CSD_package
The coordtype parameter of acpigen_write_CSD_package expects a CSD_coord
enum value, but HW_ALL that got passed as parameter is a PSD_coord enum
value, so replace that with the correct CSD_HW_ALL enum value.
TEST=Timeless build results in identical binary for Mandolin.
Signed-off-by: Felix Held <felix-coreboot@felixheld.de>
Found-by: Paul Menzel <pmenzel@molgen.mpg.de>
Change-Id: I90b19345b8dc6d386b6acfa81c6c072dcd6981ca
Reviewed-on: https://review.coreboot.org/c/coreboot/+/51931
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Raul Rangel <rrangel@chromium.org>
Reviewed-by: Marshall Dawson <marshalldawson3rd@gmail.com>
| -rw-r--r-- | src/soc/amd/picasso/acpi.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/soc/amd/picasso/acpi.c b/src/soc/amd/picasso/acpi.c index d629d1c6e3..4ec08e16d1 100644 --- a/src/soc/amd/picasso/acpi.c +++ b/src/soc/amd/picasso/acpi.c @@ -354,7 +354,7 @@ void generate_cpu_entries(const struct device *device) acpigen_write_CST_package(cstate_info, ARRAY_SIZE(cstate_info)); - acpigen_write_CSD_package(cpu >> 1, threads_per_core, HW_ALL, 0); + acpigen_write_CSD_package(cpu >> 1, threads_per_core, CSD_HW_ALL, 0); acpigen_pop_len(); } |