diff options
author | Thierry Reding <thierry.reding@avionic-design.de> | 2013-01-21 05:08:59 -0500 |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2013-01-22 14:41:56 -0500 |
commit | c7c9e1c372452688f0ef2af794789bbd00f9fa51 (patch) | |
tree | 24f46d6b8a6393db716fda9fd94916c2faafd427 /drivers/char | |
parent | 75096579c3ac39ddc2f8b0d9a8924eba31f4d920 (diff) |
char: 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.
Signed-off-by: Thierry Reding <thierry.reding@avionic-design.de>
Cc: Arnd Bergmann <arnd@arndb.de>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'drivers/char')
-rw-r--r-- | drivers/char/hw_random/exynos-rng.c | 9 | ||||
-rw-r--r-- | drivers/char/hw_random/omap-rng.c | 6 | ||||
-rw-r--r-- | drivers/char/hw_random/tx4939-rng.c | 7 |
3 files changed, 12 insertions, 10 deletions
diff --git a/drivers/char/hw_random/exynos-rng.c b/drivers/char/hw_random/exynos-rng.c index 48bbfeca4b5d..4673fc4ad931 100644 --- a/drivers/char/hw_random/exynos-rng.c +++ b/drivers/char/hw_random/exynos-rng.c | |||
@@ -104,6 +104,7 @@ static int exynos_read(struct hwrng *rng, void *buf, | |||
104 | static int exynos_rng_probe(struct platform_device *pdev) | 104 | static int exynos_rng_probe(struct platform_device *pdev) |
105 | { | 105 | { |
106 | struct exynos_rng *exynos_rng; | 106 | struct exynos_rng *exynos_rng; |
107 | struct resource *res; | ||
107 | 108 | ||
108 | exynos_rng = devm_kzalloc(&pdev->dev, sizeof(struct exynos_rng), | 109 | exynos_rng = devm_kzalloc(&pdev->dev, sizeof(struct exynos_rng), |
109 | GFP_KERNEL); | 110 | GFP_KERNEL); |
@@ -120,10 +121,10 @@ static int exynos_rng_probe(struct platform_device *pdev) | |||
120 | return -ENOENT; | 121 | return -ENOENT; |
121 | } | 122 | } |
122 | 123 | ||
123 | exynos_rng->mem = devm_request_and_ioremap(&pdev->dev, | 124 | res = platform_get_resource(pdev, IORESOURCE_MEM, 0); |
124 | platform_get_resource(pdev, IORESOURCE_MEM, 0)); | 125 | exynos_rng->mem = devm_ioremap_resource(&pdev->dev, res); |
125 | if (!exynos_rng->mem) | 126 | if (IS_ERR(exynos_rng->mem)) |
126 | return -EBUSY; | 127 | return PTR_ERR(exynos_rng->mem); |
127 | 128 | ||
128 | platform_set_drvdata(pdev, exynos_rng); | 129 | platform_set_drvdata(pdev, exynos_rng); |
129 | 130 | ||
diff --git a/drivers/char/hw_random/omap-rng.c b/drivers/char/hw_random/omap-rng.c index d8c54e253761..749dc16ca2cc 100644 --- a/drivers/char/hw_random/omap-rng.c +++ b/drivers/char/hw_random/omap-rng.c | |||
@@ -124,9 +124,9 @@ static int omap_rng_probe(struct platform_device *pdev) | |||
124 | goto err_ioremap; | 124 | goto err_ioremap; |
125 | } | 125 | } |
126 | 126 | ||
127 | priv->base = devm_request_and_ioremap(&pdev->dev, priv->mem_res); | 127 | priv->base = devm_ioremap_resource(&pdev->dev, priv->mem_res); |
128 | if (!priv->base) { | 128 | if (IS_ERR(priv->base)) { |
129 | ret = -ENOMEM; | 129 | ret = PTR_ERR(priv->base); |
130 | goto err_ioremap; | 130 | goto err_ioremap; |
131 | } | 131 | } |
132 | dev_set_drvdata(&pdev->dev, priv); | 132 | dev_set_drvdata(&pdev->dev, priv); |
diff --git a/drivers/char/hw_random/tx4939-rng.c b/drivers/char/hw_random/tx4939-rng.c index de473ef3882b..30991989d65b 100644 --- a/drivers/char/hw_random/tx4939-rng.c +++ b/drivers/char/hw_random/tx4939-rng.c | |||
@@ -7,6 +7,7 @@ | |||
7 | * License. See the file "COPYING" in the main directory of this archive | 7 | * License. See the file "COPYING" in the main directory of this archive |
8 | * for more details. | 8 | * for more details. |
9 | */ | 9 | */ |
10 | #include <linux/err.h> | ||
10 | #include <linux/module.h> | 11 | #include <linux/module.h> |
11 | #include <linux/kernel.h> | 12 | #include <linux/kernel.h> |
12 | #include <linux/init.h> | 13 | #include <linux/init.h> |
@@ -115,9 +116,9 @@ static int __init tx4939_rng_probe(struct platform_device *dev) | |||
115 | rngdev = devm_kzalloc(&dev->dev, sizeof(*rngdev), GFP_KERNEL); | 116 | rngdev = devm_kzalloc(&dev->dev, sizeof(*rngdev), GFP_KERNEL); |
116 | if (!rngdev) | 117 | if (!rngdev) |
117 | return -ENOMEM; | 118 | return -ENOMEM; |
118 | rngdev->base = devm_request_and_ioremap(&dev->dev, r); | 119 | rngdev->base = devm_ioremap_resource(&dev->dev, r); |
119 | if (!rngdev->base) | 120 | if (IS_ERR(rngdev->base)) |
120 | return -EBUSY; | 121 | return PTR_ERR(rngdev->base); |
121 | 122 | ||
122 | rngdev->rng.name = dev_name(&dev->dev); | 123 | rngdev->rng.name = dev_name(&dev->dev); |
123 | rngdev->rng.data_present = tx4939_rng_data_present; | 124 | rngdev->rng.data_present = tx4939_rng_data_present; |