[U-Boot] [PATCH 2/4] net: tweak eth_device layout to simplify enetaddr use

Wolfgang Denk wd at denx.de
Mon Dec 5 22:06:20 CET 2011


Dear Mike Frysinger,

In message <201111111003.15436.vapier at gentoo.org> you wrote:
>
> > I'm OK with expanding the name[] field, but as Thomas pointed out,
> > providing "convenient" u32 / u16 variables for the MAC address is
> > actually a faux ami that misleads people into using these variables
> > without thinking about endianess and such.
> > 
> > Please omit this part.
> 
> there's always the endian issue.  i'd wager that the vast majority of the 
> time, the endian of the target hardware is the same as the core.  so omitting 
> this for odd devices or device driver writers who aren't careful is being too 
> pessimistic imo.  i can add qualifiers to the name like "enetaddr_cpu32" if you 

No. Please drop this.

> want.  looking at the generated code shows really nice improvements: a single 
> cpu load instead of 4 loads interspersed with bitshifts.

This is neither memory nor performance critical, but it is easy to
misuse and thus dangerous.

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
There's no honorable way to kill, no gentle way to destroy.  There is
nothing good in war.  Except its ending.
	-- Abraham Lincoln, "The Savage Curtain", stardate 5906.5


More information about the U-Boot mailing list