Dear Jiri Prchal,
In message <[email protected]> you
wrote:
...
> +err_spi_setup_slave:
> + if (!is_valid_ether_addr(sernum)) {
> + debug("Using random MAC address\n");
> + eth_random_enetaddr(sernum);
> + }
Pleas change this debug() into a puts() [or rather a printf() and also
print the value.] The user is supposed to know when something goes
badly wrong and random addresses get used, and what these are.
> + eth_setenv_enetaddr("ethaddr", sernum);
NAK. Here you unconditionally overwrite the MAC address, even if one
is already present in the environment. This must never be done. If
the user defines an address, shis shall always be respected.
Thanks.
Best regards,
Wolfgang Denk
--
DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: [email protected]
Unser Kopf ist rund, damit das Denken die Richtung wechseln kann.
-- Francis Picabia
_______________________________________________
U-Boot mailing list
[email protected]
http://lists.denx.de/mailman/listinfo/u-boot
