annotate yaffs_mtdif.c @ 66:f5ca021b3db5

Check the return value of mtd->write_ecc() calls. If not done, write failures will not be detected. Thanks to Artis Kugevics.
author luc <luc>
date Tue, 02 Aug 2005 19:18:44 +0000
parents dfb2a2e863b4
children 0113df80498c
Ignore whitespace changes - Everywhere: Within whitespace: At end of lines:
rev   line source
7
9fd733a84a74 *** empty log message ***
charles <charles>
parents:
diff changeset
1 /*
9fd733a84a74 *** empty log message ***
charles <charles>
parents:
diff changeset
2 * YAFFS: Yet another FFS. A NAND-flash specific file system.
9fd733a84a74 *** empty log message ***
charles <charles>
parents:
diff changeset
3 * yaffs_mtdif.c NAND mtd wrapper functions.
9fd733a84a74 *** empty log message ***
charles <charles>
parents:
diff changeset
4 *
9fd733a84a74 *** empty log message ***
charles <charles>
parents:
diff changeset
5 * Copyright (C) 2002 Aleph One Ltd.
9fd733a84a74 *** empty log message ***
charles <charles>
parents:
diff changeset
6 * for Toby Churchill Ltd and Brightstar Engineering
9fd733a84a74 *** empty log message ***
charles <charles>
parents:
diff changeset
7 *
9fd733a84a74 *** empty log message ***
charles <charles>
parents:
diff changeset
8 * Created by Charles Manning <charles@aleph1.co.uk>
9fd733a84a74 *** empty log message ***
charles <charles>
parents:
diff changeset
9 *
9fd733a84a74 *** empty log message ***
charles <charles>
parents:
diff changeset
10 * This program is free software; you can redistribute it and/or modify
9fd733a84a74 *** empty log message ***
charles <charles>
parents:
diff changeset
11 * it under the terms of the GNU General Public License version 2 as
9fd733a84a74 *** empty log message ***
charles <charles>
parents:
diff changeset
12 * published by the Free Software Foundation.
9fd733a84a74 *** empty log message ***
charles <charles>
parents:
diff changeset
13 *
9fd733a84a74 *** empty log message ***
charles <charles>
parents:
diff changeset
14 */
9fd733a84a74 *** empty log message ***
charles <charles>
parents:
diff changeset
15
66
f5ca021b3db5 Check the return value of mtd->write_ecc() calls.
luc <luc>
parents: 65
diff changeset
16 const char *yaffs_mtdif_c_version = "$Id: yaffs_mtdif.c,v 1.9 2005-08-02 19:18:44 luc Exp $";
45
1f2f55618d63 If kernel/config.h is included in yportenv.h, then yportenv.h must be
marty <marty>
parents: 43
diff changeset
17
1f2f55618d63 If kernel/config.h is included in yportenv.h, then yportenv.h must be
marty <marty>
parents: 43
diff changeset
18 #include "yportenv.h"
7
9fd733a84a74 *** empty log message ***
charles <charles>
parents:
diff changeset
19
49
05fb5e11130f Remove CONFIG_YAFFS_USE_CHUNK_SIZE
marty <marty>
parents: 45
diff changeset
20 #ifdef CONFIG_YAFFS_YAFFS1
7
9fd733a84a74 *** empty log message ***
charles <charles>
parents:
diff changeset
21
9fd733a84a74 *** empty log message ***
charles <charles>
parents:
diff changeset
22 #include "yaffs_mtdif.h"
9fd733a84a74 *** empty log message ***
charles <charles>
parents:
diff changeset
23
9fd733a84a74 *** empty log message ***
charles <charles>
parents:
diff changeset
24 #include "linux/mtd/mtd.h"
9fd733a84a74 *** empty log message ***
charles <charles>
parents:
diff changeset
25 #include "linux/types.h"
9fd733a84a74 *** empty log message ***
charles <charles>
parents:
diff changeset
26 #include "linux/time.h"
9fd733a84a74 *** empty log message ***
charles <charles>
parents:
diff changeset
27 #include "linux/mtd/nand.h"
9fd733a84a74 *** empty log message ***
charles <charles>
parents:
diff changeset
28
43
ea3f08fe855b applied patch 13-sparse-fix--make-some-symbol-static.diff
marty <marty>
parents: 42
diff changeset
29 static struct nand_oobinfo yaffs_oobinfo = {
42
b965eaa9d6e1 applied patch 11-sparse-fix--use-C99-struct-initializer.diff
marty <marty>
parents: 23
diff changeset
30 .useecc = 1,
65
dfb2a2e863b4 Initialize the field eccbytes in nand_oobinfo.
luc <luc>
parents: 57
diff changeset
31 #if (LINUX_VERSION_CODE >= KERNEL_VERSION(2,6,8))
dfb2a2e863b4 Initialize the field eccbytes in nand_oobinfo.
luc <luc>
parents: 57
diff changeset
32 .eccbytes = 6,
dfb2a2e863b4 Initialize the field eccbytes in nand_oobinfo.
luc <luc>
parents: 57
diff changeset
33 #endif
42
b965eaa9d6e1 applied patch 11-sparse-fix--use-C99-struct-initializer.diff
marty <marty>
parents: 23
diff changeset
34 .eccpos = {8, 9, 10, 13, 14, 15}
7
9fd733a84a74 *** empty log message ***
charles <charles>
parents:
diff changeset
35 };
9fd733a84a74 *** empty log message ***
charles <charles>
parents:
diff changeset
36
43
ea3f08fe855b applied patch 13-sparse-fix--make-some-symbol-static.diff
marty <marty>
parents: 42
diff changeset
37 static struct nand_oobinfo yaffs_noeccinfo = {
42
b965eaa9d6e1 applied patch 11-sparse-fix--use-C99-struct-initializer.diff
marty <marty>
parents: 23
diff changeset
38 .useecc = 0,
7
9fd733a84a74 *** empty log message ***
charles <charles>
parents:
diff changeset
39 };
9fd733a84a74 *** empty log message ***
charles <charles>
parents:
diff changeset
40
9fd733a84a74 *** empty log message ***
charles <charles>
parents:
diff changeset
41
23
7942535c12c6 Apply Luc's patches
charles <charles>
parents: 7
diff changeset
42 int nandmtd_WriteChunkToNAND(yaffs_Device *dev,int chunkInNAND,const __u8 *data, const yaffs_Spare *spare)
7
9fd733a84a74 *** empty log message ***
charles <charles>
parents:
diff changeset
43 {
9fd733a84a74 *** empty log message ***
charles <charles>
parents:
diff changeset
44 struct mtd_info *mtd = (struct mtd_info *)(dev->genericDevice);
9fd733a84a74 *** empty log message ***
charles <charles>
parents:
diff changeset
45 size_t dummy;
9fd733a84a74 *** empty log message ***
charles <charles>
parents:
diff changeset
46 int retval = 0;
9fd733a84a74 *** empty log message ***
charles <charles>
parents:
diff changeset
47
9fd733a84a74 *** empty log message ***
charles <charles>
parents:
diff changeset
48 loff_t addr = ((loff_t)chunkInNAND) * dev->nBytesPerChunk;
9fd733a84a74 *** empty log message ***
charles <charles>
parents:
diff changeset
49
9fd733a84a74 *** empty log message ***
charles <charles>
parents:
diff changeset
50 __u8 *spareAsBytes = (__u8 *)spare;
9fd733a84a74 *** empty log message ***
charles <charles>
parents:
diff changeset
51
9fd733a84a74 *** empty log message ***
charles <charles>
parents:
diff changeset
52 if(data && spare)
9fd733a84a74 *** empty log message ***
charles <charles>
parents:
diff changeset
53 {
9fd733a84a74 *** empty log message ***
charles <charles>
parents:
diff changeset
54 if(dev->useNANDECC)
66
f5ca021b3db5 Check the return value of mtd->write_ecc() calls.
luc <luc>
parents: 65
diff changeset
55 retval = mtd->write_ecc(mtd,addr,dev->nBytesPerChunk,&dummy,data,spareAsBytes,&yaffs_oobinfo);
7
9fd733a84a74 *** empty log message ***
charles <charles>
parents:
diff changeset
56 else
66
f5ca021b3db5 Check the return value of mtd->write_ecc() calls.
luc <luc>
parents: 65
diff changeset
57 retval = mtd->write_ecc(mtd,addr,dev->nBytesPerChunk,&dummy,data,spareAsBytes,&yaffs_noeccinfo);
7
9fd733a84a74 *** empty log message ***
charles <charles>
parents:
diff changeset
58 }
9fd733a84a74 *** empty log message ***
charles <charles>
parents:
diff changeset
59 else
9fd733a84a74 *** empty log message ***
charles <charles>
parents:
diff changeset
60 {
9fd733a84a74 *** empty log message ***
charles <charles>
parents:
diff changeset
61 if(data)
9fd733a84a74 *** empty log message ***
charles <charles>
parents:
diff changeset
62 retval = mtd->write(mtd,addr,dev->nBytesPerChunk,&dummy,data);
9fd733a84a74 *** empty log message ***
charles <charles>
parents:
diff changeset
63 if(spare)
9fd733a84a74 *** empty log message ***
charles <charles>
parents:
diff changeset
64 retval = mtd->write_oob(mtd,addr,YAFFS_BYTES_PER_SPARE,&dummy,spareAsBytes);
9fd733a84a74 *** empty log message ***
charles <charles>
parents:
diff changeset
65 }
9fd733a84a74 *** empty log message ***
charles <charles>
parents:
diff changeset
66
9fd733a84a74 *** empty log message ***
charles <charles>
parents:
diff changeset
67 if (retval == 0)
9fd733a84a74 *** empty log message ***
charles <charles>
parents:
diff changeset
68 return YAFFS_OK;
9fd733a84a74 *** empty log message ***
charles <charles>
parents:
diff changeset
69 else
9fd733a84a74 *** empty log message ***
charles <charles>
parents:
diff changeset
70 return YAFFS_FAIL;
9fd733a84a74 *** empty log message ***
charles <charles>
parents:
diff changeset
71 }
9fd733a84a74 *** empty log message ***
charles <charles>
parents:
diff changeset
72
9fd733a84a74 *** empty log message ***
charles <charles>
parents:
diff changeset
73 int nandmtd_ReadChunkFromNAND(yaffs_Device *dev,int chunkInNAND, __u8 *data, yaffs_Spare *spare)
9fd733a84a74 *** empty log message ***
charles <charles>
parents:
diff changeset
74 {
9fd733a84a74 *** empty log message ***
charles <charles>
parents:
diff changeset
75 struct mtd_info *mtd = (struct mtd_info *)(dev->genericDevice);
9fd733a84a74 *** empty log message ***
charles <charles>
parents:
diff changeset
76 size_t dummy;
9fd733a84a74 *** empty log message ***
charles <charles>
parents:
diff changeset
77 int retval = 0;
9fd733a84a74 *** empty log message ***
charles <charles>
parents:
diff changeset
78
9fd733a84a74 *** empty log message ***
charles <charles>
parents:
diff changeset
79 loff_t addr = ((loff_t)chunkInNAND) * dev->nBytesPerChunk;
9fd733a84a74 *** empty log message ***
charles <charles>
parents:
diff changeset
80
9fd733a84a74 *** empty log message ***
charles <charles>
parents:
diff changeset
81 __u8 *spareAsBytes = (__u8 *)spare;
9fd733a84a74 *** empty log message ***
charles <charles>
parents:
diff changeset
82
9fd733a84a74 *** empty log message ***
charles <charles>
parents:
diff changeset
83 if(data && spare)
9fd733a84a74 *** empty log message ***
charles <charles>
parents:
diff changeset
84 {
9fd733a84a74 *** empty log message ***
charles <charles>
parents:
diff changeset
85 if(dev->useNANDECC)
9fd733a84a74 *** empty log message ***
charles <charles>
parents:
diff changeset
86 { // Careful, this call adds 2 ints to the end of the spare data. Calling function should
9fd733a84a74 *** empty log message ***
charles <charles>
parents:
diff changeset
87 // allocate enough memory for spare, i.e. [YAFFS_BYTES_PER_SPARE+2*sizeof(int)].
9fd733a84a74 *** empty log message ***
charles <charles>
parents:
diff changeset
88 retval = mtd->read_ecc(mtd,addr,dev->nBytesPerChunk,&dummy,data,spareAsBytes,&yaffs_oobinfo);
9fd733a84a74 *** empty log message ***
charles <charles>
parents:
diff changeset
89 }
9fd733a84a74 *** empty log message ***
charles <charles>
parents:
diff changeset
90 else
9fd733a84a74 *** empty log message ***
charles <charles>
parents:
diff changeset
91 {
9fd733a84a74 *** empty log message ***
charles <charles>
parents:
diff changeset
92 retval = mtd->read_ecc(mtd,addr,dev->nBytesPerChunk,&dummy,data,spareAsBytes,&yaffs_noeccinfo);
9fd733a84a74 *** empty log message ***
charles <charles>
parents:
diff changeset
93 }
9fd733a84a74 *** empty log message ***
charles <charles>
parents:
diff changeset
94 }
9fd733a84a74 *** empty log message ***
charles <charles>
parents:
diff changeset
95 else
9fd733a84a74 *** empty log message ***
charles <charles>
parents:
diff changeset
96 {
9fd733a84a74 *** empty log message ***
charles <charles>
parents:
diff changeset
97 if(data)
9fd733a84a74 *** empty log message ***
charles <charles>
parents:
diff changeset
98 retval = mtd->read(mtd,addr,dev->nBytesPerChunk,&dummy,data);
9fd733a84a74 *** empty log message ***
charles <charles>
parents:
diff changeset
99 if(spare)
9fd733a84a74 *** empty log message ***
charles <charles>
parents:
diff changeset
100 retval = mtd->read_oob(mtd,addr,YAFFS_BYTES_PER_SPARE,&dummy,spareAsBytes);
9fd733a84a74 *** empty log message ***
charles <charles>
parents:
diff changeset
101 }
9fd733a84a74 *** empty log message ***
charles <charles>
parents:
diff changeset
102
9fd733a84a74 *** empty log message ***
charles <charles>
parents:
diff changeset
103 if (retval == 0)
9fd733a84a74 *** empty log message ***
charles <charles>
parents:
diff changeset
104 return YAFFS_OK;
9fd733a84a74 *** empty log message ***
charles <charles>
parents:
diff changeset
105 else
9fd733a84a74 *** empty log message ***
charles <charles>
parents:
diff changeset
106 return YAFFS_FAIL;
9fd733a84a74 *** empty log message ***
charles <charles>
parents:
diff changeset
107 }
9fd733a84a74 *** empty log message ***
charles <charles>
parents:
diff changeset
108
9fd733a84a74 *** empty log message ***
charles <charles>
parents:
diff changeset
109 // Callback not needed for NAND
9fd733a84a74 *** empty log message ***
charles <charles>
parents:
diff changeset
110 #if 0
9fd733a84a74 *** empty log message ***
charles <charles>
parents:
diff changeset
111 static void nandmtd_EraseCallback(struct erase_info *ei)
9fd733a84a74 *** empty log message ***
charles <charles>
parents:
diff changeset
112 {
9fd733a84a74 *** empty log message ***
charles <charles>
parents:
diff changeset
113 yaffs_Device *dev = (yaffs_Device *)ei->priv;
9fd733a84a74 *** empty log message ***
charles <charles>
parents:
diff changeset
114 up(&dev->sem);
9fd733a84a74 *** empty log message ***
charles <charles>
parents:
diff changeset
115 }
9fd733a84a74 *** empty log message ***
charles <charles>
parents:
diff changeset
116 #endif
9fd733a84a74 *** empty log message ***
charles <charles>
parents:
diff changeset
117
9fd733a84a74 *** empty log message ***
charles <charles>
parents:
diff changeset
118
9fd733a84a74 *** empty log message ***
charles <charles>
parents:
diff changeset
119 int nandmtd_EraseBlockInNAND(yaffs_Device *dev, int blockNumber)
9fd733a84a74 *** empty log message ***
charles <charles>
parents:
diff changeset
120 {
9fd733a84a74 *** empty log message ***
charles <charles>
parents:
diff changeset
121 struct mtd_info *mtd = (struct mtd_info *)(dev->genericDevice);
9fd733a84a74 *** empty log message ***
charles <charles>
parents:
diff changeset
122 __u32 addr = ((loff_t) blockNumber) * dev->nBytesPerChunk * dev->nChunksPerBlock;
9fd733a84a74 *** empty log message ***
charles <charles>
parents:
diff changeset
123 struct erase_info ei;
9fd733a84a74 *** empty log message ***
charles <charles>
parents:
diff changeset
124 int retval = 0;
9fd733a84a74 *** empty log message ***
charles <charles>
parents:
diff changeset
125
9fd733a84a74 *** empty log message ***
charles <charles>
parents:
diff changeset
126 ei.mtd = mtd;
9fd733a84a74 *** empty log message ***
charles <charles>
parents:
diff changeset
127 ei.addr = addr;
9fd733a84a74 *** empty log message ***
charles <charles>
parents:
diff changeset
128 ei.len = dev->nBytesPerChunk * dev->nChunksPerBlock;
9fd733a84a74 *** empty log message ***
charles <charles>
parents:
diff changeset
129 ei.time = 1000;
9fd733a84a74 *** empty log message ***
charles <charles>
parents:
diff changeset
130 ei.retries = 2;
9fd733a84a74 *** empty log message ***
charles <charles>
parents:
diff changeset
131 ei.callback = NULL;
9fd733a84a74 *** empty log message ***
charles <charles>
parents:
diff changeset
132 ei.priv = (u_long)dev;
9fd733a84a74 *** empty log message ***
charles <charles>
parents:
diff changeset
133
9fd733a84a74 *** empty log message ***
charles <charles>
parents:
diff changeset
134 // Todo finish off the ei if required
9fd733a84a74 *** empty log message ***
charles <charles>
parents:
diff changeset
135
9fd733a84a74 *** empty log message ***
charles <charles>
parents:
diff changeset
136 sema_init(&dev->sem,0);
9fd733a84a74 *** empty log message ***
charles <charles>
parents:
diff changeset
137
9fd733a84a74 *** empty log message ***
charles <charles>
parents:
diff changeset
138 retval = mtd->erase(mtd,&ei);
9fd733a84a74 *** empty log message ***
charles <charles>
parents:
diff changeset
139
9fd733a84a74 *** empty log message ***
charles <charles>
parents:
diff changeset
140 //No need for callback
9fd733a84a74 *** empty log message ***
charles <charles>
parents:
diff changeset
141 // down(&dev->sem); // Wait for the erasure to complete
9fd733a84a74 *** empty log message ***
charles <charles>
parents:
diff changeset
142
9fd733a84a74 *** empty log message ***
charles <charles>
parents:
diff changeset
143 if (retval == 0)
9fd733a84a74 *** empty log message ***
charles <charles>
parents:
diff changeset
144 return YAFFS_OK;
9fd733a84a74 *** empty log message ***
charles <charles>
parents:
diff changeset
145 else
9fd733a84a74 *** empty log message ***
charles <charles>
parents:
diff changeset
146 return YAFFS_FAIL;
9fd733a84a74 *** empty log message ***
charles <charles>
parents:
diff changeset
147 }
9fd733a84a74 *** empty log message ***
charles <charles>
parents:
diff changeset
148
9fd733a84a74 *** empty log message ***
charles <charles>
parents:
diff changeset
149 int nandmtd_InitialiseNAND(yaffs_Device *dev)
9fd733a84a74 *** empty log message ***
charles <charles>
parents:
diff changeset
150 {
9fd733a84a74 *** empty log message ***
charles <charles>
parents:
diff changeset
151 return YAFFS_OK;
9fd733a84a74 *** empty log message ***
charles <charles>
parents:
diff changeset
152 }
9fd733a84a74 *** empty log message ***
charles <charles>
parents:
diff changeset
153
49
05fb5e11130f Remove CONFIG_YAFFS_USE_CHUNK_SIZE
marty <marty>
parents: 45
diff changeset
154 #endif // CONFIG_YAFFS_YAFFS1
7
9fd733a84a74 *** empty log message ***
charles <charles>
parents:
diff changeset
155