public inbox for linux-mtd@lists.infradead.org
 help / color / mirror / Atom feed
* [PATCH] Bug in 2.6.10 mtd driver for physmem mapped flash chips
@ 2004-12-27 15:20 K.F.J. Martens
  0 siblings, 0 replies; only message in thread
From: K.F.J. Martens @ 2004-12-27 15:20 UTC (permalink / raw)
  To: linux-mtd; +Cc: dwmw2, trivial, linux-kernel


The patch below fixes a small but fatal bug in the code that handles
non-buswidth-aligned writes. The problem is that the code used the same
index in both map_word and buf, therefore putting the wrong words in the
map_word that partially contains old data and partially contains new
data. The result: corrupt data is being written.

Signed-off-by: Koen Martens <kmartens@sonologic.nl>

--- linux-2.6.10/include/linux/mtd/map.h        2004-12-24 22:34:31.000000000 +0100
+++ linux-2.6.10-gmc/include/linux/mtd/map.h    2004-12-27 15:59:02.631211329 +0100
@@ -322,7 +322,7 @@ static inline map_word map_word_load_par
                        bitpos = (map_bankwidth(map)-1-i)*8;
 #endif
                        orig.x[0] &= ~(0xff << bitpos);
-                       orig.x[0] |= buf[i] << bitpos;
+                       orig.x[0] |= buf[i-start] << bitpos;
                }
        }
        return orig;

-- 
K.F.J. Martens, Sonologic, http://www.sonologic.nl/
Networking, embedded systems, unix expertise, artificial intelligence.
Public PGP key: http://www.metro.cx/pubkey-gmc.asc
Wondering about the funny attachment your mail program
can't read? Visit http://www.openpgp.org/

^ permalink raw reply	[flat|nested] only message in thread

only message in thread, other threads:[~2004-12-27 15:21 UTC | newest]

Thread overview: (only message) (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2004-12-27 15:20 [PATCH] Bug in 2.6.10 mtd driver for physmem mapped flash chips K.F.J. Martens

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox