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

[Nico Huber (Code Review)]

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

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

Patch Set 2:

(1 comment)

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.

There is the SF200 now too. So it boils down to the question if we trust it
to always work with v2. I'm ok with either way.