Dear Rob Herring, In message <1328138854-28612-1-git-send-email-robherri...@gmail.com> you wrote: > > --- a/doc/README.enetaddr > +++ b/doc/README.enetaddr > @@ -32,7 +32,9 @@ Correct flow of setting up the MAC address (summarized): > > 1. Read from hardware in initialize() function > 2. Read from environment in net/eth.c after initialize() > -3. Give priority to the value in the environment if a conflict > +3. Write value to environment if setup in struct eth_device->enetaddr by > driver > + initialize() function. Give priority to the value in the environment if a > + conflict.
Sorry, but this description is not correct. You say here that the environment variable should always be written, but this is not the case. Only if it does not exist it shall be set. If it exists, it shall only be read, and in case of inconsistencies a warning shall be printed. 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: w...@denx.de Too many people are ready to carry the stool when the piano needs to be moved. _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
