Attention is currently required from: Subrata Banik, Angel Pons, Nikolai Artemiev.
View Change
1 comment:
Commit Message:
Patch Set #2, Line 7: ichspi.c: Read chip ID and use it to identify chip
> > Maybe we should just print a warning if >1 chip is detected? […]
In flashrom.c the top level of the CFG is:
```
@@ -915,7 +915,7 @@ int probe_flash(struct registered_master *mst, int startchip, struct flashctx *f
if (!w29ee011_can_override(flash->chip->name, chip_to_probe))
goto notfound;
- if (probe_func(flash) != 1)
```
you could return '2' from this function to mean two chips found and handle it in the above.
The actionable here, however, is likely just read the descriptor to find the number of flashes and print that when it is >0 (i.e., >1chip) but just assume the first chip.
To view, visit change 69195. To unsubscribe, or for help writing mail filters, visit settings.
Gerrit-Project: flashrom
Gerrit-Branch: master
Gerrit-Change-Id: Ia408e1e45dc6f53c0934afd6558e301abfa48ee6
Gerrit-Change-Number: 69195
Gerrit-PatchSet: 9
Gerrit-Owner: Nikolai Artemiev <nartemiev@google.com>
Gerrit-Reviewer: Angel Pons <th3fanbus@gmail.com>
Gerrit-Reviewer: Edward O'Callaghan <quasisec@chromium.org>
Gerrit-Reviewer: Subrata Banik <subratabanik@google.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply@coreboot.org>
Gerrit-Attention: Subrata Banik <subratabanik@google.com>
Gerrit-Attention: Angel Pons <th3fanbus@gmail.com>
Gerrit-Attention: Nikolai Artemiev <nartemiev@google.com>
Gerrit-Comment-Date: Sun, 04 Dec 2022 23:45:25 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: Subrata Banik <subratabanik@google.com>
Comment-In-Reply-To: Angel Pons <th3fanbus@gmail.com>
Comment-In-Reply-To: Nikolai Artemiev <nartemiev@google.com>
Gerrit-MessageType: comment