[flashrom-gerrit] Change in ...flashrom[master]: dediprog: Allow 4BA on SF100

[David Hendricks (Code Review)]

David Hendricks has posted comments on this change. ( https://review.coreboot.org/c/flashrom/+/33195 )

Change subject: dediprog: Allow 4BA on SF100
......................................................................

Patch Set 2: Code-Review-1

(1 comment)

I think we can simplify that and just check protocol(). Does that sound right, Nico?

https://review.coreboot.org/#/c/33195/1/dediprog.c 
File dediprog.c:

https://review.coreboot.org/#/c/33195/1/dediprog.c@1137 
PS1, Line 1137: (dediprog_devicetype == DEV_SF600 || dediprog_devicetype == DEV_SF100) &&
Checking the devicetype seems kind of pointless, I think. The protocol() check should be sufficient.