[coreboot-gerrit] Change in coreboot[master]: soc/intel/denverton_ns: Refactor `detect_num_cpus_via_mch()`

[Angel Pons (Code Review)]

Attention is currently required from: Mariusz Szafrański, Suresh Bellampalli, Vanessa Eusebio, Michal Motyl, Patrick Rudolph, Felix Held.
Angel Pons has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/58912 )

Change subject: soc/intel/denverton_ns: Refactor `detect_num_cpus_via_mch()`
......................................................................

Patch Set 1:

(1 comment)

Patchset:

PS1:
> looks good to me, but might be worth mentioning that this function might now detect more than CONFIG […]

Well, if it does, then CONFIG_MAX_CPUS is wrong. There's a check in `src/cpu/x86/mp_init.c` function `allocate_cpu_devices()`. I think undercounting CPUs is more serious.