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 | 082c8b559cd9f3262c9af58ac2f17f2cc8a09d8b (patch) | |
tree | ce63847ee6cc1e63289e3f658a5d57ebd35b3619 /buspirate_spi.c | |
parent | 0528b7fefa6daf35365c1dee0d21bc94e8120f78 (diff) | |
download | ast2050-flashrom-082c8b559cd9f3262c9af58ac2f17f2cc8a09d8b.zip ast2050-flashrom-082c8b559cd9f3262c9af58ac2f17f2cc8a09d8b.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)) { |