summaryrefslogtreecommitdiff
path: root/drivers/mtd/nand
diff options
context:
space:
mode:
authorStefan Roese <sr@denx.de>2014-09-05 09:57:01 +0200
committerScott Wood <scottwood@freescale.com>2014-09-25 13:43:00 -0500
commitbe16aba5ce8d04b8ca0ebd5ac95d878064e44bd8 (patch)
tree4a663dff5b74a59d3c900d9d4bc5560004962bcb /drivers/mtd/nand
parent2f665945b39a9972260cbcf60d5e7f2a60587f5e (diff)
downloadu-boot-imx-be16aba5ce8d04b8ca0ebd5ac95d878064e44bd8.zip
u-boot-imx-be16aba5ce8d04b8ca0ebd5ac95d878064e44bd8.tar.gz
u-boot-imx-be16aba5ce8d04b8ca0ebd5ac95d878064e44bd8.tar.bz2
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 <sr@denx.de> Cc: Heiko Schocher <hs@denx.de> Cc: Tom Rini <trini@ti.com> Cc: Scott Wood <scottwood@freescale.com> Acked-by: Heiko Schocher <hs@denx.de>
Diffstat (limited to 'drivers/mtd/nand')
-rw-r--r--drivers/mtd/nand/nand_base.c4
1 files changed, 1 insertions, 3 deletions
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);