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 /dediprog.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 'dediprog.c')
-rw-r--r-- | dediprog.c | 6 |
1 files changed, 4 insertions, 2 deletions
@@ -496,7 +496,8 @@ static int dediprog_command_f(int timeout) static int parse_voltage(char *voltage) { char *tmp = NULL; - int i, millivolt, fraction = 0; + int i; + int millivolt = 0, fraction = 0; if (!voltage || !strlen(voltage)) { msg_perr("Empty voltage= specified.\n"); @@ -574,7 +575,8 @@ int dediprog_init(void) { struct usb_device *dev; char *voltage; - int millivolt = 3500, ret; + int millivolt = 3500; + int ret; msg_pspew("%s\n", __func__); |