mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
* [PATCH 1/6] power: reset: at91-poweroff: Convert to devm_platform_ioremap_resource()
@ 2023-07-04 13:03 Yangtao Li
  2023-07-04 13:03 ` [PATCH 2/6] power: reset: at91-sama5d2_shdwc: " Yangtao Li
                   ` (5 more replies)
  0 siblings, 6 replies; 8+ messages in thread
From: Yangtao Li @ 2023-07-04 13:03 UTC (permalink / raw)
  To: Sebastian Reichel, Nicolas Ferre, Alexandre Belloni, Claudiu Beznea
  Cc: Yangtao Li, linux-pm, linux-arm-kernel, linux-kernel

Use devm_platform_ioremap_resource() to simplify code.

Signed-off-by: Yangtao Li <frank.li@vivo.com>
---
 drivers/power/reset/at91-poweroff.c | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

diff --git a/drivers/power/reset/at91-poweroff.c b/drivers/power/reset/at91-poweroff.c
index 9e74e131c675..dd5399785b69 100644
--- a/drivers/power/reset/at91-poweroff.c
+++ b/drivers/power/reset/at91-poweroff.c
@@ -151,13 +151,11 @@ static void at91_poweroff_dt_set_wakeup_mode(struct platform_device *pdev)
 
 static int __init at91_poweroff_probe(struct platform_device *pdev)
 {
-	struct resource *res;
 	struct device_node *np;
 	u32 ddr_type;
 	int ret;
 
-	res = platform_get_resource(pdev, IORESOURCE_MEM, 0);
-	at91_shdwc.shdwc_base = devm_ioremap_resource(&pdev->dev, res);
+	at91_shdwc.shdwc_base = devm_platform_ioremap_resource(pdev, 0);
 	if (IS_ERR(at91_shdwc.shdwc_base))
 		return PTR_ERR(at91_shdwc.shdwc_base);
 
-- 
2.39.0


^ permalink raw reply	[flat|nested] 8+ messages in thread

* [PATCH 2/6] power: reset: at91-sama5d2_shdwc: Convert to devm_platform_ioremap_resource()
  2023-07-04 13:03 [PATCH 1/6] power: reset: at91-poweroff: Convert to devm_platform_ioremap_resource() Yangtao Li
@ 2023-07-04 13:03 ` Yangtao Li
  2023-07-04 13:03 ` [PATCH 3/6] power: reset: brcm-kona: " Yangtao Li
                   ` (4 subsequent siblings)
  5 siblings, 0 replies; 8+ messages in thread
From: Yangtao Li @ 2023-07-04 13:03 UTC (permalink / raw)
  To: Claudiu Beznea, Sebastian Reichel, Nicolas Ferre, Alexandre Belloni
  Cc: Yangtao Li, linux-pm, linux-arm-kernel, linux-kernel

Use devm_platform_ioremap_resource() to simplify code.

Signed-off-by: Yangtao Li <frank.li@vivo.com>
---
 drivers/power/reset/at91-sama5d2_shdwc.c | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

diff --git a/drivers/power/reset/at91-sama5d2_shdwc.c b/drivers/power/reset/at91-sama5d2_shdwc.c
index d8ecffe72f16..e76b102b57b1 100644
--- a/drivers/power/reset/at91-sama5d2_shdwc.c
+++ b/drivers/power/reset/at91-sama5d2_shdwc.c
@@ -331,7 +331,6 @@ static const struct of_device_id at91_pmc_ids[] = {
 
 static int __init at91_shdwc_probe(struct platform_device *pdev)
 {
-	struct resource *res;
 	const struct of_device_id *match;
 	struct device_node *np;
 	u32 ddr_type;
@@ -349,8 +348,7 @@ static int __init at91_shdwc_probe(struct platform_device *pdev)
 
 	platform_set_drvdata(pdev, at91_shdwc);
 
-	res = platform_get_resource(pdev, IORESOURCE_MEM, 0);
-	at91_shdwc->shdwc_base = devm_ioremap_resource(&pdev->dev, res);
+	at91_shdwc->shdwc_base = devm_platform_ioremap_resource(pdev, 0);
 	if (IS_ERR(at91_shdwc->shdwc_base))
 		return PTR_ERR(at91_shdwc->shdwc_base);
 
-- 
2.39.0


^ permalink raw reply	[flat|nested] 8+ messages in thread

* [PATCH 3/6] power: reset: brcm-kona: Convert to devm_platform_ioremap_resource()
  2023-07-04 13:03 [PATCH 1/6] power: reset: at91-poweroff: Convert to devm_platform_ioremap_resource() Yangtao Li
  2023-07-04 13:03 ` [PATCH 2/6] power: reset: at91-sama5d2_shdwc: " Yangtao Li
@ 2023-07-04 13:03 ` Yangtao Li
  2023-07-04 14:48   ` Florian Fainelli
  2023-07-04 13:03 ` [PATCH 4/6] power: reset: gemini-poweroff: " Yangtao Li
                   ` (3 subsequent siblings)
  5 siblings, 1 reply; 8+ messages in thread
From: Yangtao Li @ 2023-07-04 13:03 UTC (permalink / raw)
  To: Sebastian Reichel, Florian Fainelli, Ray Jui, Scott Branden,
	Broadcom internal kernel review list
  Cc: Yangtao Li, linux-pm, linux-kernel

Use devm_platform_ioremap_resource() to simplify code.

Signed-off-by: Yangtao Li <frank.li@vivo.com>
---
 drivers/power/reset/brcm-kona-reset.c | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

diff --git a/drivers/power/reset/brcm-kona-reset.c b/drivers/power/reset/brcm-kona-reset.c
index 3de024e3ceb7..6444f6154761 100644
--- a/drivers/power/reset/brcm-kona-reset.c
+++ b/drivers/power/reset/brcm-kona-reset.c
@@ -38,9 +38,7 @@ static struct notifier_block kona_reset_nb = {
 
 static int kona_reset_probe(struct platform_device *pdev)
 {
-	struct resource *res = platform_get_resource(pdev, IORESOURCE_MEM, 0);
-
-	kona_reset_base = devm_ioremap_resource(&pdev->dev, res);
+	kona_reset_base = devm_platform_ioremap_resource(pdev, 0);
 	if (IS_ERR(kona_reset_base))
 		return PTR_ERR(kona_reset_base);
 
-- 
2.39.0


^ permalink raw reply	[flat|nested] 8+ messages in thread

* [PATCH 4/6] power: reset: gemini-poweroff: Convert to devm_platform_ioremap_resource()
  2023-07-04 13:03 [PATCH 1/6] power: reset: at91-poweroff: Convert to devm_platform_ioremap_resource() Yangtao Li
  2023-07-04 13:03 ` [PATCH 2/6] power: reset: at91-sama5d2_shdwc: " Yangtao Li
  2023-07-04 13:03 ` [PATCH 3/6] power: reset: brcm-kona: " Yangtao Li
@ 2023-07-04 13:03 ` Yangtao Li
  2023-07-04 13:03 ` [PATCH 5/6] power: reset: msm: " Yangtao Li
                   ` (2 subsequent siblings)
  5 siblings, 0 replies; 8+ messages in thread
From: Yangtao Li @ 2023-07-04 13:03 UTC (permalink / raw)
  To: Sebastian Reichel; +Cc: Yangtao Li, linux-pm, linux-kernel

Use devm_platform_ioremap_resource() to simplify code.

Signed-off-by: Yangtao Li <frank.li@vivo.com>
---
 drivers/power/reset/gemini-poweroff.c | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

diff --git a/drivers/power/reset/gemini-poweroff.c b/drivers/power/reset/gemini-poweroff.c
index b7f7a8225f22..d309b610142c 100644
--- a/drivers/power/reset/gemini-poweroff.c
+++ b/drivers/power/reset/gemini-poweroff.c
@@ -91,7 +91,6 @@ static void gemini_poweroff(void)
 static int gemini_poweroff_probe(struct platform_device *pdev)
 {
 	struct device *dev = &pdev->dev;
-	struct resource *res;
 	struct gemini_powercon *gpw;
 	u32 val;
 	int irq;
@@ -101,8 +100,7 @@ static int gemini_poweroff_probe(struct platform_device *pdev)
 	if (!gpw)
 		return -ENOMEM;
 
-	res = platform_get_resource(pdev, IORESOURCE_MEM, 0);
-	gpw->base = devm_ioremap_resource(dev, res);
+	gpw->base = devm_platform_ioremap_resource(pdev, 0);
 	if (IS_ERR(gpw->base))
 		return PTR_ERR(gpw->base);
 
-- 
2.39.0


^ permalink raw reply	[flat|nested] 8+ messages in thread

* [PATCH 5/6] power: reset: msm: Convert to devm_platform_ioremap_resource()
  2023-07-04 13:03 [PATCH 1/6] power: reset: at91-poweroff: Convert to devm_platform_ioremap_resource() Yangtao Li
                   ` (2 preceding siblings ...)
  2023-07-04 13:03 ` [PATCH 4/6] power: reset: gemini-poweroff: " Yangtao Li
@ 2023-07-04 13:03 ` Yangtao Li
  2023-07-04 13:03 ` [PATCH 6/6] power: reset: ocelot: " Yangtao Li
  2023-07-19 21:37 ` [PATCH 1/6] power: reset: at91-poweroff: " Sebastian Reichel
  5 siblings, 0 replies; 8+ messages in thread
From: Yangtao Li @ 2023-07-04 13:03 UTC (permalink / raw)
  To: Andy Gross, Bjorn Andersson, Konrad Dybcio, Sebastian Reichel
  Cc: Yangtao Li, linux-arm-msm, linux-pm, linux-kernel

Use devm_platform_ioremap_resource() to simplify code.

Signed-off-by: Yangtao Li <frank.li@vivo.com>
---
 drivers/power/reset/msm-poweroff.c | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

diff --git a/drivers/power/reset/msm-poweroff.c b/drivers/power/reset/msm-poweroff.c
index 0c439f83bf65..5921acc5c51d 100644
--- a/drivers/power/reset/msm-poweroff.c
+++ b/drivers/power/reset/msm-poweroff.c
@@ -36,10 +36,8 @@ static void do_msm_poweroff(void)
 static int msm_restart_probe(struct platform_device *pdev)
 {
 	struct device *dev = &pdev->dev;
-	struct resource *mem;
 
-	mem = platform_get_resource(pdev, IORESOURCE_MEM, 0);
-	msm_ps_hold = devm_ioremap_resource(dev, mem);
+	msm_ps_hold = devm_platform_ioremap_resource(pdev, 0);
 	if (IS_ERR(msm_ps_hold))
 		return PTR_ERR(msm_ps_hold);
 
-- 
2.39.0


^ permalink raw reply	[flat|nested] 8+ messages in thread

* [PATCH 6/6] power: reset: ocelot: Convert to devm_platform_ioremap_resource()
  2023-07-04 13:03 [PATCH 1/6] power: reset: at91-poweroff: Convert to devm_platform_ioremap_resource() Yangtao Li
                   ` (3 preceding siblings ...)
  2023-07-04 13:03 ` [PATCH 5/6] power: reset: msm: " Yangtao Li
@ 2023-07-04 13:03 ` Yangtao Li
  2023-07-19 21:37 ` [PATCH 1/6] power: reset: at91-poweroff: " Sebastian Reichel
  5 siblings, 0 replies; 8+ messages in thread
From: Yangtao Li @ 2023-07-04 13:03 UTC (permalink / raw)
  To: Sebastian Reichel; +Cc: Yangtao Li, linux-pm, linux-kernel

Use devm_platform_ioremap_resource() to simplify code.

Signed-off-by: Yangtao Li <frank.li@vivo.com>
---
 drivers/power/reset/ocelot-reset.c | 5 +----
 1 file changed, 1 insertion(+), 4 deletions(-)

diff --git a/drivers/power/reset/ocelot-reset.c b/drivers/power/reset/ocelot-reset.c
index 8caa90cb58fc..031772a60b1a 100644
--- a/drivers/power/reset/ocelot-reset.c
+++ b/drivers/power/reset/ocelot-reset.c
@@ -69,8 +69,6 @@ static int ocelot_restart_handle(struct notifier_block *this,
 static int ocelot_reset_probe(struct platform_device *pdev)
 {
 	struct ocelot_reset_context *ctx;
-	struct resource *res;
-
 	struct device *dev = &pdev->dev;
 	int err;
 
@@ -78,8 +76,7 @@ static int ocelot_reset_probe(struct platform_device *pdev)
 	if (!ctx)
 		return -ENOMEM;
 
-	res = platform_get_resource(pdev, IORESOURCE_MEM, 0);
-	ctx->base = devm_ioremap_resource(dev, res);
+	ctx->base = devm_platform_ioremap_resource(pdev, 0);
 	if (IS_ERR(ctx->base))
 		return PTR_ERR(ctx->base);
 
-- 
2.39.0


^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [PATCH 3/6] power: reset: brcm-kona: Convert to devm_platform_ioremap_resource()
  2023-07-04 13:03 ` [PATCH 3/6] power: reset: brcm-kona: " Yangtao Li
@ 2023-07-04 14:48   ` Florian Fainelli
  0 siblings, 0 replies; 8+ messages in thread
From: Florian Fainelli @ 2023-07-04 14:48 UTC (permalink / raw)
  To: Yangtao Li, Sebastian Reichel, Ray Jui, Scott Branden,
	Broadcom internal kernel review list
  Cc: linux-pm, linux-kernel

[-- Attachment #1: Type: text/plain, Size: 221 bytes --]



On 7/4/2023 3:03 PM, Yangtao Li wrote:
> Use devm_platform_ioremap_resource() to simplify code.
> 
> Signed-off-by: Yangtao Li <frank.li@vivo.com>

Acked-by: Florian Fainelli <florian.fainelli@broadcom.com>
-- 
Florian

[-- Attachment #2: S/MIME Cryptographic Signature --]
[-- Type: application/pkcs7-signature, Size: 4221 bytes --]

^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [PATCH 1/6] power: reset: at91-poweroff: Convert to devm_platform_ioremap_resource()
  2023-07-04 13:03 [PATCH 1/6] power: reset: at91-poweroff: Convert to devm_platform_ioremap_resource() Yangtao Li
                   ` (4 preceding siblings ...)
  2023-07-04 13:03 ` [PATCH 6/6] power: reset: ocelot: " Yangtao Li
@ 2023-07-19 21:37 ` Sebastian Reichel
  5 siblings, 0 replies; 8+ messages in thread
From: Sebastian Reichel @ 2023-07-19 21:37 UTC (permalink / raw)
  To: Yangtao Li
  Cc: Nicolas Ferre, Alexandre Belloni, Claudiu Beznea, linux-pm,
	linux-arm-kernel, linux-kernel

[-- Attachment #1: Type: text/plain, Size: 1149 bytes --]

Hi,

On Tue, Jul 04, 2023 at 09:03:03PM +0800, Yangtao Li wrote:
> Use devm_platform_ioremap_resource() to simplify code.
> 
> Signed-off-by: Yangtao Li <frank.li@vivo.com>
> ---

Thanks, whole series queued.

-- Sebastian

>  drivers/power/reset/at91-poweroff.c | 4 +---
>  1 file changed, 1 insertion(+), 3 deletions(-)
> 
> diff --git a/drivers/power/reset/at91-poweroff.c b/drivers/power/reset/at91-poweroff.c
> index 9e74e131c675..dd5399785b69 100644
> --- a/drivers/power/reset/at91-poweroff.c
> +++ b/drivers/power/reset/at91-poweroff.c
> @@ -151,13 +151,11 @@ static void at91_poweroff_dt_set_wakeup_mode(struct platform_device *pdev)
>  
>  static int __init at91_poweroff_probe(struct platform_device *pdev)
>  {
> -	struct resource *res;
>  	struct device_node *np;
>  	u32 ddr_type;
>  	int ret;
>  
> -	res = platform_get_resource(pdev, IORESOURCE_MEM, 0);
> -	at91_shdwc.shdwc_base = devm_ioremap_resource(&pdev->dev, res);
> +	at91_shdwc.shdwc_base = devm_platform_ioremap_resource(pdev, 0);
>  	if (IS_ERR(at91_shdwc.shdwc_base))
>  		return PTR_ERR(at91_shdwc.shdwc_base);
>  
> -- 
> 2.39.0
> 

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]

^ permalink raw reply	[flat|nested] 8+ messages in thread

end of thread, other threads:[~2023-07-19 21:38 UTC | newest]

Thread overview: 8+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2023-07-04 13:03 [PATCH 1/6] power: reset: at91-poweroff: Convert to devm_platform_ioremap_resource() Yangtao Li
2023-07-04 13:03 ` [PATCH 2/6] power: reset: at91-sama5d2_shdwc: " Yangtao Li
2023-07-04 13:03 ` [PATCH 3/6] power: reset: brcm-kona: " Yangtao Li
2023-07-04 14:48   ` Florian Fainelli
2023-07-04 13:03 ` [PATCH 4/6] power: reset: gemini-poweroff: " Yangtao Li
2023-07-04 13:03 ` [PATCH 5/6] power: reset: msm: " Yangtao Li
2023-07-04 13:03 ` [PATCH 6/6] power: reset: ocelot: " Yangtao Li
2023-07-19 21:37 ` [PATCH 1/6] power: reset: at91-poweroff: " Sebastian Reichel

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox

all inboxes | Powered by JetHome®