summaryrefslogtreecommitdiffstats
path: root/drivers/mmc/host/mxs-mmc.c
diff options
context:
space:
mode:
authorDan Carpenter <dan.carpenter@oracle.com>2014-12-17 02:57:50 +0300
committerUlf Hansson <ulf.hansson@linaro.org>2015-01-19 09:56:10 +0100
commit9ef219c53f391ea72cd2fea836ac7cade23536eb (patch)
tree5442c108890147687b40f5491aec9ab40ccd2644 /drivers/mmc/host/mxs-mmc.c
parentde0673cdf064a102241365835673b56e10198a3e (diff)
downloadlinux-9ef219c53f391ea72cd2fea836ac7cade23536eb.tar.gz
linux-9ef219c53f391ea72cd2fea836ac7cade23536eb.tar.bz2
linux-9ef219c53f391ea72cd2fea836ac7cade23536eb.zip
mmc: mxs-mmc: remove unneeded NULL check
Static checkers complain about the inconsistent NULL checking here: drivers/mmc/host/mxs-mmc.c:680 mxs_mmc_probe() warn: variable dereferenced before check 'ssp->dmach' (see line 660) The variable can't actually be NULL so we can remove the check. Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com> Signed-off-by: Ulf Hansson <ulf.hansson@linaro.org>
Diffstat (limited to 'drivers/mmc/host/mxs-mmc.c')
-rw-r--r--drivers/mmc/host/mxs-mmc.c3
1 files changed, 1 insertions, 2 deletions
diff --git a/drivers/mmc/host/mxs-mmc.c b/drivers/mmc/host/mxs-mmc.c
index 60c4ca97a727..a82411a2c024 100644
--- a/drivers/mmc/host/mxs-mmc.c
+++ b/drivers/mmc/host/mxs-mmc.c
@@ -677,8 +677,7 @@ static int mxs_mmc_probe(struct platform_device *pdev)
return 0;
out_free_dma:
- if (ssp->dmach)
- dma_release_channel(ssp->dmach);
+ dma_release_channel(ssp->dmach);
out_clk_disable:
clk_disable_unprepare(ssp->clk);
out_mmc_free: