Jean-Christophe PLAGNIOL-VILLARD wrote:
> On 15:49 Sun 12 Apr     , David Brownell wrote:
>   
>> From: David Brownell <dbrown...@users.sourceforge.net>
>>
>> Chips without the EMAC controller won't need the utilities
>> it uses to read an Ethernet address from EEPROM; so don't 
>> include them needlessly.
>>
>> Use is_valid_ether() to validate the address from EEPROM.
>> All-zero addresses aren't the only invalid addresses.
>> A fully erased EEPROM returns all-ones, also invalid...
>>
>> Switch those Ethernet utilities to use "%pM" for printing
>> MAC addresses; and not say ROM when they mean EEPROM.
>>
>> Signed-off-by: David Brownell <dbrown...@users.sourceforge.net>
>>     
Acked-by: Ben Warren <biggerbadder...@gmail.com>
> Looks fine
>
> Ben any comments?
>
> Best Regards,
> J.
>   
Please pull this in with the other changes.

regards,
Ben
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Reply via email to