Hi Ladislav, 2016-04-13 13:39 GMT+02:00 Enric Balletbo Serra <eballe...@gmail.com>: > 2016-04-13 7:19 GMT+02:00 Heiko Schocher <h...@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 <la...@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... >>>> >>>> ladis >>> >>> >>> Your 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? >
If you don't have enough time maybe I can pickup the patches and finish the work, if it's ok. I'll have some time this weekend. > 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 _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot