mkimage: Fix warning from fix for generating multi and script images again
authorMarek Vasut <marex@denx.de>
Mon, 7 Dec 2015 17:01:54 +0000 (18:01 +0100)
committerRaspbian forward porter <root@raspbian.org>
Thu, 17 Dec 2015 02:12:51 +0000 (02:12 +0000)
commit0b8bc21a123d73a3c4b72d5e876588892492a035
treef7777f9478480f79bc8daa52200ee9136a3d393f
parent74a557bfc7936ef368c0a40138d10057747bafd8
mkimage: Fix warning from fix for generating multi and script images again

Seems 6ae6e160 broke creating images in certain cases, there
are two problems with that patch.

First is that the expression "!x == 4 || !x == 6" is ambiguous.  The
intention here was "!(x == 4) || !(x == 6)" based on reading further in
the file, where this was borrowed from.  This however is interpreted by
gcc as "(!x) == 4 || (!x) == 6" and always false.  gcc-5.x will warn
about this case.

The second problem is that we do not want to test for the case of "(NOT x
is 4) OR (NOT x is 6)" but instead "(x is not equal to 4) AND (x is not
equal to 6)".  This is because in those two cases we already execute the
code question in another part of the file.  Rewrite the expression and
add parenthesis for clarity.

Signed-off-by: Marek Vasut <marex@denx.de>
Cc: Tom Rini <trini@konsulko.com>
Cc: Philippe De Swert <philippedeswert@gmail.com>
Cc: Simon Glass <sjg@chromium.org>
[trini: Re-word Marek's explanation]

Gbp-Pq: Topic upstream/mkimage-fix-multi-script
Gbp-Pq: Name 0002-mkimage-Fix-warning-from-fix-for-generating-multi-an.patch
tools/mkimage.c