diff options
| author | Jingoo Han <jg1.han@samsung.com> | 2014-01-02 12:24:03 +0400 |
|---|---|---|
| committer | Tejun Heo <tj@kernel.org> | 2014-01-02 16:48:33 +0400 |
| commit | 3e692a942f6eb0ec0151449e6c5fb88ffc854b6d (patch) | |
| tree | cb88e42ba0e6bce0650717acd34a7694e4b2f3ba /drivers/ata | |
| parent | 15e5318c9cb92dc49f1326bce44798575e3b5774 (diff) | |
| download | linux-3e692a942f6eb0ec0151449e6c5fb88ffc854b6d.tar.xz | |
ata: pata_samsung_cf: Use devm_ioremap_resource()
Use devm_ioremap_resource() in order to make the code simpler,
and remove redundant return value check of platform_get_resource()
because the value is checked by devm_ioremap_resource().
Signed-off-by: Jingoo Han <jg1.han@samsung.com>
Signed-off-by: Tejun Heo <tj@kernel.org>
Diffstat (limited to 'drivers/ata')
| -rw-r--r-- | drivers/ata/pata_samsung_cf.c | 18 |
1 files changed, 3 insertions, 15 deletions
diff --git a/drivers/ata/pata_samsung_cf.c b/drivers/ata/pata_samsung_cf.c index cd2746b169a9..7d2142fbcbb7 100644 --- a/drivers/ata/pata_samsung_cf.c +++ b/drivers/ata/pata_samsung_cf.c @@ -531,22 +531,10 @@ static int __init pata_s3c_probe(struct platform_device *pdev) info->irq = platform_get_irq(pdev, 0); res = platform_get_resource(pdev, IORESOURCE_MEM, 0); - if (res == NULL) { - dev_err(dev, "failed to get mem resource\n"); - return -EINVAL; - } - - if (!devm_request_mem_region(dev, res->start, - resource_size(res), DRV_NAME)) { - dev_err(dev, "error requesting register region\n"); - return -EBUSY; - } - info->ide_addr = devm_ioremap(dev, res->start, resource_size(res)); - if (!info->ide_addr) { - dev_err(dev, "failed to map IO base address\n"); - return -ENOMEM; - } + info->ide_addr = devm_ioremap_resource(dev, res); + if (IS_ERR(info->ide_addr)) + return PTR_ERR(info->ide_addr); info->clk = devm_clk_get(&pdev->dev, "cfcon"); if (IS_ERR(info->clk)) { |