summaryrefslogtreecommitdiff
path: root/src/soc/amd/picasso/acpi.c
diff options
context:
space:
mode:
authorFelix Held <felix-coreboot@felixheld.de>2022-03-02 15:00:59 +0100
committerFelix Held <felix-coreboot@felixheld.de>2022-03-03 18:33:43 +0000
commitcf2eeff3cf9f16888881fde0efd220141347c498 (patch)
treec0edc463b47b1b117a3021444e0f7360eefaf7ff /src/soc/amd/picasso/acpi.c
parentaf4bac9e50ed7e5db76028c9efb914ca89fda6b0 (diff)
soc/amd/picasso/acpi: generate PPKG object in generate_cpu_entries call
Generate the PPKG object in the generate_cpu_entries function instead of generating the PCNT object that is the used in the PPKG method in cpu.asl to provide the PPKG object. This both simplifies the code and aligns Picasso with Cezanne and Sabrina. This will also make the code behave correctly in a case where the number of CPU cores/threads isn't a power of two. TEST=Mandolin still boots successfully to Linux desktop and dmesg doesn't show any any possibly related problems. Signed-off-by: Felix Held <felix-coreboot@felixheld.de> Change-Id: Ifb84435345c6d8c5d11a8b42e5538cfb86432780 Reviewed-on: https://review.coreboot.org/c/coreboot/+/62517 Tested-by: build bot (Jenkins) <no-reply@coreboot.org> Reviewed-by: Raul Rangel <rrangel@chromium.org>
Diffstat (limited to 'src/soc/amd/picasso/acpi.c')
-rw-r--r--src/soc/amd/picasso/acpi.c4
1 files changed, 1 insertions, 3 deletions
diff --git a/src/soc/amd/picasso/acpi.c b/src/soc/amd/picasso/acpi.c
index cd8a0a0968..9c798caffd 100644
--- a/src/soc/amd/picasso/acpi.c
+++ b/src/soc/amd/picasso/acpi.c
@@ -355,7 +355,5 @@ void generate_cpu_entries(const struct device *device)
acpigen_pop_len();
}
- acpigen_write_scope("\\");
- acpigen_write_name_integer("PCNT", logical_cores);
- acpigen_pop_len();
+ acpigen_write_processor_package("PPKG", 0, logical_cores);
}