diff options
| author | Eric Miao <eric.miao@linaro.org> | 2011-08-16 19:33:44 +0000 |
|---|---|---|
| committer | Eric Miao <eric.miao@linaro.org> | 2012-01-02 21:54:14 +0800 |
| commit | 805565422fc1ccf0d5ca289db7d65f49617d5a2d (patch) | |
| tree | 3dcb3d9a89d385dffcb344c2f2cb616bb09d7848 | |
| parent | 0ce2fe60560de63a78067d80ef05e434b7963dc8 (diff) | |
net/eth.c: fix eth_write_hwaddr() to use dev->enetaddr as fall back
Ignore the return value of eth_getenv_enetaddr_by_index(), and if it
fails, fall back to use dev->enetaddr, which could be filled up by
the ethernet device driver.
Actually, this is the original behavior, and was later changed by
commit 48506a2cde2458fa1f8c5993afc98e5a4617e1d3.
Signed-off-by: Eric Miao <eric.miao@linaro.org>
Acked-by: Simon Glass <sjg@chromium.org>
| -rw-r--r-- | net/eth.c | 3 |
1 files changed, 1 insertions, 2 deletions
@@ -190,8 +190,7 @@ int eth_write_hwaddr(struct eth_device *dev, const char *base_name, unsigned char env_enetaddr[6]; int ret = 0; - if (!eth_getenv_enetaddr_by_index(base_name, eth_number, env_enetaddr)) - return -1; + eth_getenv_enetaddr_by_index(base_name, eth_number, env_enetaddr); if (memcmp(env_enetaddr, "\0\0\0\0\0\0", 6)) { if (memcmp(dev->enetaddr, "\0\0\0\0\0\0", 6) && |