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

[Anastasia Klimchuk (Code Review)]

Attention is currently required from: Anton Samsonov, David Hendricks, Matt DeVillier.

Anastasia Klimchuk 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:
> > There are certainly cases where there is no harm in selecting the first chip of multiple. […]

Anton, thank you so much for joining. You have many useful ideas!

I am not doing anything right now because I am thinking what is the right thing to do here.
I fully understand the problem Matt is trying to solve, the IDs are repeating, more and more with time (because we never remove chip definitions, we only add new ones).
I want to make SFDP detection more smooth next year but in any case this does not cover old chips.

About the idea to use first chip: I still hold my initial opinion that this is a dangerous option in general, if we look at all the variety of use cases how people use flashrom and on which environments. However if we narrow down the use case, I understand that there is a use case when this option is fine. And actually, Matt's userbase is mostly belong to that category.

I don't currently have a good idea whether it is possible to add this option so that "safe target audience" would be using it, and everyone else would know it is dangerous and won't.
Maybe there is no safe way to add such option on upstream.

Let's keep this patch here for a while? As I said, repeated IDs are real, and this is something on my mind to improve. It was very interesting to me to read Anton's thoughts.