]> www.infradead.org Git - mtd-utils.git/commitdiff
nanddump: change default to --bb=skipbad
authorBrian Norris <computersforpeace@gmail.com>
Fri, 24 Jun 2011 18:03:10 +0000 (11:03 -0700)
committerArtem Bityutskiy <Artem.Bityutskiy@nokia.com>
Sat, 25 Jun 2011 07:01:39 +0000 (10:01 +0300)
As promised, we change the default bad block handling method to
--bb=skipbad. This works as a better inverse to nandwrite, since
nandwrite skips bad blocks when writing data.

And of course, we remove the warning messages.

Signed-off-by: Brian Norris <computersforpeace@gmail.com>
Signed-off-by: Artem Bityutskiy <Artem.Bityutskiy@nokia.com>
nanddump.c

index 55c5586ebeeddc4c022addf66395cbf89acf47dd..9edfbfd39b278dfcc8f1645cba7f4d6d53509d03 100644 (file)
@@ -60,9 +60,9 @@ static void display_help(void)
 "-s addr    --startaddress=addr  Start address\n"
 "\n"
 "--bb=METHOD, where METHOD can be `padbad', `dumpbad', or `skipbad':\n"
-"    padbad:  dump flash data, substituting 0xFF for any bad blocks (default)\n"
+"    padbad:  dump flash data, substituting 0xFF for any bad blocks\n"
 "    dumpbad: dump flash data, including any bad blocks\n"
-"    skipbad: dump good data, completely skipping any bad blocks\n"
+"    skipbad: dump good data, completely skipping any bad blocks (default)\n"
 "\n"
 "Note on --oob, --omitoob:\n"
 "  To make nanddump act more like an inverse to nandwrite, we are changing\n"
@@ -105,7 +105,7 @@ static enum {
        padbad,   // dump flash data, substituting 0xFF for any bad blocks
        dumpbad,  // dump flash data, including any bad blocks
        skipbad,  // dump good data, completely skipping any bad blocks
-} bb_method = padbad;
+} bb_method = skipbad;
 
 static void process_options(int argc, char * const argv[])
 {
@@ -229,11 +229,6 @@ static void process_options(int argc, char * const argv[])
                exit(EXIT_FAILURE);
        }
 
-       if (bb_default)
-               warnmsg("you did not specify a default bad-block handling\n"
-                       "  method. In future versions, the default will change to\n"
-                       "  --bb=skipbad. Use \"nanddump --help\" for more information.");
-
        if (oob_default)
                warnmsg("in next release, nanddump will not dump OOB\n"
                        "  by default. Use `nanddump --oob' explicitly to ensure\n"