Johnny Hughes
2019-02-04 c1f36c28393a7bb126cbf436cd6a4077a5b5c313
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
From b81c9307109512d284a07d864fe886c29df61778 Mon Sep 17 00:00:00 2001
From: Marcin Wojtas <mw@semihalf.com>
Date: Thu, 18 Jan 2018 13:31:38 +0100
Subject: [PATCH 21/46] device property: Introduce fwnode_get_mac_address()
 
Until now there were two almost identical functions for
obtaining MAC address - of_get_mac_address() and, more generic,
device_get_mac_address(). However it is not uncommon,
that the network interface is represented as a child
of the actual controller, hence it is not associated
directly to any struct device, required by the latter
routine.
 
This commit allows for getting the MAC address for
children nodes in the ACPI world by introducing a new function -
fwnode_get_mac_address(). This commit also changes
device_get_mac_address() routine to be its wrapper, in order
to prevent unnecessary duplication.
 
Signed-off-by: Marcin Wojtas <mw@semihalf.com>
Acked-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
(cherry picked from commit babe2dbb28e780177ae2fba6a5640be1712f4af0)
Signed-off-by: Marcin Wojtas <mw@semihalf.com>
---
 drivers/base/property.c  | 28 ++++++++++++++++++++--------
 include/linux/property.h |  2 ++
 2 files changed, 22 insertions(+), 8 deletions(-)
 
diff --git a/drivers/base/property.c b/drivers/base/property.c
index 7ed99c1..fa488e5 100644
--- a/drivers/base/property.c
+++ b/drivers/base/property.c
@@ -1148,11 +1148,11 @@ int device_get_phy_mode(struct device *dev)
 }
 EXPORT_SYMBOL_GPL(device_get_phy_mode);
 
-static void *device_get_mac_addr(struct device *dev,
+static void *fwnode_get_mac_addr(struct fwnode_handle *fwnode,
                  const char *name, char *addr,
                  int alen)
 {
-    int ret = device_property_read_u8_array(dev, name, addr, alen);
+    int ret = fwnode_property_read_u8_array(fwnode, name, addr, alen);
 
     if (ret == 0 && alen == ETH_ALEN && is_valid_ether_addr(addr))
         return addr;
@@ -1160,8 +1160,8 @@ static void *device_get_mac_addr(struct device *dev,
 }
 
 /**
- * device_get_mac_address - Get the MAC for a given device
- * @dev:    Pointer to the device
+ * fwnode_get_mac_address - Get the MAC from the firmware node
+ * @fwnode:    Pointer to the firmware node
  * @addr:    Address of buffer to store the MAC in
  * @alen:    Length of the buffer pointed to by addr, should be ETH_ALEN
  *
@@ -1182,19 +1182,31 @@ static void *device_get_mac_addr(struct device *dev,
  * In this case, the real MAC is in 'local-mac-address', and 'mac-address'
  * exists but is all zeros.
 */
-void *device_get_mac_address(struct device *dev, char *addr, int alen)
+void *fwnode_get_mac_address(struct fwnode_handle *fwnode, char *addr, int alen)
 {
     char *res;
 
-    res = device_get_mac_addr(dev, "mac-address", addr, alen);
+    res = fwnode_get_mac_addr(fwnode, "mac-address", addr, alen);
     if (res)
         return res;
 
-    res = device_get_mac_addr(dev, "local-mac-address", addr, alen);
+    res = fwnode_get_mac_addr(fwnode, "local-mac-address", addr, alen);
     if (res)
         return res;
 
-    return device_get_mac_addr(dev, "address", addr, alen);
+    return fwnode_get_mac_addr(fwnode, "address", addr, alen);
+}
+EXPORT_SYMBOL(fwnode_get_mac_address);
+
+/**
+ * device_get_mac_address - Get the MAC for a given device
+ * @dev:    Pointer to the device
+ * @addr:    Address of buffer to store the MAC in
+ * @alen:    Length of the buffer pointed to by addr, should be ETH_ALEN
+ */
+void *device_get_mac_address(struct device *dev, char *addr, int alen)
+{
+    return fwnode_get_mac_address(dev_fwnode(dev), addr, alen);
 }
 EXPORT_SYMBOL(device_get_mac_address);
 
diff --git a/include/linux/property.h b/include/linux/property.h
index 6bebee1..04d8552 100644
--- a/include/linux/property.h
+++ b/include/linux/property.h
@@ -279,6 +279,8 @@ int device_get_phy_mode(struct device *dev);
 
 void *device_get_mac_address(struct device *dev, char *addr, int alen);
 
+void *fwnode_get_mac_address(struct fwnode_handle *fwnode,
+                 char *addr, int alen);
 struct fwnode_handle *fwnode_graph_get_next_endpoint(
     const struct fwnode_handle *fwnode, struct fwnode_handle *prev);
 struct fwnode_handle *
-- 
2.7.4