[U-Boot] [PATCH] e1000: fix bugs from recent commits
Marek Vasut
marek.vasut at gmail.com
Fri Oct 28 23:06:59 CEST 2011
> Commit 114d7fc0 "e1000: Rewrite EEPROM checksum error to give more
> information" failed to initialize the checksum variable which should
> result in random results. Fix that.
> [I wonder if that code has _ever_ been tested!!]
>
> Commit 2326a94d caused a ton of "unused variable 'x'" warnings.
> Fix these. While we are at it, remove some bogus parens.
>
> Signed-off-by: Wolfgang Denk <wd at denx.de>
> Cc: Kyle Moffett <Kyle.D.Moffett at boeing.com>
> ---
>
> Kyle,
>
> I wonder if you have ever actually build and run this code???
> With the "checksum" variable being random (due to not being
> initialized) you should have seen serious checksum problems.
> How did this escape your testing?
>
> And all these build warnings - have you ever actully compiled that
> code? What's going on here??? - wd
>
> Marek: Could you please be so kind and have a look at the debug code?
> I think this needs a major cleanup, too. Thanks in advance. - wd
I might take a sneak peek ... -mv
More information about the U-Boot
mailing list