[U-Boot] [PATCH 3/5] smc911x: silence MAC mismatch warning
Mike Frysinger
vapier at gentoo.org
Wed Nov 11 16:22:14 CET 2009
On Wednesday 11 November 2009 03:03:02 Mike Rapoport wrote:
> If there is no SROM attached to the SMSC chip it's MAC address is
> initialized to ff:ff:ff:ff:ff:ff and it causes the following
> warning:
>
> Warning: smc911x-0 MAC addresses don't match:
> Address in SROM is ff:ff:ff:ff:ff:ff
> Address in environment is 00:01:ba:dc:0d:03
>
> Set dev->enetaddr only if MAC address is valid, and thus avoid the
> above case.
someone already posted a patch for this issue:
NET: Fix MAC addr handling for smc911x
i think the approach they took is better -- they check for explicit values
that indicate 'no srom is attached' rather than 'is the mac valid' as your
code could ignore attached srom's but bad addresses. in this latter case, i
think we want the warning.
-mike
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 836 bytes
Desc: This is a digitally signed message part.
Url : http://lists.denx.de/pipermail/u-boot/attachments/20091111/e283b240/attachment.pgp
More information about the U-Boot
mailing list