[1/4] pinctrl: berlin: as370: Use devm_platform_get_and_ioremap_resource()

Message ID 20230704124742.9596-1-frank.li@vivo.com
State New
Headers
Series [1/4] pinctrl: berlin: as370: Use devm_platform_get_and_ioremap_resource() |

Commit Message

李扬韬 July 4, 2023, 12:47 p.m. UTC
  Convert platform_get_resource(), devm_ioremap_resource() to a single
call to devm_platform_get_and_ioremap_resource(), as this is exactly
what this function does.

Signed-off-by: Yangtao Li <frank.li@vivo.com>
---
 drivers/pinctrl/berlin/pinctrl-as370.c | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)
  

Comments

Linus Walleij July 16, 2023, 9:09 p.m. UTC | #1
Hi Yangtao,

On Tue, Jul 4, 2023 at 2:48 PM Yangtao Li <frank.li@vivo.com> wrote:

> Convert platform_get_resource(), devm_ioremap_resource() to a single
> call to devm_platform_get_and_ioremap_resource(), as this is exactly
> what this function does.
>
> Signed-off-by: Yangtao Li <frank.li@vivo.com>

All 4 patches applied, nice cleanups!

Yours,
Linus Walleij
  

Patch

diff --git a/drivers/pinctrl/berlin/pinctrl-as370.c b/drivers/pinctrl/berlin/pinctrl-as370.c
index 9dfdc275ee33..b631c14813a7 100644
--- a/drivers/pinctrl/berlin/pinctrl-as370.c
+++ b/drivers/pinctrl/berlin/pinctrl-as370.c
@@ -341,8 +341,7 @@  static int as370_pinctrl_probe(struct platform_device *pdev)
 	if (!rmconfig)
 		return -ENOMEM;
 
-	res = platform_get_resource(pdev, IORESOURCE_MEM, 0);
-	base = devm_ioremap_resource(&pdev->dev, res);
+	base = devm_platform_get_and_ioremap_resource(pdev, 0, &res);
 	if (IS_ERR(base))
 		return PTR_ERR(base);