[U-Boot] [PATCH] ubifs: don't self assign values

Heiko Schocher hs at denx.de
Wed Jun 11 06:28:55 CEST 2014


Hello Jeroen,

Am 10.06.2014 23:27, schrieb Jeroen Hofstee:
> It seems the code tries to trick the compiler the argument
> is actually used. However compilers became too smart too
> fool them so easily an now warn. Checking gcc and clang
> does not seem to emit a warning. If so it should be decorated
> with unused / (void).
>
> cc: Stefan Roese<sr at denx.de>
> Signed-off-by: Jeroen Hofstee<jeroen at myspectrum.nl>
> ---
>   fs/ubifs/recovery.c | 1 -
>   fs/ubifs/scan.c     | 1 -
>   2 files changed, 2 deletions(-)

I posted an update for MTD/UBI/UBIFS with Linux 3.14 here:

[U-Boot] [RFC, PATCH v2 0/4] mtd, ubi, ubifs: resync with Linux-3.14
http://lists.denx.de/pipermail/u-boot/2014-May/180001.html

Could you try this series? There only your change in "fs/ubifs/recovery.c"
is needed ... and as this code comes from Linux, maybe it is worth
to post this change also for Linux?

bye,
Heiko
-- 
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany


More information about the U-Boot mailing list