[coreboot-gerrit] Change in coreboot[master]: soc/intel/common/block/acpi: Fix get_cores_per_package

[Patrick Rudolph (Code Review)]

Patrick Rudolph has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/48691 )

Change subject: soc/intel/common/block/acpi: Fix get_cores_per_package
......................................................................

Patch Set 2:

(3 comments)

https://review.coreboot.org/c/coreboot/+/48691/1//COMMIT_MSG 
Commit Message:

https://review.coreboot.org/c/coreboot/+/48691/1//COMMIT_MSG@12 
PS1, Line 12: .
> because `numcpus` ends up being zero due to integer division truncation.

Done

https://review.coreboot.org/c/coreboot/+/48691/1//COMMIT_MSG@21 
PS1, Line 21: HyperThreading disabled.
> Maybe mention that this change is based on commit 920d2b77f2

use different function, marking as obsolete

https://review.coreboot.org/c/coreboot/+/48691/1/src/soc/intel/common/block/... 
File src/soc/intel/common/block/acpi/acpi.c:

https://review.coreboot.org/c/coreboot/+/48691/1/src/soc/intel/common/block/... 
PS1, Line 424: get_logical_cores_per_package();
> can you reuse cpu_read_topology()?

Done