On Tue, May 05, 2015 at 09:17:33AM +0200, Albert ARIBAUD wrote:
> Bonjour Simon,
> 
> Le Mon, 4 May 2015 15:37:22 -0600, Simon Glass <s...@chromium.org> a
> écrit :
> 
> > On 4 May 2015 at 13:55, Joe Hershberger <joe.hershber...@ni.com> wrote:
> > > We really don't want boards defining fixed MAC addresses in their config
> > > so we just remove the option to set it in a fixed way. If you must have
> > > a MAC address that was not provisioned, then use the random MAC address
> > > functionality.
> > >
> > > Signed-off-by: Joe Hershberger <joe.hershber...@ni.com>
> > > ---
> > 
> > Reviewed-by: Simon Glass <s...@chromium.org>
> 
> As much as I support the idea that one should not hard-code MAC
> addresses, I also support the idea that changes should not break
> previous behavior.
> 
> And regarding the WORK board which I submitted, its configuration
> header file had this address encoded in the in-house U-Boot which
> the customer was using before; removing this address might lead to
> the customer provisioning scripts failing.

Wait, no.  The policy for a long long time has been that you don't
hard-code mac addresses into the environment, at least in mainline.
"Factory" programming or provisioning or whatever it wants to be called
always ends up needing to do something crazy.  I'm all in favor of
documenting best practices and maybe even non-default configurations
that show how to do this, but the normal build for a given board needs
to be otherwise sane.

-- 
Tom

Attachment: signature.asc
Description: Digital signature

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

Reply via email to