[coreboot-gerrit] Change in coreboot[master]: mb/dell: Add Dell Optiplex 790

[Felix Held (Code Review)]

Felix Held has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/21774 )

Change subject: mb/dell: Add Dell Optiplex 790
......................................................................

Patch Set 63:

(3 comments)

https://review.coreboot.org/c/coreboot/+/21774/63/Documentation/mainboard/de... 
File Documentation/mainboard/dell/optiplex_790.md:

https://review.coreboot.org/c/coreboot/+/21774/63/Documentation/mainboard/de... 
PS63, Line 5: ## TODO
remove todo or add possibly missing info?

https://review.coreboot.org/c/coreboot/+/21774/63/Documentation/mainboard/de... 
File Documentation/mainboard/dell/servicemode.md:

https://review.coreboot.org/c/coreboot/+/21774/63/Documentation/mainboard/de... 
PS63, Line 7: 
is the jumper connected to the Flash Descriptor Override Pin Strap?

https://review.coreboot.org/c/coreboot/+/21774/63/Documentation/mainboard/de... 
PS63, Line 8: #TODO: Add a picture of the service mode jumper inside an OptiPlex 790
i'd rather add the picture to the optiplex 790 page, since the picture is specific to that device