* [PATCH 2/4] crypto: sahara - Use devm_ioremap_resource()
2014-02-12 4:22 [PATCH 1/4] crypto: picoxcell - Use devm_ioremap_resource() Jingoo Han
@ 2014-02-12 4:23 ` Jingoo Han
2014-02-12 4:24 ` [PATCH 3/4] crypto: s5p-sss " Jingoo Han
` (3 subsequent siblings)
4 siblings, 0 replies; 7+ messages in thread
From: Jingoo Han @ 2014-02-12 4:23 UTC (permalink / raw)
To: 'Herbert Xu'
Cc: 'David Miller', linux-crypto, 'Jingoo Han',
'Javier Martin'
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>
---
drivers/crypto/sahara.c | 19 +++----------------
1 file changed, 3 insertions(+), 16 deletions(-)
diff --git a/drivers/crypto/sahara.c b/drivers/crypto/sahara.c
index 785a9de..894468f 100644
--- a/drivers/crypto/sahara.c
+++ b/drivers/crypto/sahara.c
@@ -885,22 +885,9 @@ static int sahara_probe(struct platform_device *pdev)
/* Get the base address */
res = platform_get_resource(pdev, IORESOURCE_MEM, 0);
- if (!res) {
- dev_err(&pdev->dev, "failed to get memory region resource\n");
- return -ENODEV;
- }
-
- if (devm_request_mem_region(&pdev->dev, res->start,
- resource_size(res), SAHARA_NAME) == NULL) {
- dev_err(&pdev->dev, "failed to request memory region\n");
- return -ENOENT;
- }
- dev->regs_base = devm_ioremap(&pdev->dev, res->start,
- resource_size(res));
- if (!dev->regs_base) {
- dev_err(&pdev->dev, "failed to ioremap address region\n");
- return -ENOENT;
- }
+ dev->regs_base = devm_ioremap_resource(&pdev->dev, res);
+ if (IS_ERR(dev->regs_base))
+ return PTR_ERR(dev->regs_base);
/* Get the IRQ */
irq = platform_get_irq(pdev, 0);
--
1.7.10.4
^ permalink raw reply related [flat|nested] 7+ messages in thread* [PATCH 3/4] crypto: s5p-sss - Use devm_ioremap_resource()
2014-02-12 4:22 [PATCH 1/4] crypto: picoxcell - Use devm_ioremap_resource() Jingoo Han
2014-02-12 4:23 ` [PATCH 2/4] crypto: sahara " Jingoo Han
@ 2014-02-12 4:24 ` Jingoo Han
2014-02-12 4:26 ` [PATCH 4/4] crypto: tegra-aes " Jingoo Han
` (2 subsequent siblings)
4 siblings, 0 replies; 7+ messages in thread
From: Jingoo Han @ 2014-02-12 4:24 UTC (permalink / raw)
To: 'Herbert Xu'
Cc: 'David Miller', linux-crypto, 'Jingoo Han',
'Vladimir Zapolskiy'
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>
---
drivers/crypto/s5p-sss.c | 13 ++++---------
1 file changed, 4 insertions(+), 9 deletions(-)
diff --git a/drivers/crypto/s5p-sss.c b/drivers/crypto/s5p-sss.c
index cf149b1..be45762 100644
--- a/drivers/crypto/s5p-sss.c
+++ b/drivers/crypto/s5p-sss.c
@@ -568,17 +568,14 @@ static int s5p_aes_probe(struct platform_device *pdev)
if (s5p_dev)
return -EEXIST;
- res = platform_get_resource(pdev, IORESOURCE_MEM, 0);
- if (!res)
- return -ENODEV;
-
pdata = devm_kzalloc(dev, sizeof(*pdata), GFP_KERNEL);
if (!pdata)
return -ENOMEM;
- if (!devm_request_mem_region(dev, res->start,
- resource_size(res), pdev->name))
- return -EBUSY;
+ res = platform_get_resource(pdev, IORESOURCE_MEM, 0);
+ pdata->ioaddr = devm_ioremap_resource(&pdev->dev, res);
+ if (IS_ERR(pdata->ioaddr))
+ return PTR_ERR(pdata->ioaddr);
pdata->clk = devm_clk_get(dev, "secss");
if (IS_ERR(pdata->clk)) {
@@ -589,8 +586,6 @@ static int s5p_aes_probe(struct platform_device *pdev)
clk_enable(pdata->clk);
spin_lock_init(&pdata->lock);
- pdata->ioaddr = devm_ioremap(dev, res->start,
- resource_size(res));
pdata->irq_hash = platform_get_irq_byname(pdev, "hash");
if (pdata->irq_hash < 0) {
--
1.7.10.4
^ permalink raw reply related [flat|nested] 7+ messages in thread* [PATCH 4/4] crypto: tegra-aes - Use devm_ioremap_resource()
2014-02-12 4:22 [PATCH 1/4] crypto: picoxcell - Use devm_ioremap_resource() Jingoo Han
2014-02-12 4:23 ` [PATCH 2/4] crypto: sahara " Jingoo Han
2014-02-12 4:24 ` [PATCH 3/4] crypto: s5p-sss " Jingoo Han
@ 2014-02-12 4:26 ` Jingoo Han
2014-02-12 4:31 ` Varun Wadekar
2014-02-12 4:28 ` [PATCH 1/4] crypto: picoxcell " Jingoo Han
2014-02-25 12:17 ` Herbert Xu
4 siblings, 1 reply; 7+ messages in thread
From: Jingoo Han @ 2014-02-12 4:26 UTC (permalink / raw)
To: 'Herbert Xu'
Cc: 'David Miller', linux-crypto, 'Jingoo Han',
'Stephen Warren', 'Thierry Reding',
'Varun Wadekar'
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>
---
drivers/crypto/tegra-aes.c | 20 +++-----------------
1 file changed, 3 insertions(+), 17 deletions(-)
diff --git a/drivers/crypto/tegra-aes.c b/drivers/crypto/tegra-aes.c
index 060eecc..fa3dc14 100644
--- a/drivers/crypto/tegra-aes.c
+++ b/drivers/crypto/tegra-aes.c
@@ -896,23 +896,9 @@ static int tegra_aes_probe(struct platform_device *pdev)
/* Get the module base address */
res = platform_get_resource(pdev, IORESOURCE_MEM, 0);
- if (!res) {
- dev_err(dev, "invalid resource type: base\n");
- err = -ENODEV;
- goto out;
- }
-
- if (!devm_request_mem_region(&pdev->dev, res->start,
- resource_size(res),
- dev_name(&pdev->dev))) {
- dev_err(&pdev->dev, "Couldn't request MEM resource\n");
- return -ENODEV;
- }
-
- dd->io_base = devm_ioremap(dev, res->start, resource_size(res));
- if (!dd->io_base) {
- dev_err(dev, "can't ioremap register space\n");
- err = -ENOMEM;
+ dd->io_base = devm_ioremap_resource(dev, res);
+ if (IS_ERR(dd->io_base)) {
+ err = PTR_ERR(dd->io_base);
goto out;
}
--
1.7.10.4
^ permalink raw reply related [flat|nested] 7+ messages in thread* RE: [PATCH 4/4] crypto: tegra-aes - Use devm_ioremap_resource()
2014-02-12 4:26 ` [PATCH 4/4] crypto: tegra-aes " Jingoo Han
@ 2014-02-12 4:31 ` Varun Wadekar
0 siblings, 0 replies; 7+ messages in thread
From: Varun Wadekar @ 2014-02-12 4:31 UTC (permalink / raw)
To: Jingoo Han, 'Herbert Xu'
Cc: 'David Miller', linux-crypto, 'Stephen Warren',
'Thierry Reding'
Looks good. Please add my ACK.
Thanks.
-----Original Message-----
From: Jingoo Han [mailto:jg1.han@samsung.com]
Sent: Wednesday, February 12, 2014 9:57 AM
To: 'Herbert Xu'
Cc: 'David Miller'; linux-crypto@vger.kernel.org; 'Jingoo Han'; 'Stephen Warren'; 'Thierry Reding'; Varun Wadekar
Subject: [PATCH 4/4] crypto: tegra-aes - 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>
---
drivers/crypto/tegra-aes.c | 20 +++-----------------
1 file changed, 3 insertions(+), 17 deletions(-)
diff --git a/drivers/crypto/tegra-aes.c b/drivers/crypto/tegra-aes.c index 060eecc..fa3dc14 100644
--- a/drivers/crypto/tegra-aes.c
+++ b/drivers/crypto/tegra-aes.c
@@ -896,23 +896,9 @@ static int tegra_aes_probe(struct platform_device *pdev)
/* Get the module base address */
res = platform_get_resource(pdev, IORESOURCE_MEM, 0);
- if (!res) {
- dev_err(dev, "invalid resource type: base\n");
- err = -ENODEV;
- goto out;
- }
-
- if (!devm_request_mem_region(&pdev->dev, res->start,
- resource_size(res),
- dev_name(&pdev->dev))) {
- dev_err(&pdev->dev, "Couldn't request MEM resource\n");
- return -ENODEV;
- }
-
- dd->io_base = devm_ioremap(dev, res->start, resource_size(res));
- if (!dd->io_base) {
- dev_err(dev, "can't ioremap register space\n");
- err = -ENOMEM;
+ dd->io_base = devm_ioremap_resource(dev, res);
+ if (IS_ERR(dd->io_base)) {
+ err = PTR_ERR(dd->io_base);
goto out;
}
--
1.7.10.4
-----------------------------------------------------------------------------------
This email message is for the sole use of the intended recipient(s) and may contain
confidential information. Any unauthorized review, use, disclosure or distribution
is prohibited. If you are not the intended recipient, please contact the sender by
reply email and destroy all copies of the original message.
-----------------------------------------------------------------------------------
^ permalink raw reply [flat|nested] 7+ messages in thread
* [PATCH 1/4] crypto: picoxcell - Use devm_ioremap_resource()
2014-02-12 4:22 [PATCH 1/4] crypto: picoxcell - Use devm_ioremap_resource() Jingoo Han
` (2 preceding siblings ...)
2014-02-12 4:26 ` [PATCH 4/4] crypto: tegra-aes " Jingoo Han
@ 2014-02-12 4:28 ` Jingoo Han
2014-02-25 12:17 ` Herbert Xu
4 siblings, 0 replies; 7+ messages in thread
From: Jingoo Han @ 2014-02-12 4:28 UTC (permalink / raw)
To: 'Herbert Xu'
Cc: 'David Miller', linux-crypto, 'Jingoo Han',
'Mathias Krause', 'Jamie Iles'
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>
---
drivers/crypto/picoxcell_crypto.c | 16 +++++-----------
1 file changed, 5 insertions(+), 11 deletions(-)
diff --git a/drivers/crypto/picoxcell_crypto.c b/drivers/crypto/picoxcell_crypto.c
index a6175ba..5da5b98 100644
--- a/drivers/crypto/picoxcell_crypto.c
+++ b/drivers/crypto/picoxcell_crypto.c
@@ -1720,22 +1720,16 @@ static int spacc_probe(struct platform_device *pdev)
engine->name = dev_name(&pdev->dev);
mem = platform_get_resource(pdev, IORESOURCE_MEM, 0);
+ engine->regs = devm_ioremap_resource(&pdev->dev, mem);
+ if (IS_ERR(engine->regs))
+ return PTR_ERR(engine->regs);
+
irq = platform_get_resource(pdev, IORESOURCE_IRQ, 0);
- if (!mem || !irq) {
+ if (!irq) {
dev_err(&pdev->dev, "no memory/irq resource for engine\n");
return -ENXIO;
}
- if (!devm_request_mem_region(&pdev->dev, mem->start, resource_size(mem),
- engine->name))
- return -ENOMEM;
-
- engine->regs = devm_ioremap(&pdev->dev, mem->start, resource_size(mem));
- if (!engine->regs) {
- dev_err(&pdev->dev, "memory map failed\n");
- return -ENOMEM;
- }
-
if (devm_request_irq(&pdev->dev, irq->start, spacc_spacc_irq, 0,
engine->name, engine)) {
dev_err(engine->dev, "failed to request IRQ\n");
--
1.7.10.4
^ permalink raw reply related [flat|nested] 7+ messages in thread* Re: [PATCH 1/4] crypto: picoxcell - Use devm_ioremap_resource()
2014-02-12 4:22 [PATCH 1/4] crypto: picoxcell - Use devm_ioremap_resource() Jingoo Han
` (3 preceding siblings ...)
2014-02-12 4:28 ` [PATCH 1/4] crypto: picoxcell " Jingoo Han
@ 2014-02-25 12:17 ` Herbert Xu
4 siblings, 0 replies; 7+ messages in thread
From: Herbert Xu @ 2014-02-25 12:17 UTC (permalink / raw)
To: Jingoo Han
Cc: 'David Miller', linux-crypto, 'Mathias Krause',
'Jamie Iles'
On Wed, Feb 12, 2014 at 01:22:06PM +0900, Jingoo Han wrote:
> >From c0c9c31eba90635cac93f3d5e5311beb10afafb7 Mon Sep 17 00:00:00 2001
> From: Jingoo Han <jg1.han@samsung.com>
> Date: Wed, 12 Feb 2014 13:08:54 +0900
> Subject: [PATCH 1/4] crypto: picoxcell - 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>
All applied except for 4/4 since we're removing tegra. Thanks.
--
Email: Herbert Xu <herbert@gondor.apana.org.au>
Home Page: http://gondor.apana.org.au/~herbert/
PGP Key: http://gondor.apana.org.au/~herbert/pubkey.txt
^ permalink raw reply [flat|nested] 7+ messages in thread