Positive return values of imx_bbu_write_firmware() so far indicate
the last block that has been written to. This value is unused, so
return values > 0 to indicate if there are new bad blocks. This
information can be used in the next step to know if the DBBT has
to be rewritten.

Signed-off-by: Sascha Hauer <[email protected]>
---
 common/imx-bbu-nand-fcb.c | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/common/imx-bbu-nand-fcb.c b/common/imx-bbu-nand-fcb.c
index 0c7f75c..3e1f35f 100644
--- a/common/imx-bbu-nand-fcb.c
+++ b/common/imx-bbu-nand-fcb.c
@@ -400,7 +400,7 @@ static int imx_bbu_firmware_start_block(struct mtd_info 
*mtd, int num)
 static int imx_bbu_write_firmware(struct mtd_info *mtd, unsigned num, void 
*buf,
                                  size_t len)
 {
-       int ret, i;
+       int ret, i, newbadblock = 0;
        int num_blocks = imx_bbu_firmware_max_blocks(mtd);
        int block = imx_bbu_firmware_start_block(mtd, num);
 
@@ -437,6 +437,7 @@ static int imx_bbu_write_firmware(struct mtd_info *mtd, 
unsigned num, void *buf,
                if (ret == -EIO) {
                        block++;
                        num_blocks--;
+                       newbadblock = 1;
                        continue;
                }
 
@@ -449,7 +450,7 @@ static int imx_bbu_write_firmware(struct mtd_info *mtd, 
unsigned num, void *buf,
                num_blocks--;
        }
 
-       return block;
+       return newbadblock;
 }
 
 static void *dbbt_data_create(struct mtd_info *mtd)
-- 
2.7.0


_______________________________________________
barebox mailing list
[email protected]
http://lists.infradead.org/mailman/listinfo/barebox

Reply via email to