Paul Menzel has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/35484 )
Change subject: util/smc: Add and use new tool smcbiosinfo ......................................................................
Patch Set 7: Code-Review+1
(3 comments)
https://review.coreboot.org/c/coreboot/+/35484/7/Documentation/mainboard/sup... File Documentation/mainboard/supermicro/flashing_on_vendorbmc.md:
https://review.coreboot.org/c/coreboot/+/35484/7/Documentation/mainboard/sup... PS7, Line 8: Bytes bytes
https://review.coreboot.org/c/coreboot/+/35484/7/Documentation/mainboard/sup... PS7, Line 9: Besides the *BoardID* it contains the *firmware version* and *build date*. Please do not wrap the line.
https://review.coreboot.org/c/coreboot/+/35484/7/Documentation/mainboard/sup... PS7, Line 12: it's not. Fits on one line?