[U-Boot] [PATCH] net: remove scary warning about EEPROM provided MAC address

Joe Hershberger joe.hershberger at gmail.com
Tue Jan 26 16:02:23 CET 2016


On Mon, Jan 4, 2016 at 9:48 AM, Andre Przywara <andre.przywara at arm.com> wrote:
> In many parts of the computing world having a unique MAC address
> sitting in some on-NIC storage is considered the normal case.
> Remove the warning to not scare the user unnecessarily.
> This applies to Highbank/Midway and ARM's Juno, for instance.
> Besides that this fixes the formatting on Midway, for instance,
> which currently looks like:
> ...
> Net:   xgmac0
> Warning: xgmac0 using MAC address from net device
> , xgmac1
> Warning: xgmac1 using MAC address from net device
> ...
>
> Signed-off-by: Andre Przywara <andre.przywara at arm.com>
> ---
>  net/eth.c | 2 --
>  1 file changed, 2 deletions(-)
>
> diff --git a/net/eth.c b/net/eth.c
> index 6cf3a35..a1b8abc 100644
> --- a/net/eth.c
> +++ b/net/eth.c
> @@ -761,8 +761,6 @@ int eth_write_hwaddr(struct eth_device *dev, const char *base_name,
>         } else if (is_valid_ethaddr(dev->enetaddr)) {
>                 eth_setenv_enetaddr_by_index(base_name, eth_number,
>                                              dev->enetaddr);
> -               printf("\nWarning: %s using MAC address from net device\n",
> -                      dev->name);

Guess this only changes the legacy case. The DM case still warns.

-Joe

>         } else if (is_zero_ethaddr(dev->enetaddr)) {
>  #ifdef CONFIG_NET_RANDOM_ETHADDR
>                 net_random_ethaddr(dev->enetaddr);
> --
> 2.6.4
>
> _______________________________________________
> U-Boot mailing list
> U-Boot at lists.denx.de
> http://lists.denx.de/mailman/listinfo/u-boot


More information about the U-Boot mailing list