Merge branch '2022-08-04-Kconfig-migrations'
[platform/kernel/u-boot.git] / cmd / mem.c
index 0bfb608..1f4e3fc 100644 (file)
--- a/cmd/mem.c
+++ b/cmd/mem.c
 #include <cli.h>
 #include <command.h>
 #include <console.h>
+#ifdef CONFIG_MTD_NOR_FLASH
 #include <flash.h>
+#endif
 #include <hash.h>
+#include <log.h>
 #include <mapmem.h>
+#include <rand.h>
 #include <watchdog.h>
+#include <asm/global_data.h>
 #include <asm/io.h>
+#include <linux/bitops.h>
 #include <linux/compiler.h>
+#include <linux/ctype.h>
+#include <linux/delay.h>
 
 DECLARE_GLOBAL_DATA_PTR;
 
-#ifndef CONFIG_SYS_MEMTEST_SCRATCH
-#define CONFIG_SYS_MEMTEST_SCRATCH 0
+/* Create a compile-time value */
+#ifdef MEM_SUPPORT_64BIT_DATA
+#define SUPPORT_64BIT_DATA 1
+#define HELP_Q ", .q"
+#else
+#define SUPPORT_64BIT_DATA 0
+#define HELP_Q ""
 #endif
 
-static int mod_mem(cmd_tbl_t *, int, int, int, char * const []);
+static int mod_mem(struct cmd_tbl *, int, int, int, char * const []);
 
 /* Display values from last command.
  * Memory modify remembered values are different from display memory.
@@ -39,6 +52,11 @@ static ulong dp_last_length = 0x40;
 static ulong   mm_last_addr, mm_last_size;
 
 static ulong   base_address = 0;
+#ifdef CONFIG_CMD_MEM_SEARCH
+static ulong dp_last_ms_length;
+static u8 search_buf[64];
+static uint search_len;
+#endif
 
 /* Memory Display
  *
@@ -46,7 +64,8 @@ static        ulong   base_address = 0;
  *     md{.b, .w, .l, .q} {addr} {len}
  */
 #define DISP_LINE_LEN  16
-static int do_mem_md(cmd_tbl_t *cmdtp, int flag, int argc, char * const argv[])
+static int do_mem_md(struct cmd_tbl *cmdtp, int flag, int argc,
+                    char *const argv[])
 {
        ulong   addr, length, bytes;
        const void *buf;
@@ -72,14 +91,14 @@ static int do_mem_md(cmd_tbl_t *cmdtp, int flag, int argc, char * const argv[])
 
                /* Address is specified since argc > 1
                */
-               addr = simple_strtoul(argv[1], NULL, 16);
+               addr = hextoul(argv[1], NULL);
                addr += base_address;
 
                /* If another parameter, it is the length to display.
                 * Length is the number of objects, not number of bytes.
                 */
                if (argc > 2)
-                       length = simple_strtoul(argv[2], NULL, 16);
+                       length = hextoul(argv[2], NULL);
        }
 
        bytes = size * length;
@@ -96,22 +115,22 @@ static int do_mem_md(cmd_tbl_t *cmdtp, int flag, int argc, char * const argv[])
        return (rc);
 }
 
-static int do_mem_mm(cmd_tbl_t *cmdtp, int flag, int argc, char * const argv[])
+static int do_mem_mm(struct cmd_tbl *cmdtp, int flag, int argc,
+                    char *const argv[])
 {
        return mod_mem (cmdtp, 1, flag, argc, argv);
 }
-static int do_mem_nm(cmd_tbl_t *cmdtp, int flag, int argc, char * const argv[])
+
+static int do_mem_nm(struct cmd_tbl *cmdtp, int flag, int argc,
+                    char *const argv[])
 {
        return mod_mem (cmdtp, 0, flag, argc, argv);
 }
 
-static int do_mem_mw(cmd_tbl_t *cmdtp, int flag, int argc, char * const argv[])
+static int do_mem_mw(struct cmd_tbl *cmdtp, int flag, int argc,
+                    char *const argv[])
 {
-#ifdef MEM_SUPPORT_64BIT_DATA
-       u64 writeval;
-#else
-       ulong writeval;
-#endif
+       ulong writeval;  /* 64-bit if SUPPORT_64BIT_DATA */
        ulong   addr, count;
        int     size;
        void *buf, *start;
@@ -127,20 +146,19 @@ static int do_mem_mw(cmd_tbl_t *cmdtp, int flag, int argc, char * const argv[])
 
        /* Address is specified since argc > 1
        */
-       addr = simple_strtoul(argv[1], NULL, 16);
+       addr = hextoul(argv[1], NULL);
        addr += base_address;
 
        /* Get the value to write.
        */
-#ifdef MEM_SUPPORT_64BIT_DATA
-       writeval = simple_strtoull(argv[2], NULL, 16);
-#else
-       writeval = simple_strtoul(argv[2], NULL, 16);
-#endif
+       if (SUPPORT_64BIT_DATA)
+               writeval = simple_strtoull(argv[2], NULL, 16);
+       else
+               writeval = hextoul(argv[2], NULL);
 
        /* Count ? */
        if (argc == 4) {
-               count = simple_strtoul(argv[3], NULL, 16);
+               count = hextoul(argv[3], NULL);
        } else {
                count = 1;
        }
@@ -151,10 +169,8 @@ static int do_mem_mw(cmd_tbl_t *cmdtp, int flag, int argc, char * const argv[])
        while (count-- > 0) {
                if (size == 4)
                        *((u32 *)buf) = (u32)writeval;
-#ifdef MEM_SUPPORT_64BIT_DATA
-               else if (size == 8)
-                       *((u64 *)buf) = (u64)writeval;
-#endif
+               else if (SUPPORT_64BIT_DATA && size == 8)
+                       *((ulong *)buf) = writeval;
                else if (size == 2)
                        *((u16 *)buf) = (u16)writeval;
                else
@@ -166,7 +182,8 @@ static int do_mem_mw(cmd_tbl_t *cmdtp, int flag, int argc, char * const argv[])
 }
 
 #ifdef CONFIG_CMD_MX_CYCLIC
-static int do_mem_mdc(cmd_tbl_t *cmdtp, int flag, int argc, char * const argv[])
+static int do_mem_mdc(struct cmd_tbl *cmdtp, int flag, int argc,
+                     char *const argv[])
 {
        int i;
        ulong count;
@@ -174,14 +191,14 @@ static int do_mem_mdc(cmd_tbl_t *cmdtp, int flag, int argc, char * const argv[])
        if (argc < 4)
                return CMD_RET_USAGE;
 
-       count = simple_strtoul(argv[3], NULL, 10);
+       count = dectoul(argv[3], NULL);
 
        for (;;) {
                do_mem_md (NULL, 0, 3, argv);
 
                /* delay for <count> ms... */
                for (i=0; i<count; i++)
-                       udelay (1000);
+                       udelay(1000);
 
                /* check for ctrl-c to abort... */
                if (ctrlc()) {
@@ -193,7 +210,8 @@ static int do_mem_mdc(cmd_tbl_t *cmdtp, int flag, int argc, char * const argv[])
        return 0;
 }
 
-static int do_mem_mwc(cmd_tbl_t *cmdtp, int flag, int argc, char * const argv[])
+static int do_mem_mwc(struct cmd_tbl *cmdtp, int flag, int argc,
+                     char *const argv[])
 {
        int i;
        ulong count;
@@ -201,14 +219,14 @@ static int do_mem_mwc(cmd_tbl_t *cmdtp, int flag, int argc, char * const argv[])
        if (argc < 4)
                return CMD_RET_USAGE;
 
-       count = simple_strtoul(argv[3], NULL, 10);
+       count = dectoul(argv[3], NULL);
 
        for (;;) {
                do_mem_mw (NULL, 0, 3, argv);
 
                /* delay for <count> ms... */
                for (i=0; i<count; i++)
-                       udelay (1000);
+                       udelay(1000);
 
                /* check for ctrl-c to abort... */
                if (ctrlc()) {
@@ -221,18 +239,15 @@ static int do_mem_mwc(cmd_tbl_t *cmdtp, int flag, int argc, char * const argv[])
 }
 #endif /* CONFIG_CMD_MX_CYCLIC */
 
-static int do_mem_cmp(cmd_tbl_t *cmdtp, int flag, int argc, char * const argv[])
+static int do_mem_cmp(struct cmd_tbl *cmdtp, int flag, int argc,
+                     char *const argv[])
 {
        ulong   addr1, addr2, count, ngood, bytes;
        int     size;
        int     rcode = 0;
        const char *type;
        const void *buf1, *buf2, *base;
-#ifdef MEM_SUPPORT_64BIT_DATA
-       u64 word1, word2;
-#else
-       ulong word1, word2;
-#endif
+       ulong word1, word2;  /* 64-bit if SUPPORT_64BIT_DATA */
 
        if (argc != 4)
                return CMD_RET_USAGE;
@@ -245,13 +260,13 @@ static int do_mem_cmp(cmd_tbl_t *cmdtp, int flag, int argc, char * const argv[])
               size == 4 ? "word" :
               size == 2 ? "halfword" : "byte";
 
-       addr1 = simple_strtoul(argv[1], NULL, 16);
+       addr1 = hextoul(argv[1], NULL);
        addr1 += base_address;
 
-       addr2 = simple_strtoul(argv[2], NULL, 16);
+       addr2 = hextoul(argv[2], NULL);
        addr2 += base_address;
 
-       count = simple_strtoul(argv[3], NULL, 16);
+       count = hextoul(argv[3], NULL);
 
        bytes = size * count;
        base = buf1 = map_sysmem(addr1, bytes);
@@ -260,11 +275,9 @@ static int do_mem_cmp(cmd_tbl_t *cmdtp, int flag, int argc, char * const argv[])
                if (size == 4) {
                        word1 = *(u32 *)buf1;
                        word2 = *(u32 *)buf2;
-#ifdef MEM_SUPPORT_64BIT_DATA
-               } else if (size == 8) {
-                       word1 = *(u64 *)buf1;
-                       word2 = *(u64 *)buf2;
-#endif
+               } else if (SUPPORT_64BIT_DATA && size == 8) {
+                       word1 = *(ulong *)buf1;
+                       word2 = *(ulong *)buf2;
                } else if (size == 2) {
                        word1 = *(u16 *)buf1;
                        word2 = *(u16 *)buf2;
@@ -274,15 +287,9 @@ static int do_mem_cmp(cmd_tbl_t *cmdtp, int flag, int argc, char * const argv[])
                }
                if (word1 != word2) {
                        ulong offset = buf1 - base;
-#ifdef MEM_SUPPORT_64BIT_DATA
-                       printf("%s at 0x%p (%#0*llx) != %s at 0x%p (%#0*llx)\n",
-                              type, (void *)(addr1 + offset), size, word1,
-                              type, (void *)(addr2 + offset), size, word2);
-#else
                        printf("%s at 0x%08lx (%#0*lx) != %s at 0x%08lx (%#0*lx)\n",
                                type, (ulong)(addr1 + offset), size, word1,
                                type, (ulong)(addr2 + offset), size, word2);
-#endif
                        rcode = 1;
                        break;
                }
@@ -301,7 +308,8 @@ static int do_mem_cmp(cmd_tbl_t *cmdtp, int flag, int argc, char * const argv[])
        return rcode;
 }
 
-static int do_mem_cp(cmd_tbl_t *cmdtp, int flag, int argc, char * const argv[])
+static int do_mem_cp(struct cmd_tbl *cmdtp, int flag, int argc,
+                    char *const argv[])
 {
        ulong   addr, dest, count;
        void    *src, *dst;
@@ -315,13 +323,13 @@ static int do_mem_cp(cmd_tbl_t *cmdtp, int flag, int argc, char * const argv[])
        if ((size = cmd_get_data_size(argv[0], 4)) < 0)
                return 1;
 
-       addr = simple_strtoul(argv[1], NULL, 16);
+       addr = hextoul(argv[1], NULL);
        addr += base_address;
 
-       dest = simple_strtoul(argv[2], NULL, 16);
+       dest = hextoul(argv[2], NULL);
        dest += base_address;
 
-       count = simple_strtoul(argv[3], NULL, 16);
+       count = hextoul(argv[3], NULL);
 
        if (count == 0) {
                puts ("Zero length ???\n");
@@ -359,13 +367,154 @@ static int do_mem_cp(cmd_tbl_t *cmdtp, int flag, int argc, char * const argv[])
        return 0;
 }
 
-static int do_mem_base(cmd_tbl_t *cmdtp, int flag, int argc,
-                      char * const argv[])
+#ifdef CONFIG_CMD_MEM_SEARCH
+static int do_mem_search(struct cmd_tbl *cmdtp, int flag, int argc,
+                        char *const argv[])
+{
+       ulong addr, length, bytes, offset;
+       u8 *ptr, *end, *buf;
+       bool quiet = false;
+       ulong last_pos;         /* Offset of last match in 'size' units*/
+       ulong last_addr;        /* Address of last displayed line */
+       int limit = 10;
+       int used_len;
+       int count;
+       int size;
+       int i;
+
+       /* We use the last specified parameters, unless new ones are entered */
+       addr = dp_last_addr;
+       size = dp_last_size;
+       length = dp_last_ms_length;
+
+       if (argc < 3)
+               return CMD_RET_USAGE;
+
+       if (!(flag & CMD_FLAG_REPEAT)) {
+               /*
+                * Check for a size specification.
+                * Defaults to long if no or incorrect specification.
+                */
+               size = cmd_get_data_size(argv[0], 4);
+               if (size < 0 && size != CMD_DATA_SIZE_STR)
+                       return 1;
+
+               argc--;
+               argv++;
+               while (argc && *argv[0] == '-') {
+                       int ch = argv[0][1];
+
+                       if (ch == 'q')
+                               quiet = true;
+                       else if (ch == 'l' && isxdigit(argv[0][2]))
+                               limit = hextoul(argv[0] + 2, NULL);
+                       else
+                               return CMD_RET_USAGE;
+                       argc--;
+                       argv++;
+               }
+
+               /* Address is specified since argc > 1 */
+               addr = hextoul(argv[0], NULL);
+               addr += base_address;
+
+               /* Length is the number of objects, not number of bytes */
+               length = hextoul(argv[1], NULL);
+
+               /* Read the bytes to search for */
+               end = search_buf + sizeof(search_buf);
+               for (i = 2, ptr = search_buf; i < argc && ptr < end; i++) {
+                       if (MEM_SUPPORT_64BIT_DATA && size == 8) {
+                               u64 val = simple_strtoull(argv[i], NULL, 16);
+
+                               *(u64 *)ptr = val;
+                       } else if (size == -2) {  /* string */
+                               int len = min(strlen(argv[i]),
+                                             (size_t)(end - ptr));
+
+                               memcpy(ptr, argv[i], len);
+                               ptr += len;
+                               continue;
+                       } else {
+                               u32 val = hextoul(argv[i], NULL);
+
+                               switch (size) {
+                               case 1:
+                                       *ptr = val;
+                                       break;
+                               case 2:
+                                       *(u16 *)ptr = val;
+                                       break;
+                               case 4:
+                                       *(u32 *)ptr = val;
+                                       break;
+                               }
+                       }
+                       ptr += size;
+               }
+               search_len = ptr - search_buf;
+       }
+
+       /* Do the search */
+       if (size == -2)
+               size = 1;
+       bytes = size * length;
+       buf = map_sysmem(addr, bytes);
+       last_pos = 0;
+       last_addr = 0;
+       count = 0;
+       for (offset = 0;
+            offset < bytes && offset <= bytes - search_len && count < limit;
+            offset += size) {
+               void *ptr = buf + offset;
+
+               if (!memcmp(ptr, search_buf, search_len)) {
+                       uint align = (addr + offset) & 0xf;
+                       ulong match = addr + offset;
+
+                       if (!count || (last_addr & ~0xf) != (match & ~0xf)) {
+                               if (!quiet) {
+                                       if (count)
+                                               printf("--\n");
+                                       print_buffer(match - align, ptr - align,
+                                                    size,
+                                                    ALIGN(search_len + align,
+                                                          16) / size, 0);
+                               }
+                               last_addr = match;
+                               last_pos = offset / size;
+                       }
+                       count++;
+               }
+       }
+       if (!quiet) {
+               printf("%d match%s", count, count == 1 ? "" : "es");
+               if (count == limit)
+                       printf(" (repeat command to check for more)");
+               printf("\n");
+       }
+       env_set_hex("memmatches", count);
+       env_set_hex("memaddr", last_addr);
+       env_set_hex("mempos", last_pos);
+
+       unmap_sysmem(buf);
+
+       used_len = offset / size;
+       dp_last_addr = addr + used_len;
+       dp_last_size = size;
+       dp_last_ms_length = length < used_len ? 0 : length - used_len;
+
+       return count ? 0 : CMD_RET_FAILURE;
+}
+#endif
+
+static int do_mem_base(struct cmd_tbl *cmdtp, int flag, int argc,
+                      char *const argv[])
 {
        if (argc > 1) {
                /* Set new base address.
                */
-               base_address = simple_strtoul(argv[1], NULL, 16);
+               base_address = hextoul(argv[1], NULL);
        }
        /* Print the current base address.
        */
@@ -373,14 +522,12 @@ static int do_mem_base(cmd_tbl_t *cmdtp, int flag, int argc,
        return 0;
 }
 
-static int do_mem_loop(cmd_tbl_t *cmdtp, int flag, int argc,
-                      char * const argv[])
+static int do_mem_loop(struct cmd_tbl *cmdtp, int flag, int argc,
+                      char *const argv[])
 {
        ulong   addr, length, i, bytes;
        int     size;
-#ifdef MEM_SUPPORT_64BIT_DATA
-       volatile u64 *llp;
-#endif
+       volatile ulong *llp;  /* 64-bit if SUPPORT_64BIT_DATA */
        volatile u32 *longp;
        volatile u16 *shortp;
        volatile u8 *cp;
@@ -398,11 +545,11 @@ static int do_mem_loop(cmd_tbl_t *cmdtp, int flag, int argc,
 
        /* Address is always specified.
        */
-       addr = simple_strtoul(argv[1], NULL, 16);
+       addr = hextoul(argv[1], NULL);
 
        /* Length is the number of objects, not number of bytes.
        */
-       length = simple_strtoul(argv[2], NULL, 16);
+       length = hextoul(argv[2], NULL);
 
        bytes = size * length;
        buf = map_sysmem(addr, bytes);
@@ -411,13 +558,11 @@ static int do_mem_loop(cmd_tbl_t *cmdtp, int flag, int argc,
         * If we have only one object, just run infinite loops.
         */
        if (length == 1) {
-#ifdef MEM_SUPPORT_64BIT_DATA
-               if (size == 8) {
-                       llp = (u64 *)buf;
+               if (SUPPORT_64BIT_DATA && size == 8) {
+                       llp = (ulong *)buf;
                        for (;;)
                                i = *llp;
                }
-#endif
                if (size == 4) {
                        longp = (u32 *)buf;
                        for (;;)
@@ -433,16 +578,14 @@ static int do_mem_loop(cmd_tbl_t *cmdtp, int flag, int argc,
                        i = *cp;
        }
 
-#ifdef MEM_SUPPORT_64BIT_DATA
-       if (size == 8) {
+       if (SUPPORT_64BIT_DATA && size == 8) {
                for (;;) {
-                       llp = (u64 *)buf;
+                       llp = (ulong *)buf;
                        i = length;
                        while (i-- > 0)
                                *llp++;
                }
        }
-#endif
        if (size == 4) {
                for (;;) {
                        longp = (u32 *)buf;
@@ -471,17 +614,13 @@ static int do_mem_loop(cmd_tbl_t *cmdtp, int flag, int argc,
 }
 
 #ifdef CONFIG_LOOPW
-static int do_mem_loopw(cmd_tbl_t *cmdtp, int flag, int argc,
-                       char * const argv[])
+static int do_mem_loopw(struct cmd_tbl *cmdtp, int flag, int argc,
+                       char *const argv[])
 {
        ulong   addr, length, i, bytes;
        int     size;
-#ifdef MEM_SUPPORT_64BIT_DATA
-       volatile u64 *llp;
-       u64 data;
-#else
-       ulong   data;
-#endif
+       volatile ulong *llp;  /* 64-bit if SUPPORT_64BIT_DATA */
+       ulong   data;    /* 64-bit if SUPPORT_64BIT_DATA */
        volatile u32 *longp;
        volatile u16 *shortp;
        volatile u8 *cp;
@@ -499,18 +638,17 @@ static int do_mem_loopw(cmd_tbl_t *cmdtp, int flag, int argc,
 
        /* Address is always specified.
        */
-       addr = simple_strtoul(argv[1], NULL, 16);
+       addr = hextoul(argv[1], NULL);
 
        /* Length is the number of objects, not number of bytes.
        */
-       length = simple_strtoul(argv[2], NULL, 16);
+       length = hextoul(argv[2], NULL);
 
        /* data to write */
-#ifdef MEM_SUPPORT_64BIT_DATA
-       data = simple_strtoull(argv[3], NULL, 16);
-#else
-       data = simple_strtoul(argv[3], NULL, 16);
-#endif
+       if (SUPPORT_64BIT_DATA)
+               data = simple_strtoull(argv[3], NULL, 16);
+       else
+               data = hextoul(argv[3], NULL);
 
        bytes = size * length;
        buf = map_sysmem(addr, bytes);
@@ -519,13 +657,11 @@ static int do_mem_loopw(cmd_tbl_t *cmdtp, int flag, int argc,
         * If we have only one object, just run infinite loops.
         */
        if (length == 1) {
-#ifdef MEM_SUPPORT_64BIT_DATA
-               if (size == 8) {
-                       llp = (u64 *)buf;
+               if (SUPPORT_64BIT_DATA && size == 8) {
+                       llp = (ulong *)buf;
                        for (;;)
                                *llp = data;
                }
-#endif
                if (size == 4) {
                        longp = (u32 *)buf;
                        for (;;)
@@ -541,16 +677,14 @@ static int do_mem_loopw(cmd_tbl_t *cmdtp, int flag, int argc,
                        *cp = data;
        }
 
-#ifdef MEM_SUPPORT_64BIT_DATA
-       if (size == 8) {
+       if (SUPPORT_64BIT_DATA && size == 8) {
                for (;;) {
-                       llp = (u64 *)buf;
+                       llp = (ulong *)buf;
                        i = length;
                        while (i-- > 0)
                                *llp++ = data;
                }
        }
-#endif
        if (size == 4) {
                for (;;) {
                        longp = (u32 *)buf;
@@ -854,6 +988,18 @@ static ulong test_bitflip_comparison(volatile unsigned long *bufa,
        return errs;
 }
 
+static ulong mem_test_bitflip(vu_long *buf, ulong start, ulong end)
+{
+       /*
+        * Split the specified range into two halves.
+        * Note that mtest range is inclusive of start,end.
+        * Bitflip test instead uses a count (of 32-bit words).
+        */
+       ulong half_size = (end - start + 1) / 2 / sizeof(unsigned long);
+
+       return test_bitflip_comparison(buf, buf + half_size, half_size);
+}
+
 static ulong mem_test_quick(vu_long *buf, ulong start_addr, ulong end_addr,
                            vu_long pattern, int iteration)
 {
@@ -918,11 +1064,12 @@ static ulong mem_test_quick(vu_long *buf, ulong start_addr, ulong end_addr,
  * configured using CONFIG_SYS_ALT_MEMTEST. The complete test loops until
  * interrupted by ctrl-c or by a failure of one of the sub-tests.
  */
-static int do_mem_mtest(cmd_tbl_t *cmdtp, int flag, int argc,
-                       char * const argv[])
+static int do_mem_mtest(struct cmd_tbl *cmdtp, int flag, int argc,
+                       char *const argv[])
 {
        ulong start, end;
-       vu_long *buf, *dummy;
+       vu_long scratch_space;
+       vu_long *buf, *dummy = &scratch_space;
        ulong iteration_limit = 0;
        ulong count = 0;
        ulong errs = 0; /* number of errors, or -1 if interrupted */
@@ -958,7 +1105,6 @@ static int do_mem_mtest(cmd_tbl_t *cmdtp, int flag, int argc,
              start, end);
 
        buf = map_sysmem(start, end - start);
-       dummy = map_sysmem(CONFIG_SYS_MEMTEST_SCRATCH, sizeof(vu_long));
        for (iteration = 0;
                        !iteration_limit || iteration < iteration_limit;
                        iteration++) {
@@ -973,11 +1119,10 @@ static int do_mem_mtest(cmd_tbl_t *cmdtp, int flag, int argc,
                        errs = mem_test_alt(buf, start, end, dummy);
                        if (errs == -1UL)
                                break;
-                       count += errs;
-                       errs = test_bitflip_comparison(buf,
-                                                      buf + (end - start) / 2,
-                                                      (end - start) /
-                                                      sizeof(unsigned long));
+                       if (IS_ENABLED(CONFIG_SYS_ALT_MEMTEST_BITFLIP)) {
+                               count += errs;
+                               errs = mem_test_bitflip(buf, start, end);
+                       }
                } else {
                        errs = mem_test_quick(buf, start, end, pattern,
                                              iteration);
@@ -988,7 +1133,6 @@ static int do_mem_mtest(cmd_tbl_t *cmdtp, int flag, int argc,
        }
 
        unmap_sysmem((void *)buf);
-       unmap_sysmem((void *)dummy);
 
        if (errs == -1UL) {
                /* Memory test was aborted - write a newline to finish off */
@@ -1004,17 +1148,13 @@ static int do_mem_mtest(cmd_tbl_t *cmdtp, int flag, int argc,
  *
  * Syntax:
  *     mm{.b, .w, .l, .q} {addr}
- *     nm{.b, .w, .l, .q} {addr}
  */
 static int
-mod_mem(cmd_tbl_t *cmdtp, int incrflag, int flag, int argc, char * const argv[])
+mod_mem(struct cmd_tbl *cmdtp, int incrflag, int flag, int argc,
+       char *const argv[])
 {
        ulong   addr;
-#ifdef MEM_SUPPORT_64BIT_DATA
-       u64 i;
-#else
-       ulong i;
-#endif
+       ulong i;  /* 64-bit if SUPPORT_64BIT_DATA */
        int     nbytes, size;
        void *ptr = NULL;
 
@@ -1037,7 +1177,7 @@ mod_mem(cmd_tbl_t *cmdtp, int incrflag, int flag, int argc, char * const argv[])
 
                /* Address is specified since argc > 1
                */
-               addr = simple_strtoul(argv[1], NULL, 16);
+               addr = hextoul(argv[1], NULL);
                addr += base_address;
        }
 
@@ -1049,10 +1189,8 @@ mod_mem(cmd_tbl_t *cmdtp, int incrflag, int flag, int argc, char * const argv[])
                printf("%08lx:", addr);
                if (size == 4)
                        printf(" %08x", *((u32 *)ptr));
-#ifdef MEM_SUPPORT_64BIT_DATA
-               else if (size == 8)
-                       printf(" %016llx", *((u64 *)ptr));
-#endif
+               else if (SUPPORT_64BIT_DATA && size == 8)
+                       printf(" %0lx", *((ulong *)ptr));
                else if (size == 2)
                        printf(" %04x", *((u16 *)ptr));
                else
@@ -1076,11 +1214,10 @@ mod_mem(cmd_tbl_t *cmdtp, int incrflag, int flag, int argc, char * const argv[])
 #endif
                else {
                        char *endp;
-#ifdef MEM_SUPPORT_64BIT_DATA
-                       i = simple_strtoull(console_buffer, &endp, 16);
-#else
-                       i = simple_strtoul(console_buffer, &endp, 16);
-#endif
+                       if (SUPPORT_64BIT_DATA)
+                               i = simple_strtoull(console_buffer, &endp, 16);
+                       else
+                               i = hextoul(console_buffer, &endp);
                        nbytes = endp - console_buffer;
                        if (nbytes) {
                                /* good enough to not time out
@@ -1088,10 +1225,8 @@ mod_mem(cmd_tbl_t *cmdtp, int incrflag, int flag, int argc, char * const argv[])
                                bootretry_reset_cmd_timeout();
                                if (size == 4)
                                        *((u32 *)ptr) = i;
-#ifdef MEM_SUPPORT_64BIT_DATA
-                               else if (size == 8)
-                                       *((u64 *)ptr) = i;
-#endif
+                               else if (SUPPORT_64BIT_DATA && size == 8)
+                                       *((ulong *)ptr) = i;
                                else if (size == 2)
                                        *((u16 *)ptr) = i;
                                else
@@ -1111,7 +1246,8 @@ mod_mem(cmd_tbl_t *cmdtp, int incrflag, int flag, int argc, char * const argv[])
 
 #ifdef CONFIG_CMD_CRC32
 
-static int do_mem_crc(cmd_tbl_t *cmdtp, int flag, int argc, char * const argv[])
+static int do_mem_crc(struct cmd_tbl *cmdtp, int flag, int argc,
+                     char *const argv[])
 {
        int flags = 0;
        int ac;
@@ -1136,7 +1272,8 @@ static int do_mem_crc(cmd_tbl_t *cmdtp, int flag, int argc, char * const argv[])
 #endif
 
 #ifdef CONFIG_CMD_RANDOM
-static int do_random(cmd_tbl_t *cmdtp, int flag, int argc, char * const argv[])
+static int do_random(struct cmd_tbl *cmdtp, int flag, int argc,
+                    char *const argv[])
 {
        unsigned long addr, len;
        unsigned long seed; // NOT INITIALIZED ON PURPOSE
@@ -1144,16 +1281,14 @@ static int do_random(cmd_tbl_t *cmdtp, int flag, int argc, char * const argv[])
        unsigned char *buf8;
        unsigned int i;
 
-       if (argc < 3 || argc > 4) {
-               printf("usage: %s <addr> <len> [<seed>]\n", argv[0]);
-               return 0;
-       }
+       if (argc < 3 || argc > 4)
+               return CMD_RET_USAGE;
 
-       len = simple_strtoul(argv[2], NULL, 16);
-       addr = simple_strtoul(argv[1], NULL, 16);
+       len = hextoul(argv[2], NULL);
+       addr = hextoul(argv[1], NULL);
 
        if (argc == 4) {
-               seed = simple_strtoul(argv[3], NULL, 16);
+               seed = hextoul(argv[3], NULL);
                if (seed == 0) {
                        printf("The seed cannot be 0. Using 0xDEADBEEF.\n");
                        seed = 0xDEADBEEF;
@@ -1174,7 +1309,8 @@ static int do_random(cmd_tbl_t *cmdtp, int flag, int argc, char * const argv[])
 
        unmap_sysmem(start);
        printf("%lu bytes filled with random data\n", len);
-       return 1;
+
+       return CMD_RET_SUCCESS;
 }
 #endif
 
@@ -1182,64 +1318,50 @@ static int do_random(cmd_tbl_t *cmdtp, int flag, int argc, char * const argv[])
 U_BOOT_CMD(
        md,     3,      1,      do_mem_md,
        "memory display",
-#ifdef MEM_SUPPORT_64BIT_DATA
-       "[.b, .w, .l, .q] address [# of objects]"
-#else
-       "[.b, .w, .l] address [# of objects]"
-#endif
+       "[.b, .w, .l" HELP_Q "] address [# of objects]"
 );
 
 
 U_BOOT_CMD(
        mm,     2,      1,      do_mem_mm,
        "memory modify (auto-incrementing address)",
-#ifdef MEM_SUPPORT_64BIT_DATA
-       "[.b, .w, .l, .q] address"
-#else
-       "[.b, .w, .l] address"
-#endif
+       "[.b, .w, .l" HELP_Q "] address"
 );
 
 
 U_BOOT_CMD(
        nm,     2,      1,      do_mem_nm,
        "memory modify (constant address)",
-#ifdef MEM_SUPPORT_64BIT_DATA
-       "[.b, .w, .l, .q] address"
-#else
-       "[.b, .w, .l] address"
-#endif
+       "[.b, .w, .l" HELP_Q "] address"
 );
 
 U_BOOT_CMD(
        mw,     4,      1,      do_mem_mw,
        "memory write (fill)",
-#ifdef MEM_SUPPORT_64BIT_DATA
-       "[.b, .w, .l, .q] address value [count]"
-#else
-       "[.b, .w, .l] address value [count]"
-#endif
+       "[.b, .w, .l" HELP_Q "] address value [count]"
 );
 
 U_BOOT_CMD(
        cp,     4,      1,      do_mem_cp,
        "memory copy",
-#ifdef MEM_SUPPORT_64BIT_DATA
-       "[.b, .w, .l, .q] source target count"
-#else
-       "[.b, .w, .l] source target count"
-#endif
+       "[.b, .w, .l" HELP_Q "] source target count"
 );
 
 U_BOOT_CMD(
        cmp,    4,      1,      do_mem_cmp,
        "memory compare",
-#ifdef MEM_SUPPORT_64BIT_DATA
-       "[.b, .w, .l, .q] addr1 addr2 count"
-#else
-       "[.b, .w, .l] addr1 addr2 count"
-#endif
+       "[.b, .w, .l" HELP_Q "] addr1 addr2 count"
+);
+
+#ifdef CONFIG_CMD_MEM_SEARCH
+/**************************************************/
+U_BOOT_CMD(
+       ms,     255,    1,      do_mem_search,
+       "memory search",
+       "[.b, .w, .l" HELP_Q ", .s] [-q | -<n>] address #-of-objects <value>..."
+       "  -q = quiet, -l<val> = match limit"
 );
+#endif
 
 #ifdef CONFIG_CMD_CRC32
 
@@ -1265,8 +1387,8 @@ U_BOOT_CMD(
 #endif
 
 #ifdef CONFIG_CMD_MEMINFO
-static int do_mem_info(cmd_tbl_t *cmdtp, int flag, int argc,
-                      char * const argv[])
+static int do_mem_info(struct cmd_tbl *cmdtp, int flag, int argc,
+                      char *const argv[])
 {
        puts("DRAM:  ");
        print_size(gd->ram_size, "\n");
@@ -1285,22 +1407,14 @@ U_BOOT_CMD(
 U_BOOT_CMD(
        loop,   3,      1,      do_mem_loop,
        "infinite loop on address range",
-#ifdef MEM_SUPPORT_64BIT_DATA
-       "[.b, .w, .l, .q] address number_of_objects"
-#else
-       "[.b, .w, .l] address number_of_objects"
-#endif
+       "[.b, .w, .l" HELP_Q "] address number_of_objects"
 );
 
 #ifdef CONFIG_LOOPW
 U_BOOT_CMD(
        loopw,  4,      1,      do_mem_loopw,
        "infinite write loop on address range",
-#ifdef MEM_SUPPORT_64BIT_DATA
-       "[.b, .w, .l, .q] address number_of_objects data_to_write"
-#else
-       "[.b, .w, .l] address number_of_objects data_to_write"
-#endif
+       "[.b, .w, .l" HELP_Q "] address number_of_objects data_to_write"
 );
 #endif /* CONFIG_LOOPW */
 
@@ -1316,21 +1430,13 @@ U_BOOT_CMD(
 U_BOOT_CMD(
        mdc,    4,      1,      do_mem_mdc,
        "memory display cyclic",
-#ifdef MEM_SUPPORT_64BIT_DATA
-       "[.b, .w, .l, .q] address count delay(ms)"
-#else
-       "[.b, .w, .l] address count delay(ms)"
-#endif
+       "[.b, .w, .l" HELP_Q "] address count delay(ms)"
 );
 
 U_BOOT_CMD(
        mwc,    4,      1,      do_mem_mwc,
        "memory write cyclic",
-#ifdef MEM_SUPPORT_64BIT_DATA
-       "[.b, .w, .l, .q] address value delay(ms)"
-#else
-       "[.b, .w, .l] address value delay(ms)"
-#endif
+       "[.b, .w, .l" HELP_Q "] address value delay(ms)"
 );
 #endif /* CONFIG_CMD_MX_CYCLIC */