[U-Boot] [PATCH 22/27] pcs440ep: get mac address from environment and move load_sernum_ethaddr() to board init

Wolfgang Denk wd at denx.de
Mon Feb 16 23:57:22 CET 2009


Dear Mike Frysinger,

In message <1234596190-524-23-git-send-email-vapier at gentoo.org> you wrote:
> The environment is the canonical storage location of the mac address, so
> we're killing off the global data location and moving everything to
> querying the env directly.
> 
> Move all the work of load_sernum_ethaddr() into a local load_ethaddr()
> func and call that from misc_init_r().  We leave load_sernum_ethaddr()
> as a stub so that it can be removed from all places in a later commit.
> 
> Signed-off-by: Mike Frysinger <vapier at gentoo.org>
> CC: Stefan Roese <sr at denx.de>
> CC: Ben Warren <biggerbadderben at gmail.com>
...
> +void load_sernum_ethaddr (void)
> +{
> +}
> +

See previous comments - please do not add dead code just to remove it
later.

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: wd at denx.de
I think it's a new feature. Don't tell anyone it was an accident. :-)
  -- Larry Wall on s/foo/bar/eieio in <10911 at jpl-devvax.JPL.NASA.GOV>


More information about the U-Boot mailing list