diff options
| author | Masahiro Yamada <yamada.m@jp.panasonic.com> | 2013-07-11 17:27:13 +0900 |
|---|---|---|
| committer | Scott Wood <scottwood@freescale.com> | 2013-08-22 17:25:02 -0500 |
| commit | 7d25cd34e9d73ab7fff8f3c4283a11e50b0d204b (patch) | |
| tree | 90b95c0445296b5b7a8760b6495778e4c6fc924b /common | |
| parent | e40520b5b585c82d90e94c54cb3035b277e8280f (diff) | |
| download | talos-obmc-uboot-7d25cd34e9d73ab7fff8f3c4283a11e50b0d204b.tar.gz talos-obmc-uboot-7d25cd34e9d73ab7fff8f3c4283a11e50b0d204b.zip | |
cmd_nand: slight optimization of nand_dump function
If a non-zero value is given to only_oob argument,
printing the main area is skipped.
With a little modification, we can skip the whole
while loop.
Signed-off-by: Masahiro Yamada <yamada.m@jp.panasonic.com>
Diffstat (limited to 'common')
| -rw-r--r-- | common/cmd_nand.c | 13 |
1 files changed, 8 insertions, 5 deletions
diff --git a/common/cmd_nand.c b/common/cmd_nand.c index a66f569a42..adc1ce4435 100644 --- a/common/cmd_nand.c +++ b/common/cmd_nand.c @@ -77,18 +77,21 @@ static int nand_dump(nand_info_t *nand, ulong off, int only_oob, int repeat) goto free_all; } printf("Page %08lx dump:\n", off); - i = nand->writesize >> 4; - p = datbuf; - while (i--) { - if (!only_oob) + if (!only_oob) { + i = nand->writesize >> 4; + p = datbuf; + + while (i--) { printf("\t%02x %02x %02x %02x %02x %02x %02x %02x" " %02x %02x %02x %02x %02x %02x %02x %02x\n", p[0], p[1], p[2], p[3], p[4], p[5], p[6], p[7], p[8], p[9], p[10], p[11], p[12], p[13], p[14], p[15]); - p += 16; + p += 16; + } } + puts("OOB:\n"); i = nand->oobsize >> 3; p = oobbuf; |
