[flashrom-gerrit] Change in flashrom[master]: [RFC][WPTST] dummyflasher: enforce write protection for W25Q128FV

[Anastasia Klimchuk (Code Review)]

Attention is currently required from: Nico Huber, Angel Pons, Sergii Dmytruk.
Anastasia Klimchuk has posted comments on this change. ( https://review.coreboot.org/c/flashrom/+/59074 )

Change subject: [RFC][WPTST] dummyflasher: enforce write protection for W25Q128FV
......................................................................

Patch Set 12: Code-Review+1

(1 comment)

File dummyflasher.c:

https://review.coreboot.org/c/flashrom/+/59074/comment/76eed054_bd56d542 
PS10, Line 264: /* XXX: should data->erase_to_zero be taken into account here? */
> It's a question to anyone who'll read the code. It just looks like a bug, but I'm not entirely sure.

It does look like a bug to me, thanks!
You can mark the comment as FIXME (instead of XXX). 
I would say bug fix needs to go as a separate patch, outside of this chain, unless other people disagree.