Revert "generic: add mac-address property for NVMEM mac addresses"

This reverts commit b30924873062be47250e6b872347461c8221688d.

This commit could create a property without allocated memory, breaking
subsequent reads over a nodes property. Also, the mac-address-increment
was not applied when reading from nvmem.

Revert this commit for now, which breaks the label-mac-address logic.
Possibly, traversing the device-tree from the netdev side is easier
anyways.

Signed-off-by: David Bauer <mail@david-bauer.net>

diff --git a/target/linux/generic/hack-5.10/683-NET-add-mac-address-property.patch b/target/linux/generic/hack-5.10/683-NET-add-mac-address-property.patch
deleted file mode 100644 (file)
index de561c5..0000000
--- a/target/linux/generic/hack-5.10/683-NET-add-mac-address-property.patch
+++ /dev/null
@@ -1,59 +0,0 @@
-From: David Bauer <mail@david-bauer.net>
-Date: Tue, 20 Jul 2021 22:00:10 +0200
-Subject: [PATCH] generic: add mac-address property for NVMEM mac addresses
-
-Traversing the device-tree by referencing a network device to determine
-a devices labe-mac does not work with the generic nvmem implementation,
-as the userspace expects the MAC-address to be available as a
-device-tree property.
-
-The legacy mtd-mac-address implementation did create such a node. Do the
-same when using the nvmem implementation to allow reading the MAC
-address.
-
-Fixes commit d284e6ef0f06 ("treewide: convert mtd-mac-address-increment*
-to generic implementation")
-
-Signed-off-by: David Bauer <mail@david-bauer.net>
-
---- a/drivers/of/of_net.c
-+++ b/drivers/of/of_net.c
-@@ -61,6 +61,7 @@ static void *of_get_mac_addr_nvmem(struc
-       void *mac;
-       u8 nvmem_mac[ETH_ALEN];
-       struct platform_device *pdev = of_find_device_by_node(np);
-+      struct property *prop;
- 
-       if (!pdev) {
-               *err = -ENODEV;
-@@ -78,10 +79,29 @@ static void *of_get_mac_addr_nvmem(struc
-       put_device(&pdev->dev);
-       if (!mac) {
-               *err = -ENOMEM;
--              return NULL;
-+              goto out_err;
-+      }
-+
-+      prop = devm_kzalloc(&pdev->dev, sizeof(*prop), GFP_KERNEL);
-+      if (!prop) {
-+              *err = -ENOMEM;
-+              goto out_err;
-+      }
-+      prop->name = "mac-address";
-+      prop->length = ETH_ALEN;
-+      prop->value = devm_kmemdup(&pdev->dev, mac, ETH_ALEN, GFP_KERNEL);
-+      if (!prop->value || of_add_property(np, prop)) {
-+              *err = -ENOMEM;
-+              goto out_err;
-       }
- 
-       return mac;
-+
-+out_err:
-+      devm_kfree(&pdev->dev, prop->value);
-+      devm_kfree(&pdev->dev, prop);
-+      devm_kfree(&pdev->dev, mac);
-+      return NULL;
- }
- 
- static void *of_get_mac_address_mtd(struct device_node *np)

diff --git a/target/linux/generic/hack-5.4/683-NET-add-mac-address-property.patch b/target/linux/generic/hack-5.4/683-NET-add-mac-address-property.patch
deleted file mode 100644 (file)
index f0418db..0000000
--- a/target/linux/generic/hack-5.4/683-NET-add-mac-address-property.patch
+++ /dev/null
@@ -1,59 +0,0 @@
-From: David Bauer <mail@david-bauer.net>
-Date: Tue, 20 Jul 2021 22:00:10 +0200
-Subject: [PATCH] generic: add mac-address property for NVMEM mac addresses
-
-Traversing the device-tree by referencing a network device to determine
-a devices labe-mac does not work with the generic nvmem implementation,
-as the userspace expects the MAC-address to be available as a
-device-tree property.
-
-The legacy mtd-mac-address implementation did create such a node. Do the
-same when using the nvmem implementation to allow reading the MAC
-address.
-
-Fixes commit d284e6ef0f06 ("treewide: convert mtd-mac-address-increment*
-to generic implementation")
-
-Signed-off-by: David Bauer <mail@david-bauer.net>
-
---- a/drivers/of/of_net.c
-+++ b/drivers/of/of_net.c
-@@ -55,6 +55,7 @@ static void *of_get_mac_addr_nvmem(struc
-       void *mac;
-       u8 nvmem_mac[ETH_ALEN];
-       struct platform_device *pdev = of_find_device_by_node(np);
-+      struct property *prop;
- 
-       if (!pdev) {
-               *err = -ENODEV;
-@@ -72,10 +73,29 @@ static void *of_get_mac_addr_nvmem(struc
-       put_device(&pdev->dev);
-       if (!mac) {
-               *err = -ENOMEM;
--              return NULL;
-+              goto out_err;
-+      }
-+
-+      prop = devm_kzalloc(&pdev->dev, sizeof(*prop), GFP_KERNEL);
-+      if (!prop) {
-+              *err = -ENOMEM;
-+              goto out_err;
-+      }
-+      prop->name = "mac-address";
-+      prop->length = ETH_ALEN;
-+      prop->value = devm_kmemdup(&pdev->dev, mac, ETH_ALEN, GFP_KERNEL);
-+      if (!prop->value || of_add_property(np, prop)) {
-+              *err = -ENOMEM;
-+              goto out_err;
-       }
- 
-       return mac;
-+
-+out_err:
-+      devm_kfree(&pdev->dev, prop->value);
-+      devm_kfree(&pdev->dev, prop);
-+      devm_kfree(&pdev->dev, mac);
-+      return NULL;
- }
- 
- static void *of_get_mac_address_mtd(struct device_node *np)