[coreboot-gerrit] [M] Change in coreboot[master]: soc/intel/xeon_sp: Report platform cpu info

[Paul Menzel (Code Review)]

Attention is currently required from: Patrick Rudolph, Jonathan Zhang, Johnny Lin, Christian Walter, Arthur Heymans, Tim Chu.

Paul Menzel has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/73391 )

Change subject: soc/intel/xeon_sp: Report platform cpu info
......................................................................

Patch Set 2:

(4 comments)

Commit Message:

https://review.coreboot.org/c/coreboot/+/73391/comment/5d42eec8_63e5722a 
PS2, Line 9: know
known

https://review.coreboot.org/c/coreboot/+/73391/comment/5c5db510_cad17dd3 
PS2, Line 11: 
Maybe add example output?

File src/include/cpu/intel/cpu_ids.h:

https://review.coreboot.org/c/coreboot/+/73391/comment/404378b6_59ea5795 
PS2, Line 50: 0x806F1
            : #define CPUID_SPR_S2			0x806F2
            : #define CPUID_SPR_S3			0x806F3
            : #define CPUID_SPR_S4			0x806F4
            : #define CPUID_SPR_S6			0x806F6
Please use lowercase letters.

File src/soc/intel/xeon_sp/report_platform.c:

https://review.coreboot.org/c/coreboot/+/73391/comment/301af7ef_a6e2136c 
PS2, Line 24: i
No need to fix the length?

https://notabs.org/coding/smallIntsBigPenalty.htm