Paul Menzel has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/35426 )
Change subject: mb/supermicro: restructure x11ssh-tf to represent a x11 board series ......................................................................
Patch Set 27:
(3 comments)
https://review.coreboot.org/c/coreboot/+/35426/27//COMMIT_MSG Commit Message:
https://review.coreboot.org/c/coreboot/+/35426/27//COMMIT_MSG@11 PS27, Line 11: see CR 35427 Need needed. If it is, please use the Change-Id and the commit message summary.
https://review.coreboot.org/c/coreboot/+/35426/27//COMMIT_MSG@9 PS27, Line 9: Most of the X11 boards with socket LGA1151 are basically the same : boards with just some minor differences. : To be able to add more x11 boards easily like x11ssm (see CR 35427) : this restructures the x11ssh tree. Please do not break the line after a sentence. If these are supposed to be paragraphs please add a blank line between them.
https://review.coreboot.org/c/coreboot/+/35426/27//COMMIT_MSG@13 PS27, Line 13: Please elaborate a little bit. You rename the directory. What else?