diff options
| author | Andy Shevchenko <andriy.shevchenko@linux.intel.com> | 2013-06-05 08:26:43 -0400 |
|---|---|---|
| committer | Vinod Koul <vinod.koul@intel.com> | 2013-07-05 02:10:42 -0400 |
| commit | 0b95961e03ecee31d6151db79cc0826e702d1e0a (patch) | |
| tree | 1203a668e450bc0b3e98743f678f210832ff6a17 | |
| parent | fed8c45727abd273fd74b3e78b35be4929121334 (diff) | |
dw_dmac: don't check resource with devm_ioremap_resource
devm_ioremap_resource does sanity checks on the given resource. No need to
duplicate this in the driver.
Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
Acked-by: Viresh Kumar <viresh.kumar@linaro.org>
Acked-by: Arnd Bergmann <arnd@arndb.de>
Signed-off-by: Vinod Koul <vinod.koul@intel.com>
| -rw-r--r-- | drivers/dma/dw_dmac.c | 5 |
1 files changed, 1 insertions, 4 deletions
diff --git a/drivers/dma/dw_dmac.c b/drivers/dma/dw_dmac.c index 724083d02b34..2b65ba614e60 100644 --- a/drivers/dma/dw_dmac.c +++ b/drivers/dma/dw_dmac.c | |||
| @@ -1667,14 +1667,11 @@ static int dw_probe(struct platform_device *pdev) | |||
| 1667 | int err; | 1667 | int err; |
| 1668 | int i; | 1668 | int i; |
| 1669 | 1669 | ||
| 1670 | io = platform_get_resource(pdev, IORESOURCE_MEM, 0); | ||
| 1671 | if (!io) | ||
| 1672 | return -EINVAL; | ||
| 1673 | |||
| 1674 | irq = platform_get_irq(pdev, 0); | 1670 | irq = platform_get_irq(pdev, 0); |
| 1675 | if (irq < 0) | 1671 | if (irq < 0) |
| 1676 | return irq; | 1672 | return irq; |
| 1677 | 1673 | ||
| 1674 | io = platform_get_resource(pdev, IORESOURCE_MEM, 0); | ||
| 1678 | regs = devm_ioremap_resource(&pdev->dev, io); | 1675 | regs = devm_ioremap_resource(&pdev->dev, io); |
| 1679 | if (IS_ERR(regs)) | 1676 | if (IS_ERR(regs)) |
| 1680 | return PTR_ERR(regs); | 1677 | return PTR_ERR(regs); |
