Felix Held has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/40351 )
Change subject: mb/dell/optiplex_9010: Add Dell OptiPlex 9010 SFF support ......................................................................
Patch Set 16: Code-Review+1
(1 comment)
looks good to me
https://review.coreboot.org/c/coreboot/+/40351/16/src/mainboard/dell/optiple... File src/mainboard/dell/optiplex_9010/mainboard.c:
https://review.coreboot.org/c/coreboot/+/40351/16/src/mainboard/dell/optiple... PS16, Line 2: /* This file is part of the coreboot project. */ please remove these lines; see https://review.coreboot.org/c/coreboot/+/41194/