kernel: support reading hex MAC address from NVMEM
authorDaniel Golle <daniel@makrotopia.org>
Thu, 26 Oct 2023 02:50:40 +0000 (03:50 +0100)
committerDaniel Golle <daniel@makrotopia.org>
Fri, 3 Nov 2023 21:16:29 +0000 (21:16 +0000)
In addition to binary and ASCII-formatted MAC addresses, add support
for processing hexadecimal encoded MAC addresses from NVMEM.

Signed-off-by: Daniel Golle <daniel@makrotopia.org>
target/linux/generic/pending-5.15/804-nvmem-core-support-mac-base-fixed-layout-cells.patch
target/linux/generic/pending-6.1/804-nvmem-core-support-mac-base-fixed-layout-cells.patch

index 6791b63ac09d903c2ba431564dd9c6d90a3ea94e..95f29b1865dd5f0d3734198bbbda6b12e2a97778 100644 (file)
@@ -20,10 +20,11 @@ string.
  
 --- a/drivers/nvmem/core.c
 +++ b/drivers/nvmem/core.c
-@@ -7,9 +7,11 @@
+@@ -7,9 +7,12 @@
   */
  
  #include <linux/device.h>
++#include <linux/ctype.h>
 +#include <linux/etherdevice.h>
  #include <linux/export.h>
  #include <linux/fs.h>
@@ -32,7 +33,7 @@ string.
  #include <linux/init.h>
  #include <linux/kref.h>
  #include <linux/module.h>
-@@ -696,6 +698,37 @@ static int nvmem_validate_keepouts(struc
+@@ -696,6 +699,62 @@ static int nvmem_validate_keepouts(struc
        return 0;
  }
  
@@ -66,25 +67,55 @@ string.
 +
 +      return 0;
 +}
++
++static int nvmem_mac_base_hex_read(void *context, const char *id, int index, unsigned int offset,
++                                 void *buf, size_t bytes)
++{
++      u8 mac[ETH_ALEN], *hexstr;
++      int i;
++
++      if (WARN_ON(bytes != 2 * ETH_ALEN))
++              return -EINVAL;
++
++      hexstr = (u8 *)buf;
++      for (i = 0; i < ETH_ALEN; i++) {
++              if (!isxdigit(hexstr[i * 2]) || !isxdigit(hexstr[i * 2 + 1]))
++                      return -EINVAL;
++
++              mac[i] = (hex_to_bin(hexstr[i * 2]) << 4) | hex_to_bin(hexstr[i * 2 + 1]);
++      }
++
++      if (index)
++              eth_addr_add(mac, index);
++
++      ether_addr_copy(buf, mac);
++
++      return 0;
++}
 +
  static int nvmem_add_cells_from_dt(struct nvmem_device *nvmem, struct device_node *np)
  {
        struct nvmem_layout *layout = nvmem->layout;
-@@ -731,6 +764,20 @@ static int nvmem_add_cells_from_dt(struc
+@@ -731,6 +790,25 @@ static int nvmem_add_cells_from_dt(struc
                if (layout && layout->fixup_cell_info)
                        layout->fixup_cell_info(nvmem, layout, &info);
  
 +              if (of_device_is_compatible(np, "fixed-layout")) {
 +                      if (of_device_is_compatible(child, "mac-base")) {
-+                              if (info.bytes == 6) {
++                              if (info.bytes == ETH_ALEN) {
 +                                      info.raw_len = info.bytes;
 +                                      info.bytes = ETH_ALEN;
 +                                      info.read_post_process = nvmem_mac_base_raw_read;
++                              } else if (info.bytes == 2 * ETH_ALEN) {
++                                      info.raw_len = info.bytes;
++                                      info.bytes = ETH_ALEN;
++                                      info.read_post_process = nvmem_mac_base_hex_read;
 +                              } else if (info.bytes == 3 * ETH_ALEN - 1) {
 +                                      info.raw_len = info.bytes;
 +                                      info.bytes = ETH_ALEN;
 +                                      info.read_post_process = nvmem_mac_base_ascii_read;
 +                              }
++
 +                      }
 +              }
 +
index 6791b63ac09d903c2ba431564dd9c6d90a3ea94e..95f29b1865dd5f0d3734198bbbda6b12e2a97778 100644 (file)
@@ -20,10 +20,11 @@ string.
  
 --- a/drivers/nvmem/core.c
 +++ b/drivers/nvmem/core.c
-@@ -7,9 +7,11 @@
+@@ -7,9 +7,12 @@
   */
  
  #include <linux/device.h>
++#include <linux/ctype.h>
 +#include <linux/etherdevice.h>
  #include <linux/export.h>
  #include <linux/fs.h>
@@ -32,7 +33,7 @@ string.
  #include <linux/init.h>
  #include <linux/kref.h>
  #include <linux/module.h>
-@@ -696,6 +698,37 @@ static int nvmem_validate_keepouts(struc
+@@ -696,6 +699,62 @@ static int nvmem_validate_keepouts(struc
        return 0;
  }
  
@@ -66,25 +67,55 @@ string.
 +
 +      return 0;
 +}
++
++static int nvmem_mac_base_hex_read(void *context, const char *id, int index, unsigned int offset,
++                                 void *buf, size_t bytes)
++{
++      u8 mac[ETH_ALEN], *hexstr;
++      int i;
++
++      if (WARN_ON(bytes != 2 * ETH_ALEN))
++              return -EINVAL;
++
++      hexstr = (u8 *)buf;
++      for (i = 0; i < ETH_ALEN; i++) {
++              if (!isxdigit(hexstr[i * 2]) || !isxdigit(hexstr[i * 2 + 1]))
++                      return -EINVAL;
++
++              mac[i] = (hex_to_bin(hexstr[i * 2]) << 4) | hex_to_bin(hexstr[i * 2 + 1]);
++      }
++
++      if (index)
++              eth_addr_add(mac, index);
++
++      ether_addr_copy(buf, mac);
++
++      return 0;
++}
 +
  static int nvmem_add_cells_from_dt(struct nvmem_device *nvmem, struct device_node *np)
  {
        struct nvmem_layout *layout = nvmem->layout;
-@@ -731,6 +764,20 @@ static int nvmem_add_cells_from_dt(struc
+@@ -731,6 +790,25 @@ static int nvmem_add_cells_from_dt(struc
                if (layout && layout->fixup_cell_info)
                        layout->fixup_cell_info(nvmem, layout, &info);
  
 +              if (of_device_is_compatible(np, "fixed-layout")) {
 +                      if (of_device_is_compatible(child, "mac-base")) {
-+                              if (info.bytes == 6) {
++                              if (info.bytes == ETH_ALEN) {
 +                                      info.raw_len = info.bytes;
 +                                      info.bytes = ETH_ALEN;
 +                                      info.read_post_process = nvmem_mac_base_raw_read;
++                              } else if (info.bytes == 2 * ETH_ALEN) {
++                                      info.raw_len = info.bytes;
++                                      info.bytes = ETH_ALEN;
++                                      info.read_post_process = nvmem_mac_base_hex_read;
 +                              } else if (info.bytes == 3 * ETH_ALEN - 1) {
 +                                      info.raw_len = info.bytes;
 +                                      info.bytes = ETH_ALEN;
 +                                      info.read_post_process = nvmem_mac_base_ascii_read;
 +                              }
++
 +                      }
 +              }
 +