Attention is currently required from: Xiang Wang, Stefan Reinauer, Edward O'Callaghan.
Patch set 1:Code-Review -1
View Change
2 comments:
Commit Message:
File bitbang_spi.c:
Patch Set #1, Line 78: bitbang_spi_xfer(
this is its own patch.
I would not approve of such a patch either. Having two separate functions for read and write is much more efficient, and speed matters when bitbanging.
To view, visit change 49255. To unsubscribe, or for help writing mail filters, visit settings.
Gerrit-Project: flashrom
Gerrit-Branch: master
Gerrit-Change-Id: I04c1dfe132d756119229b27c3cd611d1be1abc8d
Gerrit-Change-Number: 49255
Gerrit-PatchSet: 1
Gerrit-Owner: Xiang Wang <merle@hardenedlinux.org>
Gerrit-Reviewer: Angel Pons <th3fanbus@gmail.com>
Gerrit-Reviewer: Edward O'Callaghan <quasisec@chromium.org>
Gerrit-Reviewer: Stefan Reinauer <stefan.reinauer@coreboot.org>
Gerrit-Reviewer: build bot (Jenkins) <no-reply@coreboot.org>
Gerrit-CC: Shawn C <citypw@hardenedlinux.org>
Gerrit-Attention: Xiang Wang <merle@hardenedlinux.org>
Gerrit-Attention: Stefan Reinauer <stefan.reinauer@coreboot.org>
Gerrit-Attention: Edward O'Callaghan <quasisec@chromium.org>
Gerrit-Comment-Date: Sat, 09 Jan 2021 11:09:37 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: Yes
Comment-In-Reply-To: Edward O'Callaghan <quasisec@chromium.org>
Gerrit-MessageType: comment