David Hendricks has posted comments on this change. ( https://review.coreboot.org/28272 )
Change subject: Add initial support for Dediprog SF200. ......................................................................
Patch Set 2:
(1 comment)
There were some whitespace issues that I went ahead and fixed, too.
https://review.coreboot.org/#/c/28272/1/dediprog.c File dediprog.c:
https://review.coreboot.org/#/c/28272/1/dediprog.c@1146 PS1, Line 1146: case DEV_SF200:
Should this comment be updated?
Yes. I also updated this to use a switch statement since we're dealing with three programmers, two of which are nearly identical.