From e013df9a6701198de71cd65ab62343fc187c8219 Mon Sep 17 00:00:00 2001 From: Marc Jones Date: Wed, 23 Aug 2017 16:28:02 -0600 Subject: amd/stoneyridge: Change ACPI _PR.CPxx to _PR.Pxxx This is a bug introduced by this commit: stoneyridge: Fix CPU ASL \_PR table [commit I870f81] The following error is found in dmesg ACPI Error: [\_PR_.P000] Namespace lookup failure, AE_NOT_FOUND... ACPI Exception: AE_NOT_FOUND, During name lookup/catalog... ACPI Exception: AE_NOT_FOUND, (SSDT:AGESA ) while loading table... ACPI Error: 1 table load failures, 3 successful... ... acpi-cpufreq: overriding BIOS provided _PSD data And, "ls -la /sys/devices/system/cpu/cpufreq/" doesn't work The cause is that the Pstate SSDT table generated by AGESA expects CPU variables \_PR.Pxxx, not \_PR.CPxx as generated by coreboot. Use Kconfig to set the required string. BRANCH=none BUG=b:64885241 TEST= Check dmeg and ls -la /sys/devices/system/cpu/cpufreq/ Change-Id: I4929f9a1c39705c6df9d965c8d030f4d1f0b5e5f Signed-off-by: Marc Jones Reviewed-on: https://review.coreboot.org/21165 Tested-by: build bot (Jenkins) Reviewed-by: Aaron Durbin --- src/soc/amd/stoneyridge/acpi.c | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) (limited to 'src/soc/amd/stoneyridge/acpi.c') diff --git a/src/soc/amd/stoneyridge/acpi.c b/src/soc/amd/stoneyridge/acpi.c index 9788a91153..78fd4b71d1 100644 --- a/src/soc/amd/stoneyridge/acpi.c +++ b/src/soc/amd/stoneyridge/acpi.c @@ -244,12 +244,11 @@ void generate_cpu_entries(device_t device) printk(BIOS_DEBUG, "ACPI \\_PR report %d core(s)\n", cores); - - /* Generate BSP \_PR.CPU0 */ + /* Generate BSP \_PR.P000 */ acpigen_write_processor(0, pcontrol_blk, plen); acpigen_pop_len(); - /* Generate AP \_PR.CPUx */ + /* Generate AP \_PR.Pxxx */ pcontrol_blk = 0; plen = 0; for (cpu = 1; cpu < cores; cpu++) { -- cgit v1.2.3