On Thu, Jul 21, 2011 at 05:54:46PM +0200, Stefan Tauner wrote:
> previously the dummies were initialized to be empty (all ones), which makes
> writes skip
> erasing altogether. with this patch the default is to fill it with random
> bytes instead and the
> old behavior can be enforced with stating "empty=yes" on the command line.
>
> Signed-off-by: Stefan Tauner <[email protected]>
I'd make the programmer options a bit more generic to allow for other
content easily and consistently:
-p dummy:content=ff
-p dummy:content=00
-p dummy:content=random
-p dummy:content=incrementing
etc.
The default should be documented in the manpage.
> + empty = extract_programmer_param("empty");
> + if (empty != NULL) {
> + if (strstr(empty, "yes"))
Why not strcasecmp() here?
Uwe.
--
http://hermann-uwe.de | http://sigrok.org
http://randomprojects.org | http://unmaintained-free-software.org
_______________________________________________
flashrom mailing list
[email protected]
http://www.flashrom.org/mailman/listinfo/flashrom