[coreboot-gerrit] Change in ...coreboot[master]: src/arch: An upgrade of SMBIOS to latest version 3.2

[Richard Spiegel (Code Review)]

Richard Spiegel has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/31997 )

Change subject: src/arch: An upgrade of SMBIOS to latest version 3.2
......................................................................

Patch Set 2:

(4 comments)

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

https://review.coreboot.org/#/c/31997/2/src/arch/x86/smbios.c@606 
PS2, Line 606: 0x0200
#define in smbios.h?

https://review.coreboot.org/#/c/31997/2/src/arch/x86/smbios.c@607 
PS2, Line 607: 0x0100
#define in smbios.h?

https://review.coreboot.org/#/c/31997/2/src/arch/x86/smbios.c@609 
PS2, Line 609: 255
maybe "#define BYTE_LIMIT 255" on smbios.h, and use BYTE_LIMIT here?

https://review.coreboot.org/#/c/31997/2/src/arch/x86/smbios.c@611 
PS2, Line 611: 255
BYTE_LIMIT?