Fixes: drivers/mmc/host/cb710-mmc.c: In function âcb710_wait_while_busyâ: drivers/mmc/host/cb710-mmc.c:182:6: warning: variable âerrâ set but not used [-Wunused-but-set-variable] Signed-off-by: Chris Ball <cjb@xxxxxxxxxx> Cc: MichaÅ MirosÅaw <mirq-linux@xxxxxxxxxxxx> Cc: Wolfram Sang <w.sang@xxxxxxxxxxxxxx> --- Hi Wolfram, > What about changing return 0 into return err? It gets checked in > cb710_mmc_powerup(). Thanks, good idea. - Chris. drivers/mmc/host/cb710-mmc.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/drivers/mmc/host/cb710-mmc.c b/drivers/mmc/host/cb710-mmc.c index 66b4ce5..ce2a47b 100644 --- a/drivers/mmc/host/cb710-mmc.c +++ b/drivers/mmc/host/cb710-mmc.c @@ -205,7 +205,7 @@ static int cb710_wait_while_busy(struct cb710_slot *slot, uint8_t mask) "WAIT12: waited %d loops, mask %02X, entry val %08X, exit val %08X\n", limit, mask, e, x); #endif - return 0; + return err; } static void cb710_mmc_set_transfer_size(struct cb710_slot *slot, -- 1.7.0.1 -- Chris Ball <cjb@xxxxxxxxxx> <http://printf.net/> One Laptop Per Child -- To unsubscribe from this list: send the line "unsubscribe linux-mmc" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
