diff options
| author | hailfinger <hailfinger@2b7e53f0-3cfb-0310-b3e9-8179ed1497e1> | 2011-08-15 19:54:20 +0000 |
|---|---|---|
| committer | hailfinger <hailfinger@2b7e53f0-3cfb-0310-b3e9-8179ed1497e1> | 2011-08-15 19:54:20 +0000 |
| commit | 6fd69f0c25c1fdd855af93c073e886d56c01a31b (patch) | |
| tree | ce63847ee6cc1e63289e3f658a5d57ebd35b3619 /buspirate_spi.c | |
| parent | 5e88d071c691206c3da74b4e9ebf1559069fc4cf (diff) | |
| download | flashrom-6fd69f0c25c1fdd855af93c073e886d56c01a31b.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.
Signed-off-by: Carl-Daniel Hailfinger <c-d.hailfinger.devel.2006@gmx.net>
Acked-by: Stefan Tauner <stefan.tauner@student.tuwien.ac.at>
git-svn-id: https://code.coreboot.org/svn/flashrom/trunk@1413 2b7e53f0-3cfb-0310-b3e9-8179ed1497e1
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)) { |