[U-Boot] [PATCHv7 6/7] igep00x0: UBIize

Enric Balletbo Serra
2016-04-13T11:39:17Z

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...
>>>
>>> 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?

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