[U-Boot] [PATCH v2] nand: fix up badblock skipping
Jiri Valek
jirik.valek at seznam.cz
Tue Feb 5 09:03:19 UTC 2019
Hi all,
yes you are right. I had checked out older version of u-boot.
off + mtd->erasesize
this is OK. I`m sending patch v2.
On 08. 12. 18 17:16, Stefano Babic wrote:
> 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
>
---
Changes for v2:
- Repaired "mtd" to struct pinter
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 c628f3adec..ddaaba87a3 100644
--- a/drivers/mtd/nand/raw/mxs_nand_spl.c
+++ b/drivers/mtd/nand/raw/mxs_nand_spl.c
@@ -240,6 +240,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;
--
2.11.0
More information about the U-Boot
mailing list