[PATCH V4 4/5] spl: spl_nand: Fix bad block handling in fitImage

2022-05-20 Thread sbabic
> If the fitImage has some bad block in fit image area, the
> offset must be recalulcated. This should be done always.
> After implementing it in mxs now is possible to call the function
> even for that platform.
> Cc: Fabio Estevam 
> Tested-By: Tim Harvey 
> Reviewed-by: Tom Rini 
> Signed-off-by: Michael Trimarchi 
Applied to u-boot-imx, master, thanks !

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: sba...@denx.de
=


[PATCH V4 4/5] spl: spl_nand: Fix bad block handling in fitImage

2022-05-15 Thread Michael Trimarchi
If the fitImage has some bad block in fit image area, the
offset must be recalulcated. This should be done always.
After implementing it in mxs now is possible to call the function
even for that platform.

Cc: Fabio Estevam 
Tested-By: Tim Harvey 
Reviewed-by: Tom Rini 
Signed-off-by: Michael Trimarchi 
---
V3->V4:
- none
V2->V3:
- Add tested-by from Tim
- Add ack from Han Xu
V1->V2:
- move out from RFC
---
 common/spl/spl_nand.c | 5 +
 1 file changed, 1 insertion(+), 4 deletions(-)

diff --git a/common/spl/spl_nand.c b/common/spl/spl_nand.c
index fc61b447a5..82a10ffa63 100644
--- a/common/spl/spl_nand.c
+++ b/common/spl/spl_nand.c
@@ -43,15 +43,12 @@ static ulong spl_nand_fit_read(struct spl_load_info *load, 
ulong offs,
   ulong size, void *dst)
 {
int err;
-#ifdef CONFIG_SYS_NAND_BLOCK_SIZE
ulong sector;
 
sector = *(int *)load->priv;
-   offs = sector + nand_spl_adjust_offset(sector, offs - sector);
-#else
offs *= load->bl_len;
size *= load->bl_len;
-#endif
+   offs = sector + nand_spl_adjust_offset(sector, offs - sector);
err = nand_spl_load_image(offs, size, dst);
if (err)
return 0;
-- 
2.25.1