Arthur Heymans has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/36840 )
Change subject: soc/intel/skylake: Clean up report_cpu_info() function ......................................................................
Patch Set 4:
(2 comments)
https://review.coreboot.org/c/coreboot/+/36840/4//COMMIT_MSG Commit Message:
https://review.coreboot.org/c/coreboot/+/36840/4//COMMIT_MSG@7 PS4, Line 7: soc/intel/skylake: Clean up report_cpu_info() function : : This patch makes below clean up for report_cpu_info() function : 1. Replace cpu_string with cpu_not_found : 2. Assign default string "Platform info not available" to cpu_not_found string : 3. Add array out of bound check while skiping leading white space in cpu brand : string name stale now?
https://review.coreboot.org/c/coreboot/+/36840/4//COMMIT_MSG@15 PS4, Line 15: TEST=Able to build and boot soraka. seems to needlessly clutter the commit message