Commit 29cfe6f4 authored by Timur Tabi's avatar Timur Tabi Committed by Paul Mackerras

[POWERPC] add of_get_mac_address and update fsl_soc.c to use it

Add function of_get_mac_address(), which obtains the best MAC address to use
from the device tree by checking various properties in order.  The order is:
'mac-address', then 'local-mac-address', then 'address'.  It skips properties
that contain invalid MAC addresses, which were probably not initialized
by U-Boot.

Update gfar_of_init() and fs_enet_of_init() in fsl_soc.c to call
of_get_mac_address().
Signed-off-by: default avatarTimur Tabi <timur@freescale.com>
Signed-off-by: default avatarPaul Mackerras <paulus@samba.org>
parent a3252544
...@@ -5,6 +5,7 @@ ...@@ -5,6 +5,7 @@
#include <linux/pci_regs.h> #include <linux/pci_regs.h>
#include <linux/module.h> #include <linux/module.h>
#include <linux/ioport.h> #include <linux/ioport.h>
#include <linux/etherdevice.h>
#include <asm/prom.h> #include <asm/prom.h>
#include <asm/pci-bridge.h> #include <asm/pci-bridge.h>
...@@ -1003,3 +1004,42 @@ int of_irq_map_one(struct device_node *device, int index, struct of_irq *out_irq ...@@ -1003,3 +1004,42 @@ int of_irq_map_one(struct device_node *device, int index, struct of_irq *out_irq
return res; return res;
} }
EXPORT_SYMBOL_GPL(of_irq_map_one); EXPORT_SYMBOL_GPL(of_irq_map_one);
/**
* 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
* address. If that isn't set, then 'local-mac-address' is checked next,
* because that is the default address. If that isn't set, then the obsolete
* 'address' is checked, just in case we're using an old device tree.
*
* Note that the 'address' property is supposed to contain a virtual address of
* the register set, but some DTS files have redefined that property to be the
* MAC address.
*
* All-zero MAC addresses are rejected, because those could be properties that
* exist in the device tree, but were not set by U-Boot. For example, the
* DTS could define 'mac-address' and 'local-mac-address', with zero MAC
* addresses. Some older U-Boots only initialized 'local-mac-address'. In
* this case, the real MAC is in 'local-mac-address', and 'mac-address' exists
* but is all zeros.
*/
const void *of_get_mac_address(struct device_node *np)
{
struct property *pp;
pp = of_find_property(np, "mac-address", NULL);
if (pp && (pp->length == 6) && is_valid_ether_addr(pp->value))
return pp->value;
pp = of_find_property(np, "local-mac-address", NULL);
if (pp && (pp->length == 6) && is_valid_ether_addr(pp->value))
return pp->value;
pp = of_find_property(np, "address", NULL);
if (pp && (pp->length == 6) && is_valid_ether_addr(pp->value))
return pp->value;
return NULL;
}
EXPORT_SYMBOL(of_get_mac_address);
...@@ -233,14 +233,7 @@ static int __init gfar_of_init(void) ...@@ -233,14 +233,7 @@ static int __init gfar_of_init(void)
goto err; goto err;
} }
mac_addr = get_property(np, "local-mac-address", NULL); mac_addr = of_get_mac_address(np);
if (mac_addr == NULL)
mac_addr = get_property(np, "mac-address", NULL);
if (mac_addr == NULL) {
/* Obsolete */
mac_addr = get_property(np, "address", NULL);
}
if (mac_addr) if (mac_addr)
memcpy(gfar_data.mac_addr, mac_addr, 6); memcpy(gfar_data.mac_addr, mac_addr, 6);
...@@ -646,8 +639,9 @@ static int __init fs_enet_of_init(void) ...@@ -646,8 +639,9 @@ static int __init fs_enet_of_init(void)
goto unreg; goto unreg;
} }
mac_addr = get_property(np, "mac-address", NULL); mac_addr = of_get_mac_address(np);
memcpy(fs_enet_data.macaddr, mac_addr, 6); if (mac_addr)
memcpy(fs_enet_data.macaddr, mac_addr, 6);
ph = get_property(np, "phy-handle", NULL); ph = get_property(np, "phy-handle", NULL);
phy = of_find_node_by_phandle(*ph); phy = of_find_node_by_phandle(*ph);
...@@ -931,8 +925,9 @@ static int __init fs_enet_of_init(void) ...@@ -931,8 +925,9 @@ static int __init fs_enet_of_init(void)
goto err; goto err;
r[0].name = enet_regs; r[0].name = enet_regs;
mac_addr = (void *)get_property(np, "mac-address", NULL); mac_addr = of_get_mac_address(np);
memcpy(fs_enet_data.macaddr, mac_addr, 6); if (mac_addr)
memcpy(fs_enet_data.macaddr, mac_addr, 6);
ph = (phandle *) get_property(np, "phy-handle", NULL); ph = (phandle *) get_property(np, "phy-handle", NULL);
if (ph != NULL) if (ph != NULL)
......
...@@ -255,6 +255,8 @@ extern void kdump_move_device_tree(void); ...@@ -255,6 +255,8 @@ extern void kdump_move_device_tree(void);
/* CPU OF node matching */ /* CPU OF node matching */
struct device_node *of_get_cpu_node(int cpu, unsigned int *thread); struct device_node *of_get_cpu_node(int cpu, unsigned int *thread);
/* Get the MAC address */
extern const void *of_get_mac_address(struct device_node *np);
/* /*
* OF interrupt mapping * OF interrupt mapping
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment