diff options
| author | Dan Carpenter <dan.carpenter@oracle.com> | 2014-12-16 18:57:50 -0500 |
|---|---|---|
| committer | Ulf Hansson <ulf.hansson@linaro.org> | 2015-01-19 03:56:10 -0500 |
| commit | 9ef219c53f391ea72cd2fea836ac7cade23536eb (patch) | |
| tree | 5442c108890147687b40f5491aec9ab40ccd2644 | |
| parent | de0673cdf064a102241365835673b56e10198a3e (diff) | |
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>
| -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) | |||
| 677 | return 0; | 677 | return 0; |
| 678 | 678 | ||
| 679 | out_free_dma: | 679 | out_free_dma: |
| 680 | if (ssp->dmach) | 680 | dma_release_channel(ssp->dmach); |
| 681 | dma_release_channel(ssp->dmach); | ||
| 682 | out_clk_disable: | 681 | out_clk_disable: |
| 683 | clk_disable_unprepare(ssp->clk); | 682 | clk_disable_unprepare(ssp->clk); |
| 684 | out_mmc_free: | 683 | out_mmc_free: |