diff options
author | Dan Carpenter <dan.carpenter@oracle.com> | 2017-10-03 06:54:50 -0400 |
---|---|---|
committer | Vinod Koul <vinod.koul@intel.com> | 2017-10-08 06:47:39 -0400 |
commit | 4219ff33b26dbaa08c728ee4ad1a3a5aae300e1a (patch) | |
tree | a53631168f12dcbaed2065dc08e1f6ceb2ef9ab1 | |
parent | a4ffb13c8946abc4b92621275de6718e19db860d (diff) |
dmaengine: stm32-dmamux: Fix a NULL vs IS_ERR() check in probe
devm_ioremap_resource() doesn't return NULL, it returns error pointers.
Fixes: df7e762db5f6 ("dmaengine: Add STM32 DMAMUX driver")
Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Acked-by: Pierre-Yves MORDRET <pierre-yves.mordret@st.com>
Signed-off-by: Vinod Koul <vinod.koul@intel.com>
-rw-r--r-- | drivers/dma/stm32-dmamux.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/drivers/dma/stm32-dmamux.c b/drivers/dma/stm32-dmamux.c index 22812e7a953b..d5db0f6e1ff8 100644 --- a/drivers/dma/stm32-dmamux.c +++ b/drivers/dma/stm32-dmamux.c | |||
@@ -257,8 +257,8 @@ static int stm32_dmamux_probe(struct platform_device *pdev) | |||
257 | return -ENODEV; | 257 | return -ENODEV; |
258 | 258 | ||
259 | iomem = devm_ioremap_resource(&pdev->dev, res); | 259 | iomem = devm_ioremap_resource(&pdev->dev, res); |
260 | if (!iomem) | 260 | if (IS_ERR(iomem)) |
261 | return -ENOMEM; | 261 | return PTR_ERR(iomem); |
262 | 262 | ||
263 | spin_lock_init(&stm32_dmamux->lock); | 263 | spin_lock_init(&stm32_dmamux->lock); |
264 | 264 | ||