[U-Boot] [PATCH v2 10/21] net: cosmetic: Un-typedef Ethernet_t
Simon Glass
sjg at chromium.org
Fri Apr 27 02:12:49 CEST 2012
Hi Joe,
On Wed, Mar 28, 2012 at 12:42 PM, Joe Hershberger <joe.hershberger at ni.com>wrote:
> Separate the Ethernet header from the 802 header.
> Base the size constants on the structs.
>
> Signed-off-by: Joe Hershberger <joe.hershberger at ni.com>
> Cc: Joe Hershberger <joe.hershberger at gmail.com>
> Cc: Simon Glass <sjg at chromium.org>
> Cc: Mike Frysinger <vapier at gentoo.org>
>
How about struct ethernet_hdr? Otherwise it looks good.
Just an aside - I assume that the sizeof() does not word align the result
on any architectures? In other words, does sizeof(struct Ethernet_hdr)
definitely give you 14 instead of 16 in all cases? I am sure this might be
right, or it would have broken things horribly.
Regards,
Simon
> ---
> Changes for v2:
> - Split apart from "Un-typedef variables in net"
> - Created a new E802_hdr struct and removed 802 members from Ethernet_hdr
> - Renamed structs from *_t to *_hdr since they are nolonger types
> - Replaced offsetof(Ethernet_t, et_dsap) with ETHER_HDR_SIZE in
> arch/powerpc/cpu/mpc8260/ether_fcc.c
>
More information about the U-Boot
mailing list