[v1,1/1] gpio: visconti: Replace of_node_to_fwnode() with more suitable API

Commit Message

Andy Shevchenko Aug. 22, 2024, 10:58 p.m. UTC

of_node_to_fwnode() is a IRQ domain specific implementation of
of_fwnode_handle(). Replace the former with more suitable API.

Signed-off-by: Andy Shevchenko <[e-mail]>
---
 drivers/gpio/gpio-visconti.c | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

Comments

Bartosz Golaszewski Sept. 2, 2024, 10:18 a.m. UTC | #1

From: Bartosz Golaszewski <[e-mail]>


On Fri, 23 Aug 2024 01:58:18 +0300, Andy Shevchenko wrote:
> of_node_to_fwnode() is a IRQ domain specific implementation of
> of_fwnode_handle(). Replace the former with more suitable API.
> 
> 

Applied, thanks!

[1/1] gpio: visconti: Replace of_node_to_fwnode() with more suitable API
      commit: a5874de9f214a9c603468927adcb7b51fd739862

Best regards,

Patch

diff --git a/drivers/gpio/gpio-visconti.c b/drivers/gpio/gpio-visconti.c
index 6734e7e1e2a4..ebc71ecdb6cf 100644
--- a/drivers/gpio/gpio-visconti.c
+++ b/drivers/gpio/gpio-visconti.c
@@ -8,6 +8,7 @@ 
  * Nobuhiro Iwamatsu <[e-mail]>
  */
 
+#include <linux/bitops.h>
 #include <linux/gpio/driver.h>
 #include <linux/init.h>
 #include <linux/interrupt.h>
@@ -15,8 +16,8 @@ 
 #include <linux/io.h>
 #include <linux/of_irq.h>
 #include <linux/platform_device.h>
+#include <linux/property.h>
 #include <linux/seq_file.h>
-#include <linux/bitops.h>
 
 /* register offset */
 #define GPIO_DIR	0x00
@@ -202,7 +203,7 @@  static int visconti_gpio_probe(struct platform_device *pdev)
 
 	girq = &priv->gpio_chip.irq;
 	gpio_irq_chip_set_chip(girq, &visconti_gpio_irq_chip);
-	girq->fwnode = of_node_to_fwnode(dev->of_node);
+	girq->fwnode = dev_fwnode(dev);
 	girq->parent_domain = parent;
 	girq->child_to_parent_hwirq = visconti_gpio_child_to_parent_hwirq;
 	girq->populate_parent_alloc_arg = visconti_gpio_populate_parent_fwspec;