[coreboot-gerrit] Change in ...coreboot[master]: arch/x86/smbios: Add type 7

[Patrick Rudolph (Code Review)]

Patrick Rudolph has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/32131 )

Change subject: arch/x86/smbios: Add type 7
......................................................................

Patch Set 3:

(3 comments)

Tested on Lenovo T520 (Intel Sandy Bridge). All caches are listed.

https://review.coreboot.org/#/c/32131/2//COMMIT_MSG 
Commit Message:

https://review.coreboot.org/#/c/32131/2//COMMIT_MSG@11 
PS2, Line 11: SMBIOS 3.1+
> Please, why isn't 3. […]

The structure was last updated with 3.1. It didn't change with 3.2.

https://review.coreboot.org/#/c/32131/2/src/arch/x86/smbios.c 
File src/arch/x86/smbios.c:

https://review.coreboot.org/#/c/32131/2/src/arch/x86/smbios.c@687 
PS2, Line 687:
> Remove the blank line?

Done

https://review.coreboot.org/#/c/32131/2/src/arch/x86/smbios.c@695 
PS2, Line 695: the
> Remove the article, because it’s not a full sentence, and no article is used in the other descriptio […]

Done