[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