[coreboot-gerrit] [M] Change in coreboot[master]: cpu/x86/topology: Add code to fill in topology on struct path

[Angel Pons (Code Review)]

Attention is currently required from: Jonathan Zhang, Johnny Lin, Shuming Chu (Shuming), Arthur Heymans, Kyösti Mälkki, TangYiwei.

Angel Pons has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/69222 )

Change subject: cpu/x86/topology: Add code to fill in topology on struct path
......................................................................

Patch Set 15:

(1 comment)

File src/cpu/x86/topology.c:

https://review.coreboot.org/c/coreboot/+/69222/comment/8a3b8a22_94f2ecd6 
PS15, Line 8: #define CPUID_EXTENDED_CPU_TOPOLOGY 0x0b
            : #define LEVEL_TYPE_CORE 2
            : #define LEVEL_TYPE_SMT 1
Isn't this already defined elsewhere? Why not define this in a header?