On Tue, July 24, 2012 01:11, Joe Hershberger wrote:
> NetConsole may call NetSendUDPPacket before NetLoop is called.  This
> will cause the source MAC address (NetOurEther) to be wrong.  Instead
> of only changing it in NetLoop, move it to NetLoopInit so that it is
> also updated when net_init() is called (especially by nc_start()).
>
> Signed-off-by: Joe Hershberger <joe.hershber...@ni.com>
> Reported-by: Michael Walle <mich...@walle.cc>
> ---
>  net/net.c |    3 +--
>  1 files changed, 1 insertions(+), 2 deletions(-)
>
> diff --git a/net/net.c b/net/net.c
> index 9de7d92..e8ff066 100644
> --- a/net/net.c
> +++ b/net/net.c
> @@ -256,6 +256,7 @@ static void NetInitLoop(void)
>  #endif
>               env_changed_id = env_id;
>       }
> +     memcpy(NetOurEther, eth_get_dev()->enetaddr, 6);
>
>       return;
>  }

This was exactly my (quick) fix, too. Therefore,

Acked-by: Michael Walle <mich...@walle.cc>

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

Reply via email to