From: Enric Balletbo Serra <eballetbo@gmail.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCHv7 6/7] igep00x0: UBIize
Date: Tue, 19 Apr 2016 12:15:28 +0200 [thread overview]
Message-ID: <CAFqH_51_qRP9x=u6ZUjS84aV0ps+S=G-0mE6uG9PmRBH5g42fg@mail.gmail.com> (raw)
In-Reply-To: <CAFqH_50RnfNqFVJ420Kj-LxgnfxxR3hyiAz05vU0FLk5-aBxrQ@mail.gmail.com>
Hi Ladislav,
2016-04-13 13:39 GMT+02:00 Enric Balletbo Serra <eballetbo@gmail.com>:
> 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?
>
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
next prev parent reply other threads:[~2016-04-19 10:15 UTC|newest]
Thread overview: 28+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-01-17 3:09 [U-Boot] [PATCHv4 0/7] spl: Lightweight UBI and UBI fastmap support Ladislav Michl
2016-01-17 3:11 ` [U-Boot] [PATCHv4 1/7] mtd: Sort subsystem directories aplhabeticaly in Makefile Ladislav Michl
2016-01-21 6:04 ` Heiko Schocher
2016-01-17 3:12 ` [U-Boot] [PATCHv4 2/7] nand_spl_simple: Add a simple NAND read function Ladislav Michl
2016-01-21 6:05 ` Heiko Schocher
2016-01-17 3:13 ` [U-Boot] [PATCHv4 3/7] spl: Lightweight UBI and UBI fastmap support Ladislav Michl
2016-01-17 3:13 ` [U-Boot] [PATCHv4 4/7] spl: support loading from UBI volumes Ladislav Michl
2016-01-21 6:06 ` Heiko Schocher
2016-01-17 3:15 ` [U-Boot] [PATCHv5 5/7] spl: zImage support in Falcon mode Ladislav Michl
2016-01-21 6:07 ` Heiko Schocher
2016-01-17 3:16 ` [U-Boot] [PATCHv7 6/7] igep00x0: UBIize Ladislav Michl
2016-01-21 6:07 ` Heiko Schocher
2016-01-22 22:35 ` Enric Balletbo Serra
2016-01-24 13:44 ` Ladislav Michl
2016-01-25 6:39 ` Heiko Schocher
2016-01-25 7:26 ` Enric Balletbo Serra
2016-01-25 8:13 ` Heiko Schocher
2016-01-25 15:56 ` Ladislav Michl
2016-01-25 16:56 ` Enric Balletbo Serra
2016-04-13 5:19 ` Heiko Schocher
2016-04-13 11:39 ` Enric Balletbo Serra
2016-04-19 10:15 ` Enric Balletbo Serra [this message]
2016-06-04 20:48 ` [U-Boot] [RFC] runtime mtdparts_default; was: " Ladislav Michl
2016-01-25 15:13 ` [U-Boot] [PATCHv7 6/7] " Tom Rini
2016-01-25 15:35 ` Heiko Schocher
2016-01-17 3:16 ` [U-Boot] [PATCHv4 7/7] igep00x0: Falcon mode Ladislav Michl
2016-01-21 6:08 ` Heiko Schocher
2016-01-22 22:38 ` Enric Balletbo Serra
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to='CAFqH_51_qRP9x=u6ZUjS84aV0ps+S=G-0mE6uG9PmRBH5g42fg@mail.gmail.com' \
--to=eballetbo@gmail.com \
--cc=u-boot@lists.denx.de \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.