[coreboot-gerrit] [M] Change in coreboot[main]: cpu/x86/topology: Simplify CPU topology initialization

[Jérémy Compostella (Code Review)]

Attention is currently required from: Christian Walter, Jincheng Li, Johnny Lin, Jonathan Zhang, Lean Sheng Tan, Patrick Rudolph, Ronak Kanabar, Shuo Liu, Tim Chu.

Jérémy Compostella has posted comments on this change by Jérémy Compostella. ( https://review.coreboot.org/c/coreboot/+/85576?usp=email )

Change subject: cpu/x86/topology: Simplify CPU topology initialization
......................................................................

Patch Set 2:

(3 comments)

Patchset:

PS1:
> LGTM. Will do a test on SPR and GNR to make sure no regression.

I will waiting for your test results.

File src/cpu/x86/topology.c:

https://review.coreboot.org/c/coreboot/+/85576/comment/4a0b4d18_483bcba0?usp... :
PS1, Line 20: };
> Add a comment to specify to refer to Intel SDM for the level type definitions?

I added a general comment to cover the entire file.

https://review.coreboot.org/c/coreboot/+/85576/comment/caade511_8733737c?usp... :
PS1, Line 34: 	uint8_t nb_bits;
> num_bits to be more straightforward?

Done