diff options
author | Thierry Reding <thierry.reding@avionic-design.de> | 2013-01-21 05:09:03 -0500 |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2013-01-24 16:33:23 -0500 |
commit | 84dbf809fbae0591d319a7ea76e6032ff434824c (patch) | |
tree | ab744305093c72ec795d8f873229e5d9fbff0316 /drivers/i2c/busses/i2c-rcar.c | |
parent | f4a18312f46a6c6e0ba7b81776b01fc36edea9fc (diff) |
i2c: Convert to devm_ioremap_resource()
Convert all uses of devm_request_and_ioremap() to the newly introduced
devm_ioremap_resource() which provides more consistent error handling.
devm_ioremap_resource() provides its own error messages so all explicit
error messages can be removed from the failure code paths.
Signed-off-by: Thierry Reding <thierry.reding@avionic-design.de>
Acked-by: Wolfram Sang <w.sang@pengutronix.de>
Cc: Ben Dooks <ben-linux@fluff.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'drivers/i2c/busses/i2c-rcar.c')
-rw-r--r-- | drivers/i2c/busses/i2c-rcar.c | 8 |
1 files changed, 3 insertions, 5 deletions
diff --git a/drivers/i2c/busses/i2c-rcar.c b/drivers/i2c/busses/i2c-rcar.c index 9bd4d73d29e3..4ba4a95b6b26 100644 --- a/drivers/i2c/busses/i2c-rcar.c +++ b/drivers/i2c/busses/i2c-rcar.c | |||
@@ -642,11 +642,9 @@ static int rcar_i2c_probe(struct platform_device *pdev) | |||
642 | if (ret < 0) | 642 | if (ret < 0) |
643 | return ret; | 643 | return ret; |
644 | 644 | ||
645 | priv->io = devm_request_and_ioremap(dev, res); | 645 | priv->io = devm_ioremap_resource(dev, res); |
646 | if (!priv->io) { | 646 | if (IS_ERR(priv->io)) |
647 | dev_err(dev, "cannot ioremap\n"); | 647 | return PTR_ERR(priv->io); |
648 | return -ENODEV; | ||
649 | } | ||
650 | 648 | ||
651 | priv->irq = platform_get_irq(pdev, 0); | 649 | priv->irq = platform_get_irq(pdev, 0); |
652 | init_waitqueue_head(&priv->wait); | 650 | init_waitqueue_head(&priv->wait); |