]> git.ipfire.org Git - people/ms/u-boot.git/commit
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)
committerTom Rini <trini@konsulko.com>
Mon, 7 Dec 2015 18:28:09 +0000 (13:28 -0500)
commit34633141259e6af20959e087b1c244c3285a306b
tree704971c26999b42a27d748d8ec664099729cc012
parentcbb2df2018ab07cb8887715d30d445584e108366
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]
tools/mkimage.c