mail archive of the barebox mailing list
 help / color / mirror / Atom feed
* [PATCH] treewide: drop useless casting to void * in of_device_id
@ 2025-09-09  8:59 Sascha Hauer
  0 siblings, 0 replies; only message in thread
From: Sascha Hauer @ 2025-09-09  8:59 UTC (permalink / raw)
  To: Barebox List

struct of_device_id::data already is a const void *, so drop explicit
casting to void * when initialized with something that already is a
pointer.

Signed-off-by: Sascha Hauer <s.hauer@pengutronix.de>
---
 drivers/hw_random/rockchip-rng.c   |  6 +++---
 drivers/pinctrl/pinctrl-rockchip.c |  2 +-
 drivers/spi/dspi_spi.c             |  2 +-
 drivers/spi/spi-nxp-fspi.c         | 10 +++++-----
 drivers/video/ssd1307fb.c          |  8 ++++----
 5 files changed, 14 insertions(+), 14 deletions(-)

diff --git a/drivers/hw_random/rockchip-rng.c b/drivers/hw_random/rockchip-rng.c
index 62a21db0e2..8856a3883e 100644
--- a/drivers/hw_random/rockchip-rng.c
+++ b/drivers/hw_random/rockchip-rng.c
@@ -187,15 +187,15 @@ static const struct rk_rng_soc_data rk_rng_v2_soc_data = {
 static const struct of_device_id rk_rng_dt_match[] = {
 	{
 		.compatible = "rockchip,rk3399-crypto",
-		.data = (void *)&rk_rng_rk3399_soc_data,
+		.data = &rk_rng_rk3399_soc_data,
 	},
 	{
 		.compatible = "rockchip,cryptov1-rng",
-		.data = (void *)&rk_rng_v1_soc_data,
+		.data = &rk_rng_v1_soc_data,
 	},
 	{
 		.compatible = "rockchip,rk3568-rng",
-		.data = (void *)&rk_rng_v2_soc_data,
+		.data = &rk_rng_v2_soc_data,
 	},
 	{ },
 };
diff --git a/drivers/pinctrl/pinctrl-rockchip.c b/drivers/pinctrl/pinctrl-rockchip.c
index 9c11104513..58e9153bd6 100644
--- a/drivers/pinctrl/pinctrl-rockchip.c
+++ b/drivers/pinctrl/pinctrl-rockchip.c
@@ -3236,7 +3236,7 @@ static const struct of_device_id rockchip_pinctrl_dt_match[] = {
 #endif
 #ifdef CONFIG_ARCH_RK3128
 	{ .compatible = "rockchip,rk3128-pinctrl",
-		.data = (void *)&rk3128_pin_ctrl },
+		.data = &rk3128_pin_ctrl },
 #endif
 #ifdef CONFIG_ARCH_RK3188
 	{ .compatible = "rockchip,rk3188-pinctrl",
diff --git a/drivers/spi/dspi_spi.c b/drivers/spi/dspi_spi.c
index 75addfd12c..356259a814 100644
--- a/drivers/spi/dspi_spi.c
+++ b/drivers/spi/dspi_spi.c
@@ -397,7 +397,7 @@ static const struct fsl_dspi_devtype_data vf610_data = {
 };
 
 static const struct of_device_id dspi_dt_ids[] = {
-	{ .compatible = "fsl,vf610-dspi", .data = (void *)&vf610_data, },
+	{ .compatible = "fsl,vf610-dspi", .data = &vf610_data, },
 	{ /* sentinel */ }
 };
 MODULE_DEVICE_TABLE(of, dspi_dt_ids);
diff --git a/drivers/spi/spi-nxp-fspi.c b/drivers/spi/spi-nxp-fspi.c
index 6f91e75d5e..4937932a36 100644
--- a/drivers/spi/spi-nxp-fspi.c
+++ b/drivers/spi/spi-nxp-fspi.c
@@ -1039,11 +1039,11 @@ static int nxp_fspi_probe(struct device *dev)
 }
 
 static const struct of_device_id nxp_fspi_dt_ids[] = {
-	{ .compatible = "nxp,imx8mm-fspi", .data = (void *)&imx8mm_data, },
-	{ .compatible = "nxp,imx8mn-fspi", .data = (void *)&imx8mm_data, },
-	{ .compatible = "nxp,imx8mp-fspi", .data = (void *)&imx8mm_data, },
-	{ .compatible = "nxp,imx8qxp-fspi", .data = (void *)&imx8qxp_data, },
-	{ .compatible = "nxp,imx8dxl-fspi", .data = (void *)&imx8dxl_data, },
+	{ .compatible = "nxp,imx8mm-fspi", .data = &imx8mm_data, },
+	{ .compatible = "nxp,imx8mn-fspi", .data = &imx8mm_data, },
+	{ .compatible = "nxp,imx8mp-fspi", .data = &imx8mm_data, },
+	{ .compatible = "nxp,imx8qxp-fspi", .data = &imx8qxp_data, },
+	{ .compatible = "nxp,imx8dxl-fspi", .data = &imx8dxl_data, },
 	{ /* sentinel */ }
 };
 MODULE_DEVICE_TABLE(of, nxp_fspi_dt_ids);
diff --git a/drivers/video/ssd1307fb.c b/drivers/video/ssd1307fb.c
index 5880f2b4a5..abb8031636 100644
--- a/drivers/video/ssd1307fb.c
+++ b/drivers/video/ssd1307fb.c
@@ -404,11 +404,11 @@ static struct ssd1307fb_deviceinfo ssd1307fb_ssd1309_deviceinfo = {
 static const struct of_device_id ssd1307fb_of_match[] = {
 	{
 		.compatible = "solomon,ssd1305fb-i2c",
-		.data = (void *)&ssd1307fb_ssd1305_deviceinfo,
+		.data = &ssd1307fb_ssd1305_deviceinfo,
 	},
 	{
 		.compatible = "solomon,ssd1306fb-i2c",
-		.data = (void *)&ssd1307fb_ssd1306_deviceinfo,
+		.data = &ssd1307fb_ssd1306_deviceinfo,
 	},
 	{
 		/*
@@ -416,11 +416,11 @@ static const struct of_device_id ssd1307fb_of_match[] = {
 		 * documented as device tree binding.
 		 */
 		.compatible = "solomon,ssd1306",
-		.data = (void *)&ssd1307fb_ssd1306_deviceinfo,
+		.data = &ssd1307fb_ssd1306_deviceinfo,
 	},
 	{
 		.compatible = "solomon,ssd1309fb-i2c",
-		.data = (void *)&ssd1307fb_ssd1309_deviceinfo,
+		.data = &ssd1307fb_ssd1309_deviceinfo,
 	},
 	{},
 };
-- 
2.47.3




^ permalink raw reply	[flat|nested] only message in thread

only message in thread, other threads:[~2025-09-09 10:20 UTC | newest]

Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2025-09-09  8:59 [PATCH] treewide: drop useless casting to void * in of_device_id Sascha Hauer

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