diff options
author | Axel Lin <axel.lin@ingics.com> | 2016-04-27 04:19:26 -0400 |
---|---|---|
committer | Mark Brown <broonie@kernel.org> | 2016-04-29 07:34:20 -0400 |
commit | 866e48b0994c6338e6aa4ae7c413b5888ab432d1 (patch) | |
tree | 32fab7a1b816e9d5d41d3415507af8e2db680abb | |
parent | 2452ee25255af95d122ff66ea390facb67a61fc3 (diff) |
spi: pic32: Fix checking return value of devm_ioremap_resource
devm_ioremap_resource() returns ERR_PTR on error.
Also remove the redundant dev_err message, the implementation of
devm_ioremap_resource() already print error messages on error paths.
Signed-off-by: Axel Lin <axel.lin@ingics.com>
Signed-off-by: Mark Brown <broonie@kernel.org>
-rw-r--r-- | drivers/spi/spi-pic32.c | 7 |
1 files changed, 3 insertions, 4 deletions
diff --git a/drivers/spi/spi-pic32.c b/drivers/spi/spi-pic32.c index 4eeb8a85b030..73db87f805a1 100644 --- a/drivers/spi/spi-pic32.c +++ b/drivers/spi/spi-pic32.c | |||
@@ -711,10 +711,9 @@ static int pic32_spi_hw_probe(struct platform_device *pdev, | |||
711 | 711 | ||
712 | mem = platform_get_resource(pdev, IORESOURCE_MEM, 0); | 712 | mem = platform_get_resource(pdev, IORESOURCE_MEM, 0); |
713 | pic32s->regs = devm_ioremap_resource(&pdev->dev, mem); | 713 | pic32s->regs = devm_ioremap_resource(&pdev->dev, mem); |
714 | if (!pic32s->regs) { | 714 | if (IS_ERR(pic32s->regs)) |
715 | dev_err(&pdev->dev, "ioremap() failed\n"); | 715 | return PTR_ERR(pic32s->regs); |
716 | return -ENOMEM; | 716 | |
717 | } | ||
718 | pic32s->dma_base = mem->start; | 717 | pic32s->dma_base = mem->start; |
719 | 718 | ||
720 | /* get irq resources: err-irq, rx-irq, tx-irq */ | 719 | /* get irq resources: err-irq, rx-irq, tx-irq */ |