[S] Change in coreboot[master]: xeon/spr: Set ACPI CPU string for 12bit

[Felix Held (Code Review)]

Felix Held has submitted this change. ( https://review.coreboot.org/c/coreboot/+/73368 )

Change subject: xeon/spr: Set ACPI CPU string for 12bit
......................................................................

xeon/spr: Set ACPI CPU string for 12bit

On platforms with more than 255 cores the ACPI CPU string
would overflow and generate duplicates. Fix that by changing
the string to hex and use 3 digits.

Test:
Able to boot without ACPI errors on IBM/SBP1 which has
384 actives cores.

Change-Id: I1887928da0c049c27e2ec129f49051b24048b33b
Signed-off-by: Naresh Solanki 
Reviewed-on: https://review.coreboot.org/c/coreboot/+/73368
Tested-by: build bot (Jenkins) 
Reviewed-by: Angel Pons 
Reviewed-by: Jonathan Zhang 
---
M src/soc/intel/xeon_sp/spr/Kconfig
1 file changed, 26 insertions(+), 0 deletions(-)

Approvals:
  build bot (Jenkins): Verified
  Angel Pons: Looks good to me, but someone else must approve
  Jonathan Zhang: Looks good to me, approved

diff --git a/src/soc/intel/xeon_sp/spr/Kconfig b/src/soc/intel/xeon_sp/spr/Kconfig
index 899f5ae..7aa1fec 100644
--- a/src/soc/intel/xeon_sp/spr/Kconfig
+++ b/src/soc/intel/xeon_sp/spr/Kconfig
@@ -22,6 +22,10 @@
    int
    default 255

+config ACPI_CPU_STRING
+	string
+	default "\_SB.C%03X"
+
 config MAX_SOCKET_UPD
    int
    default 2