From be16aba5ce8d04b8ca0ebd5ac95d878064e44bd8 Mon Sep 17 00:00:00 2001 From: Stefan Roese Date: Fri, 5 Sep 2014 09:57:01 +0200 Subject: mtd: nand: Fix length bug in ioread16_rep() and iowrite16_rep() The ioread16_rep() and iowrite16_rep() implementations are U-Boot specific and have been introduced with the Linux MTD v3.14 sync. While introducing these functions, the length for the loop has been miscalculated. The ">> 1" is already present in the caller. So lets remove it in the function. Tested on omap3_ha. Signed-off-by: Stefan Roese Cc: Heiko Schocher Cc: Tom Rini Cc: Scott Wood Acked-by: Heiko Schocher diff --git a/drivers/mtd/nand/nand_base.c b/drivers/mtd/nand/nand_base.c index 7153e3c..0b6e7ee 100644 --- a/drivers/mtd/nand/nand_base.c +++ b/drivers/mtd/nand/nand_base.c @@ -308,8 +308,7 @@ static void ioread16_rep(void *addr, void *buf, int len) { int i; u16 *p = (u16 *) buf; - len >>= 1; - + for (i = 0; i < len; i++) p[i] = readw(addr); } @@ -318,7 +317,6 @@ static void iowrite16_rep(void *addr, void *buf, int len) { int i; u16 *p = (u16 *) buf; - len >>= 1; for (i = 0; i < len; i++) writew(p[i], addr); -- cgit v0.10.2