i2c: mxs: 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> Acked-by: Shawn Guo <shawn.guo@linaro.org> Acked-by: Marek Vasut <marex@denx.de> Signed-off-by: Wolfram Sang <wsa@the-dreams.de>
This commit is contained in:
parent
ae50b1df50
commit
0977f27338
1 changed files with 6 additions and 12 deletions
|
@ -806,7 +806,6 @@ static int mxs_i2c_probe(struct platform_device *pdev)
|
|||
struct mxs_i2c_dev *i2c;
|
||||
struct i2c_adapter *adap;
|
||||
struct resource *res;
|
||||
resource_size_t res_size;
|
||||
int err, irq;
|
||||
|
||||
i2c = devm_kzalloc(dev, sizeof(struct mxs_i2c_dev), GFP_KERNEL);
|
||||
|
@ -819,18 +818,13 @@ static int mxs_i2c_probe(struct platform_device *pdev)
|
|||
}
|
||||
|
||||
res = platform_get_resource(pdev, IORESOURCE_MEM, 0);
|
||||
i2c->regs = devm_ioremap_resource(&pdev->dev, res);
|
||||
if (IS_ERR(i2c->regs))
|
||||
return PTR_ERR(i2c->regs);
|
||||
|
||||
irq = platform_get_irq(pdev, 0);
|
||||
|
||||
if (!res || irq < 0)
|
||||
return -ENOENT;
|
||||
|
||||
res_size = resource_size(res);
|
||||
if (!devm_request_mem_region(dev, res->start, res_size, res->name))
|
||||
return -EBUSY;
|
||||
|
||||
i2c->regs = devm_ioremap_nocache(dev, res->start, res_size);
|
||||
if (!i2c->regs)
|
||||
return -EBUSY;
|
||||
if (irq < 0)
|
||||
return irq;
|
||||
|
||||
err = devm_request_irq(dev, irq, mxs_i2c_isr, 0, dev_name(dev), i2c);
|
||||
if (err)
|
||||
|
|
Loading…
Reference in a new issue