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 /it87spi.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 'it87spi.c')
-rw-r--r-- | it87spi.c | 3 |
1 files changed, 2 insertions, 1 deletions
@@ -203,7 +203,8 @@ static uint16_t it87spi_probe(uint16_t port) int init_superio_ite(void) { - int i, ret = 0; + int i; + int ret = 0; for (i = 0; i < superio_count; i++) { if (superios[i].vendor != SUPERIO_VENDOR_ITE) |