diff mbox series

pinctrl: renesas: core: Drop unneeded #ifdef CONFIG_OF

Message ID 55d72ce46b43ec2f41681cb5ba7ca7fcebdb98d1.1679416005.git.geert+renesas@glider.be
State New
Headers show
Series pinctrl: renesas: core: Drop unneeded #ifdef CONFIG_OF | expand

Commit Message

Geert Uytterhoeven March 21, 2023, 4:28 p.m. UTC
As the of_node member of struct device always exists, and there is a
dummy of of_device_get_match_data() for the !CONFIG_OF case, there is no
longer a need to protect code using these interfaces with an #ifdef.

Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
---
To be queued in renesas-pinctrl-for-v6.4.
---
 drivers/pinctrl/renesas/core.c | 2 --
 1 file changed, 2 deletions(-)

Comments

Wolfram Sang March 21, 2023, 5:04 p.m. UTC | #1
On Tue, Mar 21, 2023 at 05:28:00PM +0100, Geert Uytterhoeven wrote:
> As the of_node member of struct device always exists, and there is a
> dummy of of_device_get_match_data() for the !CONFIG_OF case, there is no
> longer a need to protect code using these interfaces with an #ifdef.
> 
> Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>

Reviewed-by: Wolfram Sang <wsa+renesas@sang-engineering.com>
diff mbox series

Patch

diff --git a/drivers/pinctrl/renesas/core.c b/drivers/pinctrl/renesas/core.c
index 52365b4e3faf31cc..2c8ca3d59612eaf5 100644
--- a/drivers/pinctrl/renesas/core.c
+++ b/drivers/pinctrl/renesas/core.c
@@ -1364,11 +1364,9 @@  static int sh_pfc_probe(struct platform_device *pdev)
 	struct sh_pfc *pfc;
 	int ret;
 
-#ifdef CONFIG_OF
 	if (pdev->dev.of_node)
 		info = of_device_get_match_data(&pdev->dev);
 	else
-#endif
 		info = (const void *)platform_get_device_id(pdev)->driver_data;
 
 	pfc = devm_kzalloc(&pdev->dev, sizeof(*pfc), GFP_KERNEL);