From mboxrd@z Thu Jan 1 00:00:00 1970 From: Heiko Schocher Date: Wed, 11 Jun 2014 06:28:55 +0200 Subject: [U-Boot] [PATCH] ubifs: don't self assign values In-Reply-To: <1402435643-11294-1-git-send-email-jeroen@myspectrum.nl> References: <1402435643-11294-1-git-send-email-jeroen@myspectrum.nl> Message-ID: <5397DB07.1010506@denx.de> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de 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 > Signed-off-by: Jeroen Hofstee > --- > 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