Author: stefanct
Date: Mon Jul 14 01:53:40 2014
New Revision: 1827
URL: http://flashrom.org/trac/flashrom/changeset/1827

Log:
Improve messages on failed erases with default verbosity.

Without this...
Erasing and writing flash chip... ERASE FAILED at 0x00001000! Expected=0xff, 
Read=0xb4, failed byte count from 0x00000000-0x0000ffff: 0xef09
ERASE FAILED!
Reading current flash chip contents... done.
<loooooong break while the next eraser and writing is tried>
Erase/write done.
Verifying flash... VERIFIED.

Even if there is not a long temporal pause, it is very confusing for
the user to first see a failed erase, followed by a read, a done
message and eventually the verification message.
This patch inserts "Looking for another erase function." where there is
just a silent pause above.

Signed-off-by: Stefan Tauner <[email protected]>
Acked-by: Stefan Tauner <[email protected]>

Modified:
   trunk/flashrom.c

Modified: trunk/flashrom.c
==============================================================================
--- trunk/flashrom.c    Sun Jul 13 19:06:11 2014        (r1826)
+++ trunk/flashrom.c    Mon Jul 14 01:53:40 2014        (r1827)
@@ -1463,9 +1463,9 @@
 
        for (k = 0; k < NUM_ERASEFUNCTIONS; k++) {
                if (k != 0)
-                       msg_cdbg("Looking for another erase function.\n");
+                       msg_cinfo("Looking for another erase function.\n");
                if (!usable_erasefunctions) {
-                       msg_cdbg("No usable erase functions left.\n");
+                       msg_cinfo("No usable erase functions left.\n");
                        break;
                }
                msg_cdbg("Trying erase function %i... ", k);

_______________________________________________
flashrom mailing list
[email protected]
http://www.flashrom.org/mailman/listinfo/flashrom

Reply via email to