From mboxrd@z Thu Jan 1 00:00:00 1970 From: Eric Nelson Date: Wed, 21 Aug 2013 06:28:14 -0700 Subject: [U-Boot] [PATCH] fec_mxc: set ethaddr if fuses burned and not previously set In-Reply-To: <5214A23F.2070108@denx.de> References: <1375465020-20058-1-git-send-email-eric.nelson@boundarydevices.com> <5214A23F.2070108@denx.de> Message-ID: <5214C06E.6010406@boundarydevices.com> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de On 08/21/2013 04:19 AM, Stefano Babic wrote: > Hi Eric, > > On 02/08/2013 19:37, Eric Nelson wrote: >> Without this change, the following message is generated: >> Warning: FEC using MAC address from net device >> >> See doc/README.enetaddr for details. >> >> Signed-off-by: Eric Nelson >> --- >> drivers/net/fec_mxc.c | 2 ++ >> 1 file changed, 2 insertions(+) >> >> diff --git a/drivers/net/fec_mxc.c b/drivers/net/fec_mxc.c >> index e14a359..886be9c 100644 >> --- a/drivers/net/fec_mxc.c >> +++ b/drivers/net/fec_mxc.c >> @@ -980,6 +980,8 @@ static int fec_probe(bd_t *bd, int dev_id, uint32_t base_addr, >> if (fec_get_hwaddr(edev, dev_id, ethaddr) == 0) { >> debug("got MAC%d address from fuse: %pM\n", dev_id, ethaddr); >> memcpy(edev->enetaddr, ethaddr, 6); >> + if (!getenv("ethaddr")) >> + eth_setenv_enetaddr("ethaddr",mac); > ^-- > of course, this should be ethaddr and not mac. I fixed it myself. > Thanks Stefano.