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 | olio-uboot-2014.01-7d25cd34e9d73ab7fff8f3c4283a11e50b0d204b.tar.xz olio-uboot-2014.01-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 a66f569a4..adc1ce443 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; |