[coreboot-gerrit] Change in coreboot[master]: mb/ongy/h61m-s1: Add new mainboard {LGA1155,H61}

[Paul Menzel (Code Review)]

Paul Menzel has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/40813 )

Change subject: mb/ongy/h61m-s1: Add new mainboard {LGA1155,H61}
......................................................................

Patch Set 33: Code-Review+1

(4 comments)

https://review.coreboot.org/c/coreboot/+/40813/33//COMMIT_MSG 
Commit Message:

https://review.coreboot.org/c/coreboot/+/40813/33//COMMIT_MSG@7 
PS33, Line 7: mb/ongy/h61m-s1: Add new mainboard {LGA1155,H61}
Please add a space after the ,.

https://review.coreboot.org/c/coreboot/+/40813/33//COMMIT_MSG@15 
PS33, Line 15: - USB boot (USB 2.0 only)
With what payload? (Further down you mention SeaBIOS, maybe move that up.)

https://review.coreboot.org/c/coreboot/+/40813/33//COMMIT_MSG@20 
PS33, Line 20: ubuntu
Ubuntu

https://review.coreboot.org/c/coreboot/+/40813/33//COMMIT_MSG@26 
PS33, Line 26: - Tianocore payload (CorebootPayload)
Just a black screen?