kernel: drop support for mtd-mac-address
authorAnsuel Smith <ansuelsmth@gmail.com>
Sun, 22 Aug 2021 12:58:53 +0000 (14:58 +0200)
committerAdrian Schmutzler <freifunk@adrianschmutzler.de>
Sat, 9 Oct 2021 19:14:54 +0000 (21:14 +0200)
Now that we have fully switched to nvmem interface we can drop
the use of mtd-mac-address patches as it's not used anymore and
the new nvmem implementation should be used for any new device.

Signed-off-by: Ansuel Smith <ansuelsmth@gmail.com>
target/linux/generic/pending-5.10/681-NET-add-mtd-mac-address-support-to-of_get_mac_addres.patch [deleted file]
target/linux/generic/pending-5.10/682-of_net-add-mac-address-increment-support.patch
target/linux/generic/pending-5.10/683-of_net-add-mac-address-to-of-tree.patch
target/linux/generic/pending-5.4/681-NET-add-mtd-mac-address-support-to-of_get_mac_addres.patch [deleted file]
target/linux/generic/pending-5.4/682-of_net-add-mac-address-increment-support.patch
target/linux/generic/pending-5.4/683-of_net-add-mac-address-to-of-tree.patch

diff --git a/target/linux/generic/pending-5.10/681-NET-add-mtd-mac-address-support-to-of_get_mac_addres.patch b/target/linux/generic/pending-5.10/681-NET-add-mtd-mac-address-support-to-of_get_mac_addres.patch
deleted file mode 100644 (file)
index 30e292a..0000000
+++ /dev/null
@@ -1,102 +0,0 @@
-From 6f8e5369ae054ec6c9265581d5a7e39738a5cd84 Mon Sep 17 00:00:00 2001
-From: Ansuel Smith <ansuelsmth@gmail.com>
-Date: Tue, 30 Mar 2021 13:16:38 +0200
-Subject: [PATCH 1/2] NET: add mtd-mac-address support to of_get_mac_address()
-
-Many embedded devices have information such as mac addresses stored inside mtd
-devices. This patch allows us to add a property inside a node describing a
-network interface. The new property points at a mtd partition with an offset
-where the mac address can be found.
-
-Signed-off-by: John Crispin <blogic@openwrt.org>
-Signed-off-by: Felix Fietkau <nbd@nbd.name>
-Signed-off-by: Ansuel Smith <ansuelsmth@gmail.com>
----
- drivers/of/of_net.c | 75 ++++++++++++++++++++++++++++++++++++++++++++-
- 1 file changed, 74 insertions(+), 1 deletion(-)
-
---- a/drivers/of/of_net.c
-+++ b/drivers/of/of_net.c
-@@ -12,6 +12,7 @@
- #include <linux/export.h>
- #include <linux/device.h>
- #include <linux/nvmem-consumer.h>
-+#include <linux/mtd/mtd.h>
- /**
-  * of_get_phy_mode - Get phy mode for given device_node
-@@ -95,6 +96,52 @@ static int of_get_mac_addr_nvmem(struct
-       return 0;
- }
-+static int of_get_mac_address_mtd(struct device_node *np, u8 *addr)
-+{
-+#ifdef CONFIG_MTD
-+      struct platform_device *pdev = of_find_device_by_node(np);
-+      struct device_node *mtd_np = NULL;
-+      size_t retlen;
-+      int size, ret;
-+      struct mtd_info *mtd;
-+      const char *part;
-+      const __be32 *list;
-+      phandle phandle;
-+      u8 mac[ETH_ALEN];
-+
-+      list = of_get_property(np, "mtd-mac-address", &size);
-+      if (!list || (size != (2 * sizeof(*list))))
-+              return -ENODEV;
-+
-+      phandle = be32_to_cpup(list++);
-+      if (phandle)
-+              mtd_np = of_find_node_by_phandle(phandle);
-+
-+      if (!mtd_np)
-+              return -ENODEV;
-+
-+      part = of_get_property(mtd_np, "label", NULL);
-+      if (!part)
-+              part = mtd_np->name;
-+
-+      mtd = get_mtd_device_nm(part);
-+      if (IS_ERR(mtd))
-+              return -ENODEV;
-+
-+      ret = mtd_read(mtd, be32_to_cpup(list), 6, &retlen, mac);
-+      put_mtd_device(mtd);
-+
-+      if (!is_valid_ether_addr(mac))
-+              return -EINVAL;
-+
-+      memcpy(addr, mac, ETH_ALEN);
-+
-+      return 0;
-+#endif
-+      return -EINVAL;
-+}
-+
-+
- /**
-  * Search the device tree for the best MAC address to use.  'mac-address' is
-  * checked first, because that is supposed to contain to "most recent" MAC
-@@ -115,6 +162,10 @@ static int of_get_mac_addr_nvmem(struct
-  * this case, the real MAC is in 'local-mac-address', and 'mac-address' exists
-  * but is all zeros.
-  *
-+ *
-+ * If a mtd-mac-address property exists, try to fetch the MAC address from the
-+ * specified mtd device.
-+ *
-  * Return: 0 on success and errno in case of error.
- */
- int of_get_mac_address(struct device_node *np, u8 *addr)
-@@ -136,6 +187,10 @@ int of_get_mac_address(struct device_nod
-       if (!ret)
-               return 0;
-+      ret = of_get_mac_address_mtd(np, addr);
-+      if (!ret)
-+              return 0;
-+
-       return of_get_mac_addr_nvmem(np, addr);
- }
- EXPORT_SYMBOL(of_get_mac_address);
index e48e6311c6f57622bf05eaec94ee470da330d8d6..cd5cad38f75dd8978795723393ea8e2ba70cc3b5 100644 (file)
@@ -20,9 +20,9 @@ Signed-off-by: Ansuel Smith <ansuelsmth@gmail.com>
 
 --- a/drivers/of/of_net.c
 +++ b/drivers/of/of_net.c
-@@ -166,31 +166,56 @@ static int of_get_mac_address_mtd(struct
-  * If a mtd-mac-address property exists, try to fetch the MAC address from the
-  * specified mtd device.
+@@ -115,27 +115,52 @@ static int of_get_mac_addr_nvmem(struct
+  * this case, the real MAC is in 'local-mac-address', and 'mac-address' exists
+  * but is all zeros.
   *
 + * DT can tell the system to increment the mac-address after is extracted by
 + * using:
@@ -63,11 +63,6 @@ Signed-off-by: Ansuel Smith <ansuelsmth@gmail.com>
        ret = of_get_mac_addr(np, "address", addr);
        if (!ret)
 -              return 0;
-+              goto found;
-       ret = of_get_mac_address_mtd(np, addr);
-       if (!ret)
--              return 0;
 +              goto found;
 +
 +      ret = of_get_mac_addr_nvmem(np, addr);
index 02f7519f7410421758d82d439e63664643ee8dea..0cb96025cf7c167a848c8a2c14c7bf5659b9ad8a 100644 (file)
@@ -1,7 +1,7 @@
 --- a/drivers/of/of_net.c
 +++ b/drivers/of/of_net.c
-@@ -141,6 +141,26 @@ static int of_get_mac_address_mtd(struct
-       return -EINVAL;
+@@ -95,6 +95,27 @@ static int of_get_mac_addr_nvmem(struct
+       return 0;
  }
  
 +static int of_add_mac_address(struct device_node *np, u8* addr)
 +      kfree(prop);
 +      return -ENOMEM;
 +}
++
  /**
   * Search the device tree for the best MAC address to use.  'mac-address' is
-@@ -216,6 +236,7 @@ found:
+  * checked first, because that is supposed to contain to "most recent" MAC
+@@ -161,6 +182,7 @@ found:
        if (!of_property_read_u32(np, "mac-address-increment", &mac_inc))
                addr[inc_idx] += mac_inc;
  
diff --git a/target/linux/generic/pending-5.4/681-NET-add-mtd-mac-address-support-to-of_get_mac_addres.patch b/target/linux/generic/pending-5.4/681-NET-add-mtd-mac-address-support-to-of_get_mac_addres.patch
deleted file mode 100644 (file)
index a9e06cd..0000000
+++ /dev/null
@@ -1,102 +0,0 @@
-From 6f8e5369ae054ec6c9265581d5a7e39738a5cd84 Mon Sep 17 00:00:00 2001
-From: Ansuel Smith <ansuelsmth@gmail.com>
-Date: Tue, 30 Mar 2021 13:16:38 +0200
-Subject: [PATCH 1/2] NET: add mtd-mac-address support to of_get_mac_address()
-
-Many embedded devices have information such as mac addresses stored inside mtd
-devices. This patch allows us to add a property inside a node describing a
-network interface. The new property points at a mtd partition with an offset
-where the mac address can be found.
-
-Signed-off-by: John Crispin <blogic@openwrt.org>
-Signed-off-by: Felix Fietkau <nbd@nbd.name>
-Signed-off-by: Ansuel Smith <ansuelsmth@gmail.com>
----
- drivers/of/of_net.c | 75 ++++++++++++++++++++++++++++++++++++++++++++-
- 1 file changed, 74 insertions(+), 1 deletion(-)
-
---- a/drivers/of/of_net.c
-+++ b/drivers/of/of_net.c
-@@ -12,6 +12,7 @@
- #include <linux/export.h>
- #include <linux/device.h>
- #include <linux/nvmem-consumer.h>
-+#include <linux/mtd/mtd.h>
- /**
-  * of_get_phy_mode - Get phy mode for given device_node
-@@ -89,6 +90,52 @@ static int of_get_mac_addr_nvmem(struct
-       return 0;
- }
-+static int of_get_mac_address_mtd(struct device_node *np, u8 *addr)
-+{
-+#ifdef CONFIG_MTD
-+      struct platform_device *pdev = of_find_device_by_node(np);
-+      struct device_node *mtd_np = NULL;
-+      size_t retlen;
-+      int size, ret;
-+      struct mtd_info *mtd;
-+      const char *part;
-+      const __be32 *list;
-+      phandle phandle;
-+      u8 mac[ETH_ALEN];
-+
-+      list = of_get_property(np, "mtd-mac-address", &size);
-+      if (!list || (size != (2 * sizeof(*list))))
-+              return -ENODEV;
-+
-+      phandle = be32_to_cpup(list++);
-+      if (phandle)
-+              mtd_np = of_find_node_by_phandle(phandle);
-+
-+      if (!mtd_np)
-+              return -ENODEV;
-+
-+      part = of_get_property(mtd_np, "label", NULL);
-+      if (!part)
-+              part = mtd_np->name;
-+
-+      mtd = get_mtd_device_nm(part);
-+      if (IS_ERR(mtd))
-+              return -ENODEV;
-+
-+      ret = mtd_read(mtd, be32_to_cpup(list), 6, &retlen, mac);
-+      put_mtd_device(mtd);
-+
-+      if (!is_valid_ether_addr(mac))
-+              return -EINVAL;
-+
-+      memcpy(addr, mac, ETH_ALEN);
-+
-+      return 0;
-+#endif
-+      return -EINVAL;
-+}
-+
-+
- /**
-  * Search the device tree for the best MAC address to use.  'mac-address' is
-  * checked first, because that is supposed to contain to "most recent" MAC
-@@ -109,6 +156,10 @@ static int of_get_mac_addr_nvmem(struct
-  * this case, the real MAC is in 'local-mac-address', and 'mac-address' exists
-  * but is all zeros.
-  *
-+ *
-+ * If a mtd-mac-address property exists, try to fetch the MAC address from the
-+ * specified mtd device.
-+ *
-  * Return: 0 on success and errno in case of error.
- */
- int of_get_mac_address(struct device_node *np, u8 *addr)
-@@ -130,6 +181,10 @@ int of_get_mac_address(struct device_nod
-       if (!ret)
-               return 0;
-+      ret = of_get_mac_address_mtd(np, addr);
-+      if (!ret)
-+              return 0;
-+
-       return of_get_mac_addr_nvmem(np, addr);
- }
- EXPORT_SYMBOL(of_get_mac_address);
index 35c717ca51eb1b810ddd7353b63a4b473c8add46..acc7eaf47bd98df574b65f7467e711bd69e0e8c9 100644 (file)
@@ -20,9 +20,9 @@ Signed-off-by: Ansuel Smith <ansuelsmth@gmail.com>
 
 --- a/drivers/of/of_net.c
 +++ b/drivers/of/of_net.c
-@@ -160,31 +160,56 @@ static int of_get_mac_address_mtd(struct
-  * If a mtd-mac-address property exists, try to fetch the MAC address from the
-  * specified mtd device.
+@@ -109,27 +109,52 @@ static int of_get_mac_addr_nvmem(struct
+  * this case, the real MAC is in 'local-mac-address', and 'mac-address' exists
+  * but is all zeros.
   *
 + * DT can tell the system to increment the mac-address after is extracted by
 + * using:
@@ -63,11 +63,6 @@ Signed-off-by: Ansuel Smith <ansuelsmth@gmail.com>
        ret = of_get_mac_addr(np, "address", addr);
        if (!ret)
 -              return 0;
-+              goto found;
-       ret = of_get_mac_address_mtd(np, addr);
-       if (!ret)
--              return 0;
 +              goto found;
 +
 +      ret = of_get_mac_addr_nvmem(np, addr);
index ea5883f037f89378d70dccb1f485e164de76d267..78514dd9e3d88bec189b8901ca67e35df1e8daf2 100644 (file)
@@ -1,7 +1,7 @@
 --- a/drivers/of/of_net.c
 +++ b/drivers/of/of_net.c
-@@ -135,6 +135,26 @@ static int of_get_mac_address_mtd(struct
-       return -EINVAL;
+@@ -89,6 +89,27 @@ static int of_get_mac_addr_nvmem(struct
+       return 0;
  }
  
 +static int of_add_mac_address(struct device_node *np, u8* addr)
 +      kfree(prop);
 +      return -ENOMEM;
 +}
++
  /**
   * Search the device tree for the best MAC address to use.  'mac-address' is
-@@ -210,6 +230,7 @@ found:
+  * checked first, because that is supposed to contain to "most recent" MAC
+@@ -155,6 +176,7 @@ found:
        if (!of_property_read_u32(np, "mac-address-increment", &mac_inc))
                addr[inc_idx] += mac_inc;