board:keymile: remove unnecessary double braces
authorJeroen Hofstee <jeroen@myspectrum.nl>
Tue, 10 Jun 2014 22:34:39 +0000 (00:34 +0200)
committerTom Rini <trini@ti.com>
Wed, 11 Jun 2014 20:27:06 +0000 (16:27 -0400)
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>
board/keymile/common/ivm.c

index bffc08be951e0a7c4cdec47f47c8cc48558c0852..b6b19ccb8ea1379b2e0fbed5b86df26e40aa1a1e 100644 (file)
@@ -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++;
        }