View Change
1 comment:
File digilent_spi.c:
Patch Set #4, Line 276:
ret = spi_start_io(read_follows, writecnt);
if (ret != 0)
The overhead is already reduced by setting it to 252. Now, I agree, […]
Hmmm, might have overlooked something here. The "empirically
determined" 252 were tested with 3B addresses I guess? but we
advertise 4B address support. So this should probably be 251B
(256B - 4B address - 1B opcode).
(One nice day, this should be refactored throughout flashrom,
so the SPI drivers wouldn't have to care about the protocol
overhead.)
To view, visit change 23338. To unsubscribe, or for help writing mail filters, visit settings.
Gerrit-Project: flashrom
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I7ffcd9a2db4395816f0e8b6ce6c3b0d8e930c9e6
Gerrit-Change-Number: 23338
Gerrit-PatchSet: 8
Gerrit-Owner: Lubomir Rintel <lkundrak@v3.sk>
Gerrit-Reviewer: Lubomir Rintel <lkundrak@v3.sk>
Gerrit-Reviewer: Nico Huber <nico.h@gmx.de>
Gerrit-Reviewer: Paul Menzel <paulepanter@users.sourceforge.net>
Gerrit-Reviewer: build bot (Jenkins) <no-reply@coreboot.org>
Gerrit-Comment-Date: Thu, 16 Aug 2018 20:26:38 +0000
Gerrit-HasComments: Yes
Gerrit-HasLabels: No