diff options
Diffstat (limited to 'common')
| -rw-r--r-- | common/cmd_nand.c | 53 | ||||
| -rw-r--r-- | common/cmd_onenand.c | 30 | ||||
| -rw-r--r-- | common/env_onenand.c | 6 | 
3 files changed, 62 insertions, 27 deletions
| diff --git a/common/cmd_nand.c b/common/cmd_nand.c index e9d3d3c1b..8b1e01ae8 100644 --- a/common/cmd_nand.c +++ b/common/cmd_nand.c @@ -62,8 +62,8 @@ static int nand_dump(nand_info_t *nand, ulong off, int only_oob, int repeat)  	ops.oobbuf = oobbuf;  	ops.len = nand->writesize;  	ops.ooblen = nand->oobsize; -	ops.mode = MTD_OOB_RAW; -	i = nand->read_oob(nand, addr, &ops); +	ops.mode = MTD_OPS_RAW; +	i = mtd_read_oob(nand, addr, &ops);  	if (i < 0) {  		printf("Error (%d) reading page %08lx\n", i, off);  		free(datbuf); @@ -404,13 +404,13 @@ static int raw_access(nand_info_t *nand, ulong addr, loff_t off, ulong count,  			.oobbuf = ((u8 *)addr) + nand->writesize,  			.len = nand->writesize,  			.ooblen = nand->oobsize, -			.mode = MTD_OOB_RAW +			.mode = MTD_OPS_RAW  		};  		if (read) -			ret = nand->read_oob(nand, off, &ops); +			ret = mtd_read_oob(nand, off, &ops);  		else -			ret = nand->write_oob(nand, off, &ops); +			ret = mtd_write_oob(nand, off, &ops);  		if (ret) {  			printf("%s: error at offset %llx, ret %d\n", @@ -425,6 +425,31 @@ static int raw_access(nand_info_t *nand, ulong addr, loff_t off, ulong count,  	return ret;  } +/* Adjust a chip/partition size down for bad blocks so we don't + * read/write/erase past the end of a chip/partition by accident. + */ +static void adjust_size_for_badblocks(loff_t *size, loff_t offset, int dev) +{ +	/* We grab the nand info object here fresh because this is usually +	 * called after arg_off_size() which can change the value of dev. +	 */ +	nand_info_t *nand = &nand_info[dev]; +	loff_t maxoffset = offset + *size; +	int badblocks = 0; + +	/* count badblocks in NAND from offset to offset + size */ +	for (; offset < maxoffset; offset += nand->erasesize) { +		if (nand_block_isbad(nand, offset)) +			badblocks++; +	} +	/* adjust size if any bad blocks found */ +	if (badblocks) { +		*size -= badblocks * nand->erasesize; +		printf("size adjusted to 0x%llx (%d bad blocks)\n", +		       (unsigned long long)*size, badblocks); +	} +} +  static int do_nand(cmd_tbl_t *cmdtp, int flag, int argc, char * const argv[])  {  	int i, ret = 0; @@ -521,6 +546,7 @@ static int do_nand(cmd_tbl_t *cmdtp, int flag, int argc, char * const argv[])  		int scrub = !strncmp(cmd, "scrub", 5);  		int spread = 0;  		int args = 2; +		int adjust_size = 0;  		const char *scrub_warn =  			"Warning: "  			"scrub option will erase all factory set bad blocks!\n" @@ -537,8 +563,10 @@ static int do_nand(cmd_tbl_t *cmdtp, int flag, int argc, char * const argv[])  				spread = 1;  			} else if (!strcmp(&cmd[5], ".part")) {  				args = 1; +				adjust_size = 1;  			} else if (!strcmp(&cmd[5], ".chip")) {  				args = 0; +				adjust_size = 1;  			} else {  				goto usage;  			} @@ -558,6 +586,10 @@ static int do_nand(cmd_tbl_t *cmdtp, int flag, int argc, char * const argv[])  				 &maxsize) != 0)  			return 1; +		/* size is unspecified */ +		if (adjust_size && !scrub) +			adjust_size_for_badblocks(&size, off, dev); +  		nand = &nand_info[dev];  		memset(&opts, 0, sizeof(opts)); @@ -642,6 +674,9 @@ static int do_nand(cmd_tbl_t *cmdtp, int flag, int argc, char * const argv[])  						&off, &size, &maxsize) != 0)  				return 1; +			/* size is unspecified */ +			if (argc < 5) +				adjust_size_for_badblocks(&size, off, dev);  			rwsize = size;  		} @@ -680,13 +715,13 @@ static int do_nand(cmd_tbl_t *cmdtp, int flag, int argc, char * const argv[])  			mtd_oob_ops_t ops = {  				.oobbuf = (u8 *)addr,  				.ooblen = rwsize, -				.mode = MTD_OOB_RAW +				.mode = MTD_OPS_RAW  			};  			if (read) -				ret = nand->read_oob(nand, off, &ops); +				ret = mtd_read_oob(nand, off, &ops);  			else -				ret = nand->write_oob(nand, off, &ops); +				ret = mtd_write_oob(nand, off, &ops);  		} else if (raw) {  			ret = raw_access(nand, addr, off, pagecount, read);  		} else { @@ -729,7 +764,7 @@ static int do_nand(cmd_tbl_t *cmdtp, int flag, int argc, char * const argv[])  		while (argc > 0) {  			addr = simple_strtoul(*argv, NULL, 16); -			if (nand->block_markbad(nand, addr)) { +			if (mtd_block_markbad(nand, addr)) {  				printf("block 0x%08lx NOT marked "  					"as bad! ERROR %d\n",  					addr, ret); diff --git a/common/cmd_onenand.c b/common/cmd_onenand.c index a0d25e552..06cc14056 100644 --- a/common/cmd_onenand.c +++ b/common/cmd_onenand.c @@ -83,7 +83,7 @@ static int onenand_block_read(loff_t from, size_t len,  		ops.len = blocksize;  	while (blocks) { -		ret = mtd->block_isbad(mtd, ofs); +		ret = mtd_block_isbad(mtd, ofs);  		if (ret) {  			printk("Bad blocks %d at 0x%x\n",  			       (u32)(ofs >> this->erase_shift), (u32)ofs); @@ -97,7 +97,7 @@ static int onenand_block_read(loff_t from, size_t len,  			ops.datbuf = buf;  		ops.retlen = 0; -		ret = mtd->read_oob(mtd, ofs, &ops); +		ret = mtd_read_oob(mtd, ofs, &ops);  		if (ret) {  			printk("Read failed 0x%x, %d\n", (u32)ofs, ret);  			ofs += blocksize; @@ -118,7 +118,7 @@ static int onenand_write_oneblock_withoob(loff_t to, const u_char * buf,  	struct mtd_oob_ops ops = {  		.len = mtd->writesize,  		.ooblen = mtd->oobsize, -		.mode = MTD_OOB_AUTO, +		.mode = MTD_OPS_AUTO_OOB,  	};  	int page, ret = 0;  	for (page = 0; page < (mtd->erasesize / mtd->writesize); page ++) { @@ -126,7 +126,7 @@ static int onenand_write_oneblock_withoob(loff_t to, const u_char * buf,  		buf += mtd->writesize;  		ops.oobbuf = (u_char *)buf;  		buf += mtd->oobsize; -		ret = mtd->write_oob(mtd, to, &ops); +		ret = mtd_write_oob(mtd, to, &ops);  		if (ret)  			break;  		to += mtd->writesize; @@ -156,7 +156,7 @@ static int onenand_block_write(loff_t to, size_t len,  	ofs = to;  	while (blocks) { -		ret = mtd->block_isbad(mtd, ofs); +		ret = mtd_block_isbad(mtd, ofs);  		if (ret) {  			printk("Bad blocks %d at 0x%x\n",  			       (u32)(ofs >> this->erase_shift), (u32)ofs); @@ -165,7 +165,7 @@ static int onenand_block_write(loff_t to, size_t len,  		}  		if (!withoob) -			ret = mtd->write(mtd, ofs, blocksize, &_retlen, buf); +			ret = mtd_write(mtd, ofs, blocksize, &_retlen, buf);  		else  			ret = onenand_write_oneblock_withoob(ofs, buf, &_retlen);  		if (ret) { @@ -195,7 +195,7 @@ static int onenand_block_erase(u32 start, u32 size, int force)  	int blocksize = 1 << this->erase_shift;  	for (ofs = start; ofs < (start + size); ofs += blocksize) { -		ret = mtd->block_isbad(mtd, ofs); +		ret = mtd_block_isbad(mtd, ofs);  		if (ret && !force) {  			printf("Skip erase bad block %d at 0x%x\n",  			       (u32)(ofs >> this->erase_shift), (u32)ofs); @@ -206,7 +206,7 @@ static int onenand_block_erase(u32 start, u32 size, int force)  		instr.len = blocksize;  		instr.priv = force;  		instr.mtd = mtd; -		ret = mtd->erase(mtd, &instr); +		ret = mtd_erase(mtd, &instr);  		if (ret) {  			printf("erase failed block %d at 0x%x\n",  			       (u32)(ofs >> this->erase_shift), (u32)ofs); @@ -261,7 +261,7 @@ static int onenand_block_test(u32 start, u32 size)  	while (blocks < end_block) {  		printf("\rTesting block %d at 0x%x", (u32)(ofs >> this->erase_shift), (u32)ofs); -		ret = mtd->block_isbad(mtd, ofs); +		ret = mtd_block_isbad(mtd, ofs);  		if (ret) {  			printf("Skip erase bad block %d at 0x%x\n",  			       (u32)(ofs >> this->erase_shift), (u32)ofs); @@ -270,19 +270,19 @@ static int onenand_block_test(u32 start, u32 size)  		instr.addr = ofs;  		instr.len = blocksize; -		ret = mtd->erase(mtd, &instr); +		ret = mtd_erase(mtd, &instr);  		if (ret) {  			printk("Erase failed 0x%x, %d\n", (u32)ofs, ret);  			goto next;  		} -		ret = mtd->write(mtd, ofs, blocksize, &retlen, buf); +		ret = mtd_write(mtd, ofs, blocksize, &retlen, buf);  		if (ret) {  			printk("Write failed 0x%x, %d\n", (u32)ofs, ret);  			goto next;  		} -		ret = mtd->read(mtd, ofs, blocksize, &retlen, verify_buf); +		ret = mtd_read(mtd, ofs, blocksize, &retlen, verify_buf);  		if (ret) {  			printk("Read failed 0x%x, %d\n", (u32)ofs, ret);  			goto next; @@ -324,7 +324,7 @@ static int onenand_dump(struct mtd_info *mtd, ulong off, int only_oob)  	ops.len = mtd->writesize;  	ops.ooblen = mtd->oobsize;  	ops.retlen = 0; -	i = mtd->read_oob(mtd, addr, &ops); +	i = mtd_read_oob(mtd, addr, &ops);  	if (i < 0) {  		printf("Error (%d) reading page %08lx\n", i, off);  		free(datbuf); @@ -373,7 +373,7 @@ static int do_onenand_bad(cmd_tbl_t * cmdtp, int flag, int argc, char * const ar  	/* Currently only one OneNAND device is supported */  	printf("\nDevice %d bad blocks:\n", 0);  	for (ofs = 0; ofs < mtd->size; ofs += mtd->erasesize) { -		if (mtd->block_isbad(mtd, ofs)) +		if (mtd_block_isbad(mtd, ofs))  			printf("  %08x\n", (u32)ofs);  	} @@ -530,7 +530,7 @@ static int do_onenand_markbad(cmd_tbl_t * cmdtp, int flag, int argc, char * cons  	while (argc > 0) {  		addr = simple_strtoul(*argv, NULL, 16); -		if (mtd->block_markbad(mtd, addr)) { +		if (mtd_block_markbad(mtd, addr)) {  			printf("block 0x%08lx NOT marked "  				"as bad! ERROR %d\n",  				addr, ret); diff --git a/common/env_onenand.c b/common/env_onenand.c index faa903d2f..e8bde3726 100644 --- a/common/env_onenand.c +++ b/common/env_onenand.c @@ -68,7 +68,7 @@ void env_relocate_spec(void)  	/* Check OneNAND exist */  	if (mtd->writesize)  		/* Ignore read fail */ -		mtd->read(mtd, env_addr, ONENAND_MAX_ENV_SIZE, +		mtd_read(mtd, env_addr, ONENAND_MAX_ENV_SIZE,  				&retlen, (u_char *)buf);  	else  		mtd->writesize = MAX_ONENAND_PAGESIZE; @@ -113,12 +113,12 @@ int saveenv(void)  #endif  	instr.addr = env_addr;  	instr.mtd = mtd; -	if (mtd->erase(mtd, &instr)) { +	if (mtd_erase(mtd, &instr)) {  		printf("OneNAND: erase failed at 0x%08llx\n", env_addr);  		return 1;  	} -	if (mtd->write(mtd, env_addr, ONENAND_MAX_ENV_SIZE, &retlen, +	if (mtd_write(mtd, env_addr, ONENAND_MAX_ENV_SIZE, &retlen,  			(u_char *)&env_new)) {  		printf("OneNAND: write failed at 0x%llx\n", instr.addr);  		return 2; |