[coreboot-gerrit] Change in coreboot[master]: soc/intel/xeon_sp/cpx: Add multi-core init

[Maxim Polyakov (Code Review)]

Maxim Polyakov has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/40035 )

Change subject: soc/intel/xeon_sp/cpx: Add multi-core init
......................................................................

Patch Set 6: Code-Review+1

(2 comments)

https://review.coreboot.org/c/coreboot/+/40035/6/src/soc/intel/xeon_sp/cpx/c... 
File src/soc/intel/xeon_sp/cpx/chip.c:

https://review.coreboot.org/c/coreboot/+/40035/6/src/soc/intel/xeon_sp/cpx/c... 
PS6, Line 4: #include <arch/acpigen.h>
Is this inclusion really necessary?

https://review.coreboot.org/c/coreboot/+/40035/6/src/soc/intel/xeon_sp/cpx/c... 
File src/soc/intel/xeon_sp/cpx/cpu.c:

https://review.coreboot.org/c/coreboot/+/40035/6/src/soc/intel/xeon_sp/cpx/c... 
PS6, Line 64: 	cpu_read_topology(&num_phys, &num_virts);
Please add debug information about the number of threads and cores, as is done for other processors:
https://github.com/coreboot/coreboot/blob/bf48f6ab1127f3ba6a592f17ec49255f3e...