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

[Enric Balletbo Serra]

Hi Ladislav,

2016-04-13 13:39 GMT+02:00 Enric Balletbo Serra <eballetbo at gmail.com>:
> 2016-04-13 7:19 GMT+02:00 Heiko Schocher <hs at 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 at 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