2016-04-13 7:19 GMT+02:00 Heiko Schocher hs@denx.de:
Hello Enric,
Am 25.01.2016 um 17:56 schrieb Enric Balletbo Serra:
Hi Ladis,
2016-01-25 16:56 GMT+01:00 Ladislav Michl ladis@linux-mips.org:
Hi Enric,
On Mon, Jan 25, 2016 at 08:26:23AM +0100, Enric Balletbo Serra wrote:
The ROM boot on OMAP reads the first 4 blocks searching for the SPL, in production is a practice flash the SPL 4 times. OneNAND/NAND devices can have different block sizes and the OMAP ROM boot supports block sizes from 64KB to 512K. For IGEP boards in particular, at least there are boards that have block size of 128K and 256K. What I would meant here is set as default the mtdparts variable to reserve 2M for SPL, just to cover all the cases.
mtdparts=mtdparts=gpmc-nand.0:2m(SPL),-(UBI)\0
So far there was no ack or nack to yesterday's proposal making that dynamic; Both boot ROM and ubispl code thinks in terms of eraseblocks, only mtd needs fixed offset. So I'd like to see this offset calculated as 4*block_size, not some "worst case" value...
ladisYour proposal looks good to me.
Are there any updates for this patch? The UBI SPL support is pending for a while ...
I pushed my last (rebased to mainline) state to:
http://git.denx.de/?p=u-boot/u-boot-ubi.git;a=shortlog;h=refs/heads/ubi-spl
without the 2 igep00x0 board patches:
[U-Boot,PATCHv7,6/7] igep00x0: UBIize https://patchwork.ozlabs.org/patch/569214/
[U-Boot,PATCHv4,7/7] igep00x0: Falcon mode https://patchwork.ozlabs.org/patch/569215/
Could you please update your patches for the board support, so we can push the UBI SPL support to mainline?
Ladis, did you have a chance to look and solve the latest comments I did?
These patch series looks really interesting, like Heiko I would like to see these merged.
Thanks!
bye, Heiko -- DENX Software Engineering GmbH, Managing Director: Wolfgang Denk HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany