Thomas Heijligen has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/21774 )
Change subject: mb/dell: Add Dell Optiplex 790 ......................................................................
Patch Set 54:
(2 comments)
There are some FIXME tags. Are they solvable?
https://review.coreboot.org/#/c/21774/54//COMMIT_MSG Commit Message:
https://review.coreboot.org/#/c/21774/54//COMMIT_MSG@11 PS54, Line 11: There are (at least) three different mainboards: say for which of these boards this patch adds support
https://review.coreboot.org/#/c/21774/54//COMMIT_MSG@21 PS54, Line 21: This port has been tested with: USFF what is with the Minitower variant?