From mboxrd@z Thu Jan  1 00:00:00 1970
From: Stefan Roese <sr@denx.de>
Date: Wed, 24 Sep 2014 13:03:54 +0200
Subject: [U-Boot] [PATCH] mtd: nand: Fix length bug in ioread16_rep()
 and iowrite16_rep()
In-Reply-To: <54096F18.2000907@denx.de>
References: <1409903821-24398-1-git-send-email-sr@denx.de>
	<54096F18.2000907@denx.de>
Message-ID: <5422A51A.1000102@denx.de>
List-Id: <u-boot.lists.denx.de>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: u-boot@lists.denx.de

Hi Scott, Hi Tom,

On 05.09.2014 10:06, Heiko Schocher wrote:
>> 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>
>> ---
>>   drivers/mtd/nand/nand_base.c | 4 +---
>>   1 file changed, 1 insertion(+), 3 deletions(-)
>
> Good catch, thanks!
>
> Acked-by: Heiko Schocher <hs@denx.de>

This patch is also still missing. We really should add it in this 
upcoming release.

Thanks,
Stefan