[M] Change in coreboot[master]: soc/intel/xeon_sp: Don't sort struct device cpus for numa

[Jonathan Zhang (Code Review)]

Attention is currently required from: Arthur Heymans, Anjaneya "Reddy" Chagam, Johnny Lin, Christian Walter, Angel Pons, Shuming Chu (Shuming), TangYiwei.

Jonathan Zhang has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/68912 )

Change subject: soc/intel/xeon_sp: Don't sort struct device cpus for numa
......................................................................

Patch Set 5:

(2 comments)

File src/soc/intel/xeon_sp/cpx/cpu.c:

https://review.coreboot.org/c/coreboot/+/68912/comment/7270320e_192ba425 
PS3, Line 81: apic_id
> I don't see any place update package_id and the bottom layer uses apic_id to get package information […]

Should it be?
cpu->path.apic.package_id = get_cpu_package(cpu->path.apic.apic_id);

File src/soc/intel/xeon_sp/include/soc/util.h:

https://review.coreboot.org/c/coreboot/+/68912/comment/73dc5a46_fb2d9fa7 
PS3, Line 15: uint8_t get_cpu_package(const uint32_t apic_id);
> This function uses apic_id to get package information and the bottom layer uses the name "package",  […]

Agree. In this context, package means socket. I saw the comment.