[U-Boot] [PATCH] mtd: nand: Fix length bug in ioread16_rep() and iowrite16_rep()

Stefan Roese sr at denx.de
Wed Sep 24 13:03:54 CEST 2014


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 at denx.de>
>> Cc: Heiko Schocher<hs at denx.de>
>> Cc: Tom Rini<trini at ti.com>
>> Cc: Scott Wood<scottwood at freescale.com>
>> ---
>>   drivers/mtd/nand/nand_base.c | 4 +---
>>   1 file changed, 1 insertion(+), 3 deletions(-)
>
> Good catch, thanks!
>
> Acked-by: Heiko Schocher <hs at denx.de>

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

Thanks,
Stefan



More information about the U-Boot mailing list