Attention is currently required from: Anastasia Klimchuk.
Matt DeVillier 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)
File cli_classic.c:
https://review.coreboot.org/c/flashrom/+/85160/comment/5fbbc702_f1882400?usp... : PS2, Line 133: " --use-first-chip in cases where multiple chips are detected, use the first one found\n"
This is potentially dangerous option: because same IDs can be not only between the chips in the same […]
here's my problem: if you're flashing internally, you likely have no idea what chip is actually on the board. So when presented with 2 or more options and told to choose, how are you supposed to make that determination? With external flashing you can easily inspect the chip, so not a problem. But the vast majority of my users (ChromeOS devices owners) have no idea where the flash chip is on their device, how to access it, etc and I don't have a good way to help them resolve the question.
I don't know that we need to make the name scary. And if anything, I'd argue those overrides you used as examples are poorly named since they don't make it obvious what function they serve.