[L] Change in flashrom[main]: Fwd: Software info for Willem/EZOflash
Attention is currently required from: Balázs Vinarz, Keith Hui, Nico Huber. Anastasia Klimchuk has posted comments on this change by Nico Huber. ( https://review.coreboot.org/c/flashrom/+/23053?usp=email ) Change subject: Fwd: Software info for Willem/EZOflash ...................................................................... Patch Set 2: (2 comments) Patchset: PS2:
have the user specify the protocol via programmer options
If this is fine for the user, this is a good option. Would the user know which protocol they need? For example, if you were the user? dummyflasher.c can initialise more than one bus, and the desired option is set via programmer param. dummyflasher is an emulator-programmer, but it will do as an example how to init one bus or the other depending on the param. Also, this can be done step-by-step, doesn't have to be in one commit. First commit adds programmer and support for spi. Second commit can add programmer param, and more options (parallel). If default option stays the same (default what happens without param provided), this should be fine. Adding new programmer will be a large commit already, and lots of work, I think planning multiple commits will help. File meson.build: https://review.coreboot.org/c/flashrom/+/23053/comment/071e5f0a_231e3b68?usp... : PS2, Line 500:
we might need to use the ezo here as well to maintain consistency, but that wouldn't mention that th […] yes, let's use `ezo` it seems it can be made to work with not only SPI
-- To view, visit https://review.coreboot.org/c/flashrom/+/23053?usp=email To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings?usp=email Gerrit-MessageType: comment Gerrit-Project: flashrom Gerrit-Branch: main Gerrit-Change-Id: Idc4d98593076c80fd1d6ac4596940d1d7977343c Gerrit-Change-Number: 23053 Gerrit-PatchSet: 2 Gerrit-Owner: Nico Huber <nico.h@gmx.de> Gerrit-Reviewer: Anastasia Klimchuk <aklm@chromium.org> Gerrit-Reviewer: Balázs Vinarz <vinibali1@gmail.com> Gerrit-Reviewer: Paul Menzel <paulepanter@mailbox.org> Gerrit-Reviewer: build bot (Jenkins) <no-reply@coreboot.org> Gerrit-CC: Carl-Daniel Hailfinger <c-d.hailfinger.devel.2006@gmx.net> Gerrit-CC: Keith Hui <buurin@gmail.com> Gerrit-CC: Stefan Reinauer <stefan.reinauer@coreboot.org> Gerrit-Attention: Nico Huber <nico.h@gmx.de> Gerrit-Attention: Balázs Vinarz <vinibali1@gmail.com> Gerrit-Attention: Keith Hui <buurin@gmail.com> Gerrit-Comment-Date: Tue, 01 Oct 2024 05:26:48 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No Comment-In-Reply-To: Balázs Vinarz <vinibali1@gmail.com> Comment-In-Reply-To: Keith Hui <buurin@gmail.com>
participants (1)
-
Anastasia Klimchuk (Code Review)