diff options
author | Dan Carpenter <dan.carpenter@oracle.com> | 2014-12-17 02:57:50 +0300 |
---|---|---|
committer | Ulf Hansson <ulf.hansson@linaro.org> | 2015-01-19 09:56:10 +0100 |
commit | 9ef219c53f391ea72cd2fea836ac7cade23536eb (patch) | |
tree | 5442c108890147687b40f5491aec9ab40ccd2644 /drivers/mmc/host/mxs-mmc.c | |
parent | de0673cdf064a102241365835673b56e10198a3e (diff) | |
download | linux-stable-9ef219c53f391ea72cd2fea836ac7cade23536eb.tar.gz linux-stable-9ef219c53f391ea72cd2fea836ac7cade23536eb.tar.bz2 linux-stable-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.c | 3 |
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: |