nanddump: fail if -s parameter is unaligned

Implement the feature which we planned long time ago - make nanddump
fail if the -s parameter is not NAND page-aligned. Also bump nanddump
version.

Signed-off-by: Artem Bityutskiy <Artem.Bityutskiy@nokia.com>

diff --git a/feature-removal-schedule.txt b/feature-removal-schedule.txt
index 2899e34b1c28a25840c87286ea94422a3ce5ad0e..30f0403ce19256a00151e218124219dfcd620a08 100644 (file)
--- a/feature-removal-schedule.txt
+++ b/feature-removal-schedule.txt
@@ -26,15 +26,5 @@ to stop using them.
 The further step is to remove both of them.
 
 ---------------------------
-2. nanddump: fail when the -s parameter is not page aligned
-
-nanddump should be consistent with nandwrite, and refuse accepting non page
-aligned start addresses. These dumps are most likely useless, since nandwrite
-won't write then at this location. Currently only a warning is issued to keep
-backward compatibility during a transition period. This should become an error
-one release after the next release (current is mtd-utils-1.4.0, 18 Sep 2010).
-
-To be done by Artem Bityutskiy.
----------------------------
-3.
+2.
 ---------------------------

diff --git a/nanddump.c b/nanddump.c
index 1dbb2474c6b14dc161c5c5f48a7efff9df25a944..214fb1236607d98c394c866cc10efe56a734495c 100644 (file)
--- a/nanddump.c
+++ b/nanddump.c
@@ -14,7 +14,7 @@
  */
 
 #define PROGRAM_NAME "nanddump"
-#define VERSION "$Revision: 1.29 $"
+#define VERSION "1.30"
 
 #define _GNU_SOURCE
 #include <ctype.h>
@@ -366,11 +366,10 @@ int main(int argc, char * const argv[])
 
        /* Initialize start/end addresses and block size */
        if (start_addr & (mtd.min_io_size - 1)) {
-               fprintf(stderr, "WARNING: The start address is not page-aligned !\n"
-                               "The pagesize of this NAND Flash is 0x%x.\n"
-                               "nandwrite doesn't allow writes starting at this location.\n"
-                               "Future versions of nanddump will fail here.\n",
+               fprintf(stderr, "the start address (-s parameter) is not page-aligned!\n"
+                               "The pagesize of this NAND Flash is 0x%x.\n",
                                mtd.min_io_size);
+               goto closeall;
        }
        if (length)
                end_addr = start_addr + length;