[coreboot] [PATCH] flashrom patch for partial flash read #2
Stephan GUILLOUX
stephan.guilloux at free.fr
Wed Dec 3 22:23:10 CET 2008
After partial review, the following fixes are included :
- return used without parentheses.
I prefer parentheses, as it allows macros around. For instance, I
oftenly use macros like FUNCTION_RETURN(xx)
do dump the return code at debug time. Not the case here, so, ()
removed.
- unsigned int total_size.
errr. yes ;-)
- verification error message changed to
Error: verification aborted because of flash read failure.
- length > total_size
length == total_size is allowed !.
Anything else ?
Stephan.
More information about the coreboot
mailing list