[Buildroot] [PATCH v2] package/xr819-xradio: add patch to build with Linux >= 5.13

Giulio Benetti giulio.benetti at benettiengineering.com
Tue Jan 18 06:35:31 UTC 2022


As explained in the local patch itself of_get_mac_address() has changed, so
we need to check against Linux version and use it slightly differently.

Patch is pending upstream:
https://github.com/fifteenhex/xradio/pull/15
I've realized only later there was already an opened PR for this, anyway
IMHO I think my patch is written a little better, so please accept it.
And most of all Sergey already tested it and it works:
https://lists.buildroot.org/pipermail/buildroot/2022-January/634084.html

Signed-off-by: Giulio Benetti <giulio.benetti at benettiengineering.com>
---
V1->V2:
* update local patch to build correctly also with Linux >= 5.12.1 < 5.13.0
---
 ...fix-building-with-Linux-version-5.13.patch | 68 +++++++++++++++++++
 1 file changed, 68 insertions(+)
 create mode 100644 package/xr819-xradio/0001-main.c-fix-building-with-Linux-version-5.13.patch

diff --git a/package/xr819-xradio/0001-main.c-fix-building-with-Linux-version-5.13.patch b/package/xr819-xradio/0001-main.c-fix-building-with-Linux-version-5.13.patch
new file mode 100644
index 0000000000..3b9742012e
--- /dev/null
+++ b/package/xr819-xradio/0001-main.c-fix-building-with-Linux-version-5.13.patch
@@ -0,0 +1,68 @@
+From 4873746fa9d42a8edbc1e192899e00c29ed3d32a Mon Sep 17 00:00:00 2001
+From: Giulio Benetti <giulio.benetti at benettiengineering.com>
+Date: Mon, 17 Jan 2022 12:26:00 +0100
+Subject: [PATCH] main.c: fix building with Linux version >= 5.13
+
+of_get_mac_address() on Linux version >= 5.13 requires mac pointer as
+second argument and return an int. So let's deal with it by checking linux
+version to make it compatible with both Linux version >= 5.13 and not.
+
+Signed-off-by: Giulio Benetti <giulio.benetti at benettiengineering.com>
+---
+ main.c | 17 +++++++++++++++++
+ 1 file changed, 17 insertions(+)
+
+diff --git a/main.c b/main.c
+index b60e18d..06cb1f6 100644
+--- a/main.c
++++ b/main.c
+@@ -13,6 +13,7 @@
+ #include <net/cfg80211.h>
+ #include <linux/of_net.h>
+ #include <linux/mmc/sdio_func.h>
++#include <linux/version.h>
+ 
+ #include "xradio.h"
+ #include "fwio.h"
+@@ -499,8 +500,12 @@ int xradio_core_init(struct sdio_func* func)
+ 	int if_id;
+ 	struct ieee80211_hw *dev;
+ 	struct xradio_common *hw_priv;
++#if LINUX_VERSION_CODE >= KERNEL_VERSION(5, 13, 0)
++	unsigned char addr[ETH_ALEN];
++#else
+ 	unsigned char randomaddr[ETH_ALEN];
+ 	const unsigned char *addr = NULL;
++#endif
+ 
+ 	//init xradio_common
+ 	dev = xradio_init_common(sizeof(struct xradio_common));
+@@ -513,6 +518,16 @@ int xradio_core_init(struct sdio_func* func)
+ 	hw_priv->sdio_func = func;
+ 	sdio_set_drvdata(func, hw_priv);
+ 
++#if LINUX_VERSION_CODE >= KERNEL_VERSION(5, 13, 0)
++	// fill in mac addresses
++	if (hw_priv->pdev->of_node) {
++		err = of_get_mac_address(hw_priv->pdev->of_node, addr);
++	}
++	if (err < 0) {
++		dev_warn(hw_priv->pdev, "no mac address provided, using random\n");
++		eth_random_addr(addr);
++	}
++#else
+ 	// fill in mac addresses
+ 	if (hw_priv->pdev->of_node) {
+ 		addr = of_get_mac_address(hw_priv->pdev->of_node);
+@@ -522,6 +537,8 @@ int xradio_core_init(struct sdio_func* func)
+ 		eth_random_addr(randomaddr);
+ 		addr = randomaddr;
+ 	}
++#endif
++
+ 	memcpy(hw_priv->addresses[0].addr, addr, ETH_ALEN);
+ 	memcpy(hw_priv->addresses[1].addr, addr, ETH_ALEN);
+ 	hw_priv->addresses[1].addr[5] += 0x01;
+-- 
+2.25.1
+
-- 
2.25.1




More information about the buildroot mailing list