Paul Menzel has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/32090 )
Change subject: src/arch: This is an upgrade of SMBIOS to latest version 3.2 ......................................................................
Patch Set 1: Code-Review-1
(4 comments)
https://review.coreboot.org/#/c/32090/1//COMMIT_MSG Commit Message:
https://review.coreboot.org/#/c/32090/1//COMMIT_MSG@7 PS1, Line 7: src/arch: This is an upgrade of SMBIOS to latest version 3.2 Please use a verb in imperative mood, cf `git log --oneline` for how to word git commit summaries.
smbios: Add and adapt fields for version 3.2
https://review.coreboot.org/#/c/32090/1//COMMIT_MSG@9 PS1, Line 9: This is the second of 2 patches upgrading the SMBIOS interface to the latest 3.2 : First patch is in mosys. Newer required fields are added to various types definitions Please add dots to the end of sentences.
Better reference the commit by hash and commit message summary, if it has been submitted/committed already.
https://review.coreboot.org/#/c/32090/1//COMMIT_MSG@11 PS1, Line 11: Please elaborate more about the change. Thread counts on so on.
https://review.coreboot.org/#/c/32090/1/src/arch/x86/smbios.c File src/arch/x86/smbios.c:
https://review.coreboot.org/#/c/32090/1/src/arch/x86/smbios.c@570 PS1, Line 570: !(leaf_b.eax | leaf_b.ebx | leaf_b.ecx | leaf_b.edx)) Please indent the condition.