Paul Menzel has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/35427 )
Change subject: mb/supermicro/x11: add x11ssm-f board ......................................................................
Patch Set 26:
(4 comments)
https://review.coreboot.org/c/coreboot/+/35427/26//COMMIT_MSG Commit Message:
https://review.coreboot.org/c/coreboot/+/35427/26//COMMIT_MSG@10 PS26, Line 10: Please add a short summary, what is different from the other board? (Maybe somehow integrate this in `Documentation` and reference this here.)
https://review.coreboot.org/c/coreboot/+/35427/26/Documentation/mainboard/su... File Documentation/mainboard/supermicro/x11-lga1151-series/x11ssm-f/x11ssm-tf.md:
https://review.coreboot.org/c/coreboot/+/35427/26/Documentation/mainboard/su... PS26, Line 37: Tianocore TianoCore
https://review.coreboot.org/c/coreboot/+/35427/26/Documentation/mainboard/su... PS26, Line 64: SATA S-ATA ports?
https://review.coreboot.org/c/coreboot/+/35427/26/Documentation/mainboard/su... PS26, Line 66: - Super I/O initialisation Needed?