[U-Boot] [PATCH] omap3evm: Set environment variable 'ethaddr'
Premi, Sanjeev
premi at ti.com
Sat Sep 3 09:28:59 CEST 2011
> From: Wolfgang Denk [wd at denx.de]
> Sent: Saturday, September 03, 2011 3:37 AM
> To: Premi, Sanjeev
> Cc: u-boot at lists.denx.de
> Subject: Re: [U-Boot] [PATCH] omap3evm: Set environment variable 'ethaddr'
>
> Dear "Premi, Sanjeev",
>
> In message <B85A65D85D7EB246BE421B3FB0FBB5930257371E07 at dbde02.ent.ti.com> you wrote:
> >
> > > Your patch always and unconditionally overwrites any existing
> > > "ethaddr" settings. This is not acceptable.
> >
> > For the EVM, MAC address is always probed from the chip. Hence, I
> > assumed it safe to set the ethaddr - without checking for env var.
>
> This is not what we do in U-Boot, so please fix that.
>
> > It was unlikely that someone would be setting it - to different
> > value.
>
> Unlikely? Not so. This is expected standard behaviour. Please don't
> try to invent a wheel (in a different shape).
[sp] I agreed with your comments in the next para. Had only
described my thoughts before creating this patch.
You seem to have missed the v2 of the same patch where I
believe I have taken care of this behavior:
http://marc.info/?l=u-boot&m=131497907619918&w=2
Do let me know if this is better...
~sanjeev
>
> Best regards,
>
> Wolfgang Denk
More information about the U-Boot
mailing list