[openwrt/openwrt] ramips: 6.1: pinctrl: fix compilation with 6.1

LEDE Commits lede-commits at lists.infradead.org
Sat Feb 10 04:09:14 PST 2024


nick pushed a commit to openwrt/openwrt.git, branch main:
https://git.openwrt.org/3358045d1db3c50f252f19d0c2f0e0ccaae396ae

commit 3358045d1db3c50f252f19d0c2f0e0ccaae396ae
Author: Nick Hainke <vincent at systemli.org>
AuthorDate: Fri May 26 21:01:01 2023 +0200

    ramips: 6.1: pinctrl: fix compilation with 6.1
    
    Upstream changed in ed5c2f5fd10d ("i2c: Make remove callback return void")
    the i2c driver's remove function to return no value. Adapt the driver code
    to compile with 5.15 and 6.1 like it is done in other projects [0].
    
    Fixes errors in the form of:
      make[8]: Leaving directory '/home/nick/openwrt/build_dir/target-mipsel_24kc_musl/linux-ramips_mt7621/linux-6.1.29'
        CC      drivers/pinctrl/pinctrl-aw9523.o
      drivers/pinctrl/pinctrl-aw9523.c:1117:19: error: initialization of 'void (*)(struct i2c_client *)' from incompatible pointer type 'int (*)(struct i2c_client *)' [-Werror=incompatible-pointer-types]
     1117 |         .remove = aw9523_remove,
          |                   ^~~~~~~~~~~~~
      drivers/pinctrl/pinctrl-aw9523.c:1117:19: note: (near initialization for 'aw9523_driver.remove')
      cc1: all warnings being treated as errors
    
    [0] - https://gitlab.com/ddcci-driver-linux/ddcci-driver-linux/-/merge_requests/10/diffs
    
    Signed-off-by: Nick Hainke <vincent at systemli.org>
---
 target/linux/ramips/files/drivers/pinctrl/pinctrl-aw9523.c | 11 +++++++++++
 1 file changed, 11 insertions(+)

diff --git a/target/linux/ramips/files/drivers/pinctrl/pinctrl-aw9523.c b/target/linux/ramips/files/drivers/pinctrl/pinctrl-aw9523.c
index 4900c5a0c0..e151f44590 100644
--- a/target/linux/ramips/files/drivers/pinctrl/pinctrl-aw9523.c
+++ b/target/linux/ramips/files/drivers/pinctrl/pinctrl-aw9523.c
@@ -1097,6 +1097,13 @@ static int aw9523_remove(struct i2c_client *client)
 	return 0;
 }
 
+#if LINUX_VERSION_CODE >= KERNEL_VERSION(6, 1, 0)
+static void aw9523_remove_void(struct i2c_client *client)
+{
+	aw9523_remove(client);
+}
+#endif
+
 static const struct i2c_device_id aw9523_i2c_id_table[] = {
 	{ "aw9523_i2c", 0 },
 	{ }
@@ -1114,7 +1121,11 @@ static struct i2c_driver aw9523_driver = {
 		.of_match_table = of_aw9523_i2c_match,
 	},
 	.probe = aw9523_probe,
+	#if LINUX_VERSION_CODE >= KERNEL_VERSION(6, 1, 0)
+	.remove = aw9523_remove_void,
+	#else
 	.remove = aw9523_remove,
+	#endif
 	.id_table = aw9523_i2c_id_table,
 };
 module_i2c_driver(aw9523_driver);




More information about the lede-commits mailing list