yaffs: Add utils header file

[yaffs2.git] / yaffs_nand.c

diff --git a/yaffs_nand.c b/yaffs_nand.c
index f7810d0a0e370dff99207b3948a9a26351314b67..165d01004d6efde447d4b88a42f38c74dafa5fed 100644 (file)
--- a/yaffs_nand.c
+++ b/yaffs_nand.c
@@ -1,7 +1,7 @@
 /*
  * YAFFS: Yet Another Flash File System. A NAND-flash specific file system.
  *
- * Copyright (C) 2002-2010 Aleph One Ltd.
+ * Copyright (C) 2002-2011 Aleph One Ltd.
  *   for Toby Churchill Ltd and Brightstar Engineering
  *
  * Created by Charles Manning <charles@aleph1.co.uk>
@@ -13,31 +13,30 @@
 
 #include "yaffs_nand.h"
 #include "yaffs_tagscompat.h"
-#include "yaffs_tagsvalidity.h"
 
 #include "yaffs_getblockinfo.h"
+#include "yaffs_summary.h"
 
 int yaffs_rd_chunk_tags_nand(struct yaffs_dev *dev, int nand_chunk,
-                            u8 * buffer, struct yaffs_ext_tags *tags)
+                            u8 *buffer, struct yaffs_ext_tags *tags)
 {
        int result;
        struct yaffs_ext_tags local_tags;
-
-       int realigned_chunk = nand_chunk - dev->chunk_offset;
+       int flash_chunk = nand_chunk - dev->chunk_offset;
 
        dev->n_page_reads++;
 
-       /* If there are no tags provided, use local tags to get prioritised gc working */
+       /* If there are no tags provided use local tags. */
        if (!tags)
                tags = &local_tags;
 
        if (dev->param.read_chunk_tags_fn)
                result =
-                   dev->param.read_chunk_tags_fn(dev, realigned_chunk, buffer,
+                   dev->param.read_chunk_tags_fn(dev, flash_chunk, buffer,
                                                  tags);
        else
                result = yaffs_tags_compat_rd(dev,
-                                             realigned_chunk, buffer, tags);
+                                             flash_chunk, buffer, tags);
        if (tags && tags->ecc_result > YAFFS_ECC_RESULT_NO_ERROR) {
 
                struct yaffs_block_info *bi;
@@ -46,65 +45,61 @@ int yaffs_rd_chunk_tags_nand(struct yaffs_dev *dev, int nand_chunk,
                                          dev->param.chunks_per_block);
                yaffs_handle_chunk_error(dev, bi);
        }
-
        return result;
 }
 
 int yaffs_wr_chunk_tags_nand(struct yaffs_dev *dev,
-                            int nand_chunk,
-                            const u8 * buffer, struct yaffs_ext_tags *tags)
+                               int nand_chunk,
+                               const u8 *buffer, struct yaffs_ext_tags *tags)
 {
+       int result;
+       int flash_chunk = nand_chunk - dev->chunk_offset;
 
        dev->n_page_writes++;
 
-       nand_chunk -= dev->chunk_offset;
-
        if (tags) {
                tags->seq_number = dev->seq_number;
                tags->chunk_used = 1;
-               if (!yaffs_validate_tags(tags)) {
-                       yaffs_trace(YAFFS_TRACE_ERROR, "Writing uninitialised tags");
-                       YBUG();
-               }
                yaffs_trace(YAFFS_TRACE_WRITE,
                        "Writing chunk %d tags %d %d",
                        nand_chunk, tags->obj_id, tags->chunk_id);
        } else {
                yaffs_trace(YAFFS_TRACE_ERROR, "Writing with no tags");
-               YBUG();
-               return FALSE;
+               BUG();
+               return YAFFS_FAIL;
        }
 
        if (dev->param.write_chunk_tags_fn)
-               return dev->param.write_chunk_tags_fn(dev, nand_chunk, buffer,
-                                                     tags);
+               result = dev->param.write_chunk_tags_fn(dev, flash_chunk,
+                                                       buffer, tags);
        else
-               return yaffs_tags_compat_wr(dev, nand_chunk, buffer, tags);
+               result = yaffs_tags_compat_wr(dev, flash_chunk, buffer, tags);
+
+       yaffs_summary_add(dev, tags, nand_chunk);
+
+       return result;
 }
 
 int yaffs_mark_bad(struct yaffs_dev *dev, int block_no)
 {
        block_no -= dev->block_offset;
-
        if (dev->param.bad_block_fn)
                return dev->param.bad_block_fn(dev, block_no);
-       else
-               return yaffs_tags_compat_mark_bad(dev, block_no);
+
+       return yaffs_tags_compat_mark_bad(dev, block_no);
 }
 
 int yaffs_query_init_block_state(struct yaffs_dev *dev,
                                 int block_no,
                                 enum yaffs_block_state *state,
-                                u32 * seq_number)
+                                u32 *seq_number)
 {
        block_no -= dev->block_offset;
-
        if (dev->param.query_block_fn)
                return dev->param.query_block_fn(dev, block_no, state,
                                                 seq_number);
-       else
-               return yaffs_tags_compat_query_block(dev, block_no,
-                                                    state, seq_number);
+
+       return yaffs_tags_compat_query_block(dev, block_no, state, seq_number);
 }
 
 int yaffs_erase_block(struct yaffs_dev *dev, int flash_block)
@@ -112,11 +107,8 @@ int yaffs_erase_block(struct yaffs_dev *dev, int flash_block)
        int result;
 
        flash_block -= dev->block_offset;
-
        dev->n_erasures++;
-
        result = dev->param.erase_fn(dev, flash_block);
-
        return result;
 }