diff options
| author | Jeroen Hofstee <jeroen@myspectrum.nl> | 2014-06-10 22:34:39 (GMT) |
|---|---|---|
| committer | Tom Rini <trini@ti.com> | 2014-06-11 20:27:06 (GMT) |
| commit | 2716077cda521ae37c21a69baf83ad5c12ca89bb (patch) | |
| tree | bb9d27256408668b37467e53763e055e48bf73a3 | |
| parent | 930e4254ec29ea5133c0772d4a3804ba1c59b275 (diff) | |
| download | u-boot-fsl-qoriq-2716077cda521ae37c21a69baf83ad5c12ca89bb.tar.xz | |
board:keymile: remove unnecessary double braces
Clang interpretes an if condition like "if ((a = b) == NULL)
as it tries to assign a value in a statement. Hence if you do
"if ((something)) it warns you that you might be confused.
Hence drop the double braces for plane if statements.
cc: Holger Brunck <holger.brunck@keymile.com>
Signed-off-by: Jeroen Hofstee <jeroen@myspectrum.nl>
| -rw-r--r-- | board/keymile/common/ivm.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/board/keymile/common/ivm.c b/board/keymile/common/ivm.c index bffc08b..b6b19cc 100644 --- a/board/keymile/common/ivm.c +++ b/board/keymile/common/ivm.c @@ -120,7 +120,7 @@ static int ivm_findinventorystring(int type, /* Look for the requested number of CR. */ while ((cr != nr) && (addr < INVENTORYDATASIZE)) { - if ((buf[addr] == '\r')) + if (buf[addr] == '\r') cr++; addr++; } |