Nico Huber would like Angel Pons, Miklós Márton and David Hendricks to review this change.

View Change

stlinkv3_spi: Move a declaration out of for-loop head

GCC 4.8 wants an explicit `-std=c99` or something for this to work. It
seems easier to keep the common declaration style.

Change-Id: Ic0819f82169df4d66cc949494229b0749c06e8f6
Signed-off-by: Nico Huber <nico.h@gmx.de>
---
M stlinkv3_spi.c
1 file changed, 2 insertions(+), 1 deletion(-)

git pull ssh://review.coreboot.org:29418/flashrom refs/changes/34/38034/1
diff --git a/stlinkv3_spi.c b/stlinkv3_spi.c
index 02e87bc..7338911 100644
--- a/stlinkv3_spi.c
+++ b/stlinkv3_spi.c
@@ -333,6 +333,7 @@
int rc = 0;
int actual_length = 0;
uint32_t rw_status = 0;
+ unsigned int i;

if (stlinkv3_spi_set_SPI_NSS(SPI_NSS_LOW)) {
msg_perr("Failed to set the NSS pin to low\n");
@@ -346,7 +347,7 @@
command[2] = (uint8_t)write_cnt;
command[3] = (uint8_t)(write_cnt >> 8);

- for (unsigned int i = 0; (i < 8) && (i < write_cnt); i++)
+ for (i = 0; (i < 8) && (i < write_cnt); i++)
command[4+i] = write_arr[i];

rc = libusb_bulk_transfer(stlinkv3_handle, STLINK_EP_OUT,

To view, visit change 38034. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-Project: flashrom
Gerrit-Branch: master
Gerrit-Change-Id: Ic0819f82169df4d66cc949494229b0749c06e8f6
Gerrit-Change-Number: 38034
Gerrit-PatchSet: 1
Gerrit-Owner: Nico Huber <nico.h@gmx.de>
Gerrit-Reviewer: Angel Pons <th3fanbus@gmail.com>
Gerrit-Reviewer: David Hendricks <david.hendricks@gmail.com>
Gerrit-Reviewer: Miklós Márton <martonmiklosqdev@gmail.com>
Gerrit-MessageType: newchange