From: Guennadi Liakhovetski Date: Wed, 12 Dec 2012 14:38:13 +0000 (+0100) Subject: mmc: sh_mmcif: (cosmetic) simplify boolean return blocks X-Git-Tag: upstream/snapshot3+hdmi~5647^2~76 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=99eb9d8df996fc1695d4de687873875fbd8f6719;p=platform%2Fadaptation%2Frenesas_rcar%2Frenesas_kernel.git mmc: sh_mmcif: (cosmetic) simplify boolean return blocks Use "return condition" instead of "if (condition) return true; return false" in functions, returning bool. Signed-off-by: Guennadi Liakhovetski Signed-off-by: Chris Ball --- diff --git a/drivers/mmc/host/sh_mmcif.c b/drivers/mmc/host/sh_mmcif.c index 8aa7b0e6..de4b6d0 100644 --- a/drivers/mmc/host/sh_mmcif.c +++ b/drivers/mmc/host/sh_mmcif.c @@ -542,10 +542,7 @@ static bool sh_mmcif_next_block(struct sh_mmcif_host *host, u32 *p) host->pio_ptr = p; } - if (host->sg_idx == data->sg_len) - return false; - - return true; + return host->sg_idx != data->sg_len; } static void sh_mmcif_single_read(struct sh_mmcif_host *host, @@ -1071,9 +1068,7 @@ static bool sh_mmcif_end_cmd(struct sh_mmcif_host *host) if (!host->dma_active) { data->error = sh_mmcif_data_trans(host, host->mrq, cmd->opcode); - if (!data->error) - return true; - return false; + return !data->error; } /* Running in the IRQ thread, can sleep */