[U-Boot] [PATCH] [OneNAND] bad block aware read/write support
Kyungmin Park
kmpark at infradead.org
Tue Nov 4 01:04:45 CET 2008
Update OneNAND command to support bad block awareness
Also change the OneNAND command styel like NAND
Signed-off-by: Kyungmin Park <kyungmin.park at samsung.com>
---
diff --git a/common/cmd_onenand.c b/common/cmd_onenand.c
index 8d87b78..1eca9b0 100644
--- a/common/cmd_onenand.c
+++ b/common/cmd_onenand.c
@@ -1,7 +1,7 @@
/*
* U-Boot command for OneNAND support
*
- * Copyright (C) 2005-2007 Samsung Electronics
+ * Copyright (C) 2005-2008 Samsung Electronics
* Kyungmin Park <kyungmin.park at samsung.com>
*
* This program is free software; you can redistribute it and/or modify
@@ -18,12 +18,245 @@
#include <asm/io.h>
-extern struct mtd_info onenand_mtd;
-extern struct onenand_chip onenand_chip;
+static struct mtd_info *mtd = &onenand_mtd;
+
+static loff_t next_ofs;
+static loff_t skip_ofs;
+
+static int onenand_block_read(loff_t from, size_t len,
+ size_t *retlen, u_char *buf, int oob)
+{
+ struct onenand_chip *this = mtd->priv;
+ int blocks = (int) len >> this->erase_shift;
+ int blocksize = (1 << this->erase_shift);
+ loff_t ofs = from;
+ struct mtd_oob_ops ops = {
+ .retlen = 0,
+ };
+ int ret, dump = 0;
+
+ if (buf == NULL) {
+ buf = (u_char *) CONFIG_SYS_LOAD_ADDR;
+ dump = 1;
+ }
+
+ if (oob)
+ ops.ooblen = blocksize;
+ else
+ ops.len = blocksize;
+
+ while (blocks) {
+ ret = mtd->block_isbad(mtd, ofs);
+ if (ret) {
+ printk("Bad blocks %d at 0x%x\n", (unsigned int) ofs >> this->erase_shift, (unsigned int) ofs);
+ ofs += blocksize;
+ continue;
+ }
+
+ if (oob)
+ ops.oobbuf = buf;
+ else
+ ops.datbuf = buf;
+
+ ops.retlen = 0;
+ ret = mtd->read_oob(mtd, ofs, &ops);
+ if (ret) {
+ printk("Read failed 0x%x, %d", (unsigned int) ofs, ret);
+ mtd->block_markbad(mtd, ofs);
+ ofs += blocksize;
+ continue;
+ }
+ ofs += blocksize;
+ buf += blocksize;
+ blocks--;
+ *retlen += ops.retlen;
+
+ if (dump) {
+ int i;
+ unsigned int *p = (unsigned int *) CONFIG_SYS_LOAD_ADDR;
+ printf("Dump offset 0x%x (%d)\n", (unsigned int) ofs, (int) ofs >> this->erase_shift);
+ for (i = 0; i < 32; i++) {
+ printf("0x%08x ", *p++);
+ if (((i + 1) & (4 - 1)) == 0)
+ printf("\n");
+ }
+ buf = (u_char *) CONFIG_SYS_LOAD_ADDR;
+ }
+ }
+
+ printf("Read from 0x%x to 0x%x (%d + %d blocks)\n", (unsigned int) from, (unsigned int) ofs, (int) len >> this->erase_shift, (int) ((ofs - from) - len) >> this->erase_shift);
+
+ return 0;
+}
+
+static int onenand_block_write(loff_t to, size_t len,
+ size_t *retlen, const u_char * buf)
+{
+ struct onenand_chip *this = mtd->priv;
+ int blocks = len >> this->erase_shift;
+ int blocksize = (1 << this->erase_shift);
+ loff_t ofs;
+ size_t _retlen = 0;
+ int ret;
+
+ if (to == next_ofs) {
+ next_ofs = to + len;
+ to += skip_ofs;
+ } else {
+ next_ofs = to + len;
+ skip_ofs = 0;
+ }
+ ofs = to;
+
+ while (blocks) {
+ ret = mtd->block_isbad(mtd, ofs);
+ if (ret) {
+ printk("Bad blocks %d at 0x%x\n", (unsigned int) ofs >> this->erase_shift, (unsigned int) ofs);
+ skip_ofs += blocksize;
+ goto next;
+ }
+#ifdef USE_ERASEWRITE
+ if ((ofs & (blocksize - 1)) == 0) {
+ struct erase_info instr = {
+ .callback = NULL,
+ .addr = ofs,
+ .len = blocksize,
+ .priv = 0,
+ };
+ ret = mtd->erase(mtd, &instr);
+ if (ret) {
+ printk("Erase failed 0x%x, %d", (unsigned int) ofs, ret);
+ mtd->block_markbad(mtd, ofs);
+ skip_ofs += blocksize;
+ goto next;
+ }
+ }
+#endif
+ ret = mtd->write(mtd, ofs, blocksize, &_retlen, buf);
+ if (ret) {
+ printk("Write failed 0x%x, %d", (unsigned int) ofs, ret);
+ mtd->block_markbad(mtd, ofs);
+ skip_ofs += blocksize;
+ goto next;
+ }
+
+ buf += blocksize;
+ blocks--;
+ *retlen += _retlen;
+next:
+ ofs += blocksize;
+ }
+
+ printf("Write from 0x%x to 0x%x (%d + %d blocks)\n", (unsigned int) to, (unsigned int) ofs, (int) len >> this->erase_shift, (int) ((ofs - to) - len) >> this->erase_shift);
+
+ return 0;
+}
+
+static int onenand_block_erase(int start, int end, int force)
+{
+ struct onenand_chip *this = mtd->priv;
+ struct erase_info instr = {
+ .callback = NULL,
+ };
+ loff_t ofs;
+ int block, ret;
+ int blocksize = 1 << this->erase_shift;
+
+ for (block = start; block <= end; block++) {
+ ofs = block << this->erase_shift;
+ ret = mtd->block_isbad(mtd, ofs);
+ if (ret && !force) {
+ printf("Skip erase bad block %d at 0x%x\n", block, (unsigned int) block << this->erase_shift);
+ continue;
+ }
+
+ instr.addr = ofs;
+ instr.len = blocksize;
+ instr.priv = force;
+ ret = mtd->erase(mtd, &instr);
+ if (ret) {
+ printf("erase failed %d\n", block);
+ mtd->block_markbad(mtd, instr.addr);
+ continue;
+ }
+ }
+}
+
+static int onenand_badblocks(unsigned long start_address, unsigned long end_address)
+{
+ struct onenand_chip *this = mtd->priv;
+ struct erase_info instr = {
+ .callback = NULL,
+ .priv = 0,
+ };
+
+ int blocks;
+ loff_t ofs = 0;
+ int blocksize = 1 << this->erase_shift;
+ int start_block, end_block;
+ size_t retlen;
+ u_char *buf = (u_char *) CONFIG_SYS_LOAD_ADDR;
+ /* block size '512KiB' is enough */
+ u_char *verify_buf = (u_char *) CONFIG_SYS_LOAD_ADDR + 0x80000;
+ int ret;
+
+ start_block = start_address >> this->erase_shift;
+ end_block = end_address >> this->erase_shift;
+
+ /* Protect boot-loader from badblock testing */
+ if (start_block < 2)
+ start_block = 2;
+
+ if (end_block > (mtd->size >> this->erase_shift))
+ end_block = mtd->size >> this->erase_shift;
+
+ blocks = start_block;
+ while (blocks < end_block) {
+ ret = mtd->block_isbad(mtd, ofs);
+ if (ret) {
+ ofs += blocksize;
+ continue;
+ }
+ instr.addr = ofs;
+ instr.len = blocksize;
+ ret = mtd->erase(mtd, &instr);
+ if (ret) {
+ printk("Erase failed 0x%x, %d", (unsigned int) ofs, ret);
+ mtd->block_markbad(mtd, ofs);
+ goto next;
+ }
+ ret = mtd->write(mtd, ofs, blocksize, &retlen, buf);
+ if (ret) {
+ printk("Write failed 0x%x, %d", (unsigned int) ofs, ret);
+ mtd->block_markbad(mtd, ofs);
+ goto next;
+ }
+ ret = mtd->read(mtd, ofs, blocksize, &retlen, verify_buf);
+ if (ret) {
+ printk("Read failed 0x%x, %d", (unsigned int) ofs, ret);
+ mtd->block_markbad(mtd, ofs);
+ goto next;
+ }
+ if (memcmp(buf, verify_buf, blocksize))
+ printk("Read/Write test failed at 0x%x", (unsigned int) ofs);
+
+ printf("\rTest block at 0x%x", (unsigned int) ofs);
+
+next:
+ ofs += blocksize;
+ blocks++;
+ }
+ printf("...Done\n");
+
+ return 0;
+}
int do_onenand(cmd_tbl_t * cmdtp, int flag, int argc, char *argv[])
{
- int ret = 0;
+ struct onenand_chip *this = mtd->priv;
+ int blocksize = (1 << this->erase_shift);
+ ulong addr, ofs;
+ size_t len, retlen = 0;
switch (argc) {
case 0:
@@ -32,129 +265,100 @@ int do_onenand(cmd_tbl_t * cmdtp, int flag, int argc, char *argv[])
return 1;
case 2:
- if (strncmp(argv[1], "open", 4) == 0) {
- onenand_init();
- return 0;
- }
- printf("%s\n", onenand_mtd.name);
+ printf("%s\n", mtd->name);
return 0;
default:
/* At least 4 args */
if (strncmp(argv[1], "erase", 5) == 0) {
- struct erase_info instr = {
- .callback = NULL,
- };
- ulong start, end;
- ulong block;
- char *endtail;
+ ulong start, end, len;
+ char *endtail = NULL;
+ int force = 0;
+
+ if (strncmp(argv[1], "erasef", 6) == 0)
+ force = 1;
if (strncmp(argv[2], "block", 5) == 0) {
start = simple_strtoul(argv[3], NULL, 10);
endtail = strchr(argv[3], '-');
- end = simple_strtoul(endtail + 1, NULL, 10);
+ if (endtail)
+ end = simple_strtoul(endtail + 1, NULL, 10);
+ else
+ end = start;
} else {
- start = simple_strtoul(argv[2], NULL, 10);
- end = simple_strtoul(argv[3], NULL, 10);
+ start = simple_strtoul(argv[2], NULL, 16);
+ len = simple_strtoul(argv[3], NULL, 16);
- start >>= onenand_chip.erase_shift;
- end >>= onenand_chip.erase_shift;
+ start >>= this->erase_shift;
+ len >>= this->erase_shift;
/* Don't include the end block */
- end--;
+ end = start + len - 1;
}
if (!end || end < 0)
end = start;
- printf("Erase block from %lu to %lu\n", start, end);
+ /* Don't exceed the whole block size */
+ if (end >= mtd->size >> this->erase_shift)
+ end = (mtd->size >> this->erase_shift) - 1;
- for (block = start; block <= end; block++) {
- instr.addr = block << onenand_chip.erase_shift;
- instr.len = 1 << onenand_chip.erase_shift;
- ret = onenand_erase(&onenand_mtd, &instr);
- if (ret) {
- printf("erase failed %lu\n", block);
- break;
- }
- }
+ printf("Erase block from %lu to %lu\n", start, end);
+ onenand_block_erase(start, end, force);
return 0;
}
if (strncmp(argv[1], "read", 4) == 0) {
- ulong addr = simple_strtoul(argv[2], NULL, 16);
- ulong ofs = simple_strtoul(argv[3], NULL, 16);
- size_t len = simple_strtoul(argv[4], NULL, 16);
int oob = strncmp(argv[1], "read.oob", 8) ? 0 : 1;
- struct mtd_oob_ops ops;
-
- ops.mode = MTD_OOB_PLACE;
- if (oob) {
- ops.len = 0;
- ops.datbuf = NULL;
- ops.ooblen = len;
- ops.oobbuf = (u_char *) addr;
- } else {
- ops.len = len;
- ops.datbuf = (u_char *) addr;
- ops.ooblen = 0;
- ops.oobbuf = NULL;
- }
- ops.retlen = ops.oobretlen = 0;
+ addr = simple_strtoul(argv[2], NULL, 16);
+ ofs = simple_strtoul(argv[3], NULL, 16);
+ len = simple_strtoul(argv[4], NULL, 16);
- onenand_mtd.read_oob(&onenand_mtd, ofs, &ops);
- printf("Done\n");
+ onenand_block_read(ofs, len, &retlen, (u_char *) addr, oob);
return 0;
}
+ if (strncmp(argv[1], "badb", 4) == 0) {
+ /* Start address */
+ addr = simple_strtoul(argv[2], NULL, 16);
+ /* End address */
+ ofs = simple_strtoul(argv[3], NULL, 16);
- if (strncmp(argv[1], "write", 5) == 0) {
- ulong addr = simple_strtoul(argv[2], NULL, 16);
- ulong ofs = simple_strtoul(argv[3], NULL, 16);
- size_t len = simple_strtoul(argv[4], NULL, 16);
- size_t retlen = 0;
-
- onenand_write(&onenand_mtd, ofs, len, &retlen,
- (u_char *) addr);
- printf("Done\n");
-
- return 0;
+ return onenand_badblocks(addr, ofs);
}
- if (strncmp(argv[1], "block", 5) == 0) {
- ulong addr = simple_strtoul(argv[2], NULL, 16);
- ulong block = simple_strtoul(argv[3], NULL, 10);
- ulong page = simple_strtoul(argv[4], NULL, 10);
- size_t len = simple_strtol(argv[5], NULL, 10);
- ulong ofs;
- int oob = strncmp(argv[1], "block.oob", 9) ? 0 : 1;
- struct mtd_oob_ops ops;
-
- ops.mode = MTD_OOB_PLACE;
+ if (strncmp(argv[1], "write", 5) == 0) {
+ addr = simple_strtoul(argv[2], NULL, 16);
+ ofs = simple_strtoul(argv[3], NULL, 16);
+ len = simple_strtoul(argv[4], NULL, 16);
+ return onenand_block_write(ofs, len, &retlen, (u_char *) addr);
+ }
- ofs = block << onenand_chip.erase_shift;
- if (page)
- ofs += page << onenand_chip.page_shift;
+ if (strcmp(argv[1], "markbad") == 0) {
+ addr = (ulong)simple_strtoul(argv[2], NULL, 16);
- if (!len) {
- if (oob)
- ops.ooblen = 64;
- else
- ops.len = 512;
- }
-
- if (oob) {
- ops.datbuf = NULL;
- ops.oobbuf = (u_char *) addr;
+ int ret = mtd->block_markbad(mtd, addr);
+ if (ret == 0) {
+ printf("block 0x%08lx successfully marked as bad\n",
+ (ulong) addr);
+ return 0;
} else {
- ops.datbuf = (u_char *) addr;
- ops.oobbuf = NULL;
+ printf("block 0x%08lx NOT marked as bad! ERROR %d\n",
+ (ulong) addr, ret);
}
- ops.retlen = ops.oobretlen = 0;
+ return 1;
+ }
+
+ if (strncmp(argv[1], "dump", 4) == 0) {
+ ofs = simple_strtoul(argv[2], NULL, 16);
+ len = blocksize;
+
+ if (argc == 4)
+ len = simple_strtoul(argv[3], NULL, 16);
- onenand_read_oob(&onenand_mtd, ofs, &ops);
+ onenand_block_read(ofs, len, &retlen, NULL, 0);
return 0;
}
@@ -168,9 +372,12 @@ U_BOOT_CMD(
onenand, 6, 1, do_onenand,
"onenand - OneNAND sub-system\n",
"info - show available OneNAND devices\n"
- "onenand read[.oob] addr ofs len - read data at ofs with len to addr\n"
- "onenand write addr ofs len - write data at ofs with len from addr\n"
- "onenand erase saddr eaddr - erase block start addr to end addr\n"
- "onenand block[.oob] addr block [page] [len] - "
- "read data with (block [, page]) to addr"
+ "badb[locks] start_addr end_addr - Test & Mark bad blocks\n"
+ "onenand read[.oob] addr ofs len"
+ " - read data at ofs with len to addr\n"
+ "onenand write[.oob] addr ofs len"
+ " - write data at ofs with len from addr\n"
+ "onenand erase[force] [block] ofs len"
+ " - erase [block] at ofs with len\n"
+ "onenand markbad off - mark bad block at offset (UNSAFE)\n"
);
More information about the U-Boot
mailing list