diff options
author | David Bauer <mail@david-bauer.net> | 2021-07-28 13:38:07 +0200 |
---|---|---|
committer | David Bauer <mail@david-bauer.net> | 2021-07-28 13:48:24 +0200 |
commit | bd521f2a832ef1b4ea1a7680c0c087787df93278 (patch) | |
tree | ce419dd6d773d0d7f3d81def7c364abc6c868e26 /target/linux | |
parent | 55b4b3655263984b92e4b9fc515a5e6b8003c655 (diff) | |
download | upstream-bd521f2a832ef1b4ea1a7680c0c087787df93278.tar.gz upstream-bd521f2a832ef1b4ea1a7680c0c087787df93278.tar.bz2 upstream-bd521f2a832ef1b4ea1a7680c0c087787df93278.zip |
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>
Diffstat (limited to 'target/linux')
-rw-r--r-- | target/linux/generic/hack-5.10/683-NET-add-mac-address-property.patch | 59 | ||||
-rw-r--r-- | target/linux/generic/hack-5.4/683-NET-add-mac-address-property.patch | 59 |
2 files changed, 0 insertions, 118 deletions
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 index de561c5336..0000000000 --- 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 index f0418dbb70..0000000000 --- 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) |