[U-Boot] [PATCH 12/18] GCC4.6: Squash warnings in smsc95xx.c
Wolfgang Denk
wd at denx.de
Sun Oct 30 23:03:44 CET 2011
Dear Marek Vasut,
In message <201110302148.50367.marek.vasut at gmail.com> you wrote:
>
> > > - addr_lo = cpu_to_le32(*((u32 *)eth->enetaddr));
> > > + addr_lo = cpu_to_le32(*eth->enetaddr);
> >
> > pretty sure this is wrong. enetaddr is a uchar[], so your code now reads
> > only 1 byte instead of 4.
> >
> > that said, this code also seems to not be endian safe ...
> > -mike
>
> It's good anyone actually cares to properly review. Anyway, why does noone
> actually care to fix all the damn warnings in their drivers before submitting
> them in the first place ?!
GCC 4.6 has not been around that long. Many people use less
bleading-edge tool chains - for a good reason.
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
Whenever people agree with me, I always think I must be wrong.
- Oscar Wilde
More information about the U-Boot
mailing list