[U-Boot] [PATCH] nand: fix up badblock skipping
Stefano Babic
sbabic at denx.de
Sat Dec 8 16:16:49 UTC 2018
Hi Jiri, Miquel,
On 24/10/18 15:56, Miquel Raynal wrote:
> Hi Jiri,
>
> Jiri Valek <jirik.valek at seznam.cz> wrote on Fri, 19 Oct 2018 10:15:55
> +0200:
>
>> From: Jiri Valek <valek at 2n.cz>
>>
>> Currently the badblock skipping fails. SPL loader fails to boot from NAND.
>> End up on message "SPL: failed to boot from all boot devices"
>>
>> Signed-off-by: Jiri Valek <jirik.valek at seznam.cz>
>> ---
>>
>> drivers/mtd/nand/raw/mxs_nand_spl.c | 1 +
>> 1 file changed, 1 insertion(+)
>>
>> diff --git a/drivers/mtd/nand/raw/mxs_nand_spl.c b/drivers/mtd/nand/raw/mxs_nand_spl.c
>> index 2d7bbe83cc..56618c95dc 100644
>> --- a/drivers/mtd/nand/raw/mxs_nand_spl.c
>> +++ b/drivers/mtd/nand/raw/mxs_nand_spl.c
>> @@ -239,6 +239,7 @@ int nand_spl_load_image(uint32_t offs, unsigned int size, void *buf)
>> */
>> while (is_badblock(mtd, offs, 1)) {
>> page = page + nand_page_per_block;
>> + offs = offs + mtd.erasesize;
>> /* Check i we've reached the end of flash. */
>> if (page >= mtd->size >> chip->page_shift)
>> return -ENOMEM;
>
> Reviewed-by: Miquel Raynal <miquel.raynal at bootlin.com>
>
I agree that secto must be skipped, but implementation looks wrong. mtd
is a pointer, and the above line should be off + mtd->erasesize. Do you
build with it ? I tried to apply it, as expected it cannot be compiled
clean.
Best regards,
Stefano Babic
--
=====================================================================
DENX Software Engineering GmbH, Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-53 Fax: +49-8142-66989-80 Email: sbabic at denx.de
=====================================================================
More information about the U-Boot
mailing list