[flashrom-gerrit] Change in flashrom[master]: [RFC] cli_classic: Add `--mode (read|write|verify|erase)` parameter

[Edward O'Callaghan (Code Review)]

Attention is currently required from: Nico Huber, David Hendricks.
Edward O'Callaghan has posted comments on this change. ( https://review.coreboot.org/c/flashrom/+/30979 )

Change subject: [RFC] cli_classic: Add `--mode (read|write|verify|erase)` parameter
......................................................................

Patch Set 2:

(1 comment)

Commit Message:

https://review.coreboot.org/c/flashrom/+/30979/comment/b0840b24_3d010793 
PS2, Line 14: provide better error messages.
> The parsing part is solved, Arthur looked after that in […]

I have to say I didn't really like dealing with the optional argument in the cros tree David however I couldn't think of a better alternative.

I much much prefer Nico's idea of a mode here, the combinatorics of options is much smaller and therefore the behavior is much easier to reason about.

+1'ed as I just want to make sure Daniel Campello is on board for this?