[S] Change in flashrom[main]: cli_classic: Add option to use first detected chip if multiple found

[David Hendricks (Code Review)]

Attention is currently required from: Anastasia Klimchuk, Matt DeVillier.

David Hendricks has posted comments on this change by Matt DeVillier. ( https://review.coreboot.org/c/flashrom/+/85160?usp=email )

Change subject: cli_classic: Add option to use first detected chip if multiple found
......................................................................

Patch Set 2:

(1 comment)

Patchset:

PS2: 
Adding a couple comments which are probably out-of-scope and non-blocking, though you should check with flashrom's current maintainers:

- I've run into issues where write-protection capabilities are different with evil twins. So you might want to bail out if a write-protect command is used or add logic to compare write-protect capabilities with chips sharing the same ID.

- Some chips share an ID but use a different voltage. This is probably fine with internal programmers since the programmer's IO pins and and chip ought to match in this regard, but may pose an issue with external programmers such as Dediprog SF100/SF600/etc.

- Read, write, and erase opcodes are probably the same unless there's something weird going on with QE or 4BA enable/exit for a particular family of chips. I'm not sure if this is a real issue now, though it might be good to check those modes or feature bits.