diff options
author | Carl-Daniel Hailfinger <c-d.hailfinger.devel.2006@gmx.net> | 2011-08-15 19:54:20 +0000 |
---|---|---|
committer | Carl-Daniel Hailfinger <c-d.hailfinger.devel.2006@gmx.net> | 2011-08-15 19:54:20 +0000 |
commit | 2f664f9337046170a1d70dcd7cc075ee9b2c5405 (patch) | |
tree | ce63847ee6cc1e63289e3f658a5d57ebd35b3619 /buspirate_spi.c | |
parent | ec45370c468263bbbe8b2065b6d27101d65f6d9c (diff) | |
download | flashrom-2f664f9337046170a1d70dcd7cc075ee9b2c5405.zip flashrom-2f664f9337046170a1d70dcd7cc075ee9b2c5405.tar.gz |
Fixup of r1397
- Mixing uninitialized and initialized local variables leads to
confusion.
- ft2232_spi error cases should have gotten some error handling, and
that's the reason the curly braces were there.
- Fixing typos/wording in some places would have been nice given that
those places were touched anyway.
Corresponding to flashrom svn r1413.
Signed-off-by: Carl-Daniel Hailfinger <c-d.hailfinger.devel.2006@gmx.net>
Acked-by: Stefan Tauner <stefan.tauner@alumni.tuwien.ac.at>
Diffstat (limited to 'buspirate_spi.c')
-rw-r--r-- | buspirate_spi.c | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/buspirate_spi.c b/buspirate_spi.c index 467d271..ff1aac4 100644 --- a/buspirate_spi.c +++ b/buspirate_spi.c @@ -150,9 +150,11 @@ static int buspirate_spi_shutdown(void *data) int buspirate_spi_init(void) { unsigned char buf[512]; - int ret = 0, i, spispeed = 0x7; char *dev = NULL; char *speed = NULL; + int spispeed = 0x7; + int ret = 0; + int i; dev = extract_programmer_param("dev"); if (!dev || !strlen(dev)) { |