[PATCH V2 11/15] regulator: ti-abb: Convert to use devm_ioremap_resource

From: Tushar Behera
Date: Tue Jun 18 2013 - 00:29:59 EST


Commit 75096579c3ac ("lib: devres: Introduce devm_ioremap_resource()")
introduced devm_ioremap_resource() and deprecated the use of
devm_request_and_ioremap().

While at it, remove the error message as devm_ioremap_resource prints a similar
error message.

Signed-off-by: Tushar Behera <tushar.behera@xxxxxxxxxx>
CC: Mark Brown <broonie@xxxxxxxxxx>
CC: Liam Girdwood <lgirdwood@xxxxxxxxx>
---
Changes for V2:
* Removed error messages from devm_ioremap_resource exit path

drivers/regulator/ti-abb-regulator.c | 14 ++++++--------
1 file changed, 6 insertions(+), 8 deletions(-)

diff --git a/drivers/regulator/ti-abb-regulator.c b/drivers/regulator/ti-abb-regulator.c
index 870d209..3753ed0 100644
--- a/drivers/regulator/ti-abb-regulator.c
+++ b/drivers/regulator/ti-abb-regulator.c
@@ -722,10 +722,9 @@ static int ti_abb_probe(struct platform_device *pdev)
ret = -ENODEV;
goto err;
}
- abb->base = devm_request_and_ioremap(dev, res);
- if (!abb->base) {
- dev_err(dev, "Unable to map '%s'\n", pname);
- ret = -ENOMEM;
+ abb->base = devm_ioremap_resource(dev, res);
+ if (IS_ERR(abb->base)) {
+ ret = PTR_ERR(abb->base);
goto err;
}

@@ -776,10 +775,9 @@ static int ti_abb_probe(struct platform_device *pdev)
ret = -ENODEV;
goto skip_opt;
}
- abb->ldo_base = devm_request_and_ioremap(dev, res);
- if (!abb->ldo_base) {
- dev_err(dev, "Unable to map '%s'\n", pname);
- ret = -ENOMEM;
+ abb->ldo_base = devm_ioremap_resource(dev, res);
+ if (IS_ERR(abb->ldo_base)) {
+ ret = PTR_ERR(abb->ldo_base);
goto err;
}

--
1.7.9.5

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/