From: Hans de Goede <hdegoede@redhat.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [linux-sunxi] [RFC 11/11] mtd/nand: Sunxi NAND boot partition definitions
Date: Mon, 15 Jun 2015 10:00:46 +0200 [thread overview]
Message-ID: <557E862E.7050408@redhat.com> (raw)
In-Reply-To: <20150614133113.47b29959@bbrezillon>
Hi,
On 14-06-15 13:31, Boris Brezillon wrote:
> Roy, Yassin,
>
> On Mon, 08 Jun 2015 13:35:04 +0200
> Roy Spliet <r.spliet@ultimaker.com> wrote:
>
>> Hello Yassin,
>>
>> Op 08-06-15 om 12:48 schreef Yassin:
>>> Hi Roy
>>> Thank you for working on this, I would like to suggest if you could implement separate control commands to switch ECC and Randomisation modes per partitions. I know this is not the best approach but it will provide more controls.
>> Thanks for the suggestion. However, for now I do not see the immediate
>> need for this in U-boot. U-boot should really only care about one thing:
>> load Linux. Therefore, the only partition it needs to be able to read is
>> whatever partition is used for storing boot data on. And if we only care
>> about one NAND partition, we don't need per-partition settings.
>> I agree that it would be nice to have eventually, but I think this
>> problem first needs to be solved in Linux properly. Then in U-boot we
>> can sync MTD up with Linux and we get the support we need "for free".
>> I'd personally recommend to take this approach over now first hacking up
>> all sorts of new commands in U-boot, as the latter will create more
>> problems with syncing up MTD and doing the right thing on the longer run.
>
> Actually, the more I think about the more I agree with Yassin's
> suggestion. We currently don't have any standard way to attach a
> specific setting to a partition (even in Linux, and I hope I'll be able
> to work on this aspect soon).
> So for now, I really think we should reuse the existing/standard way of
> declaring partitions in u-boot (mtdparts + mtdids variables + the
> mtdparts command) and a sunxi specific commands to configure the ECC
> and Randomizer config.
> Once we have settled on something in Linux, we will be able to port it
> to u-boot and get rid of these sunxi specific command, but in the
> meantime this will allow us to boot a linux kernel (and even flash an
> SPL) from u-boot without introducing heavy changes in the u-boot MTD
> layer.
Ack, note we do not even need any sunxi specific commands, we will
have separate nand code for the SPL and uboot itself, since the SPL
needs a simple mini read-only implementation. So we can compile the
parameters for the boot block vs the rest into the spl resp. uboot
binary directly.
Regards,
Hans
next prev parent reply other threads:[~2015-06-15 8:00 UTC|newest]
Thread overview: 49+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-06-05 11:52 [U-Boot] [RFC] Sunxi NAND support for U-Boot Roy Spliet
2015-06-05 11:52 ` [U-Boot] [RFC 01/11] mtd/nand: define struct nand_timings Roy Spliet
2015-06-05 11:52 ` [U-Boot] [RFC 02/11] mtd/nand: add ONFI timing mode to nand_timings converter Roy Spliet
2015-06-05 22:02 ` Scott Wood
2015-06-08 8:11 ` Roy Spliet
2015-06-08 8:34 ` [U-Boot] [linux-sunxi] " Michal Suchanek
2015-06-08 8:41 ` Roy Spliet
2015-06-14 11:59 ` Boris Brezillon
2015-06-08 20:24 ` [U-Boot] " Scott Wood
2015-06-10 8:33 ` Hans de Goede
2015-06-10 19:06 ` Scott Wood
2015-06-05 11:52 ` [U-Boot] [RFC 03/11] mtd/nand: support ONFI timing mode retrieval for non-ONFI Roy Spliet
2015-06-14 11:53 ` Boris Brezillon
2015-06-05 11:52 ` [U-Boot] [RFC 04/11] mtd/nand: add page status table (pst) Roy Spliet
2015-06-14 11:52 ` Boris Brezillon
2015-06-05 11:52 ` [U-Boot] [RFC 05/11] mtd/nand: take nand_ecc_ctrl initialization out of nand_scan_tail Roy Spliet
2015-06-14 11:50 ` Boris Brezillon
2015-06-05 11:52 ` [U-Boot] [RFC 06/11] mtd/nand: Add randomisation layer Roy Spliet
2015-06-14 11:47 ` Boris Brezillon
2015-06-05 11:52 ` [U-Boot] [RFC 07/11] mtd/nand Add Sunxi NAND driver Roy Spliet
2015-06-14 11:42 ` Boris Brezillon
2015-06-14 11:45 ` Boris Brezillon
2015-06-05 11:52 ` [U-Boot] [RFC 08/11] mtd/nand: Add DT definitions for Olimex Lime Roy Spliet
2015-06-14 11:39 ` Boris Brezillon
2015-06-05 11:52 ` [U-Boot] [RFC 09/11] sunxi/nand: Enable UBI and NAND commands Roy Spliet
2015-06-06 15:13 ` Hans de Goede
2015-06-06 15:36 ` Ian Campbell
2015-06-08 7:38 ` Roy Spliet
2015-06-08 9:12 ` Ian Campbell
2015-06-05 11:52 ` [U-Boot] [RFC 10/11] mtd/nand: Define bootcmd for nand Roy Spliet
2015-06-05 11:52 ` [U-Boot] [RFC 11/11] mtd/nand: Sunxi NAND boot partition definitions Roy Spliet
2015-06-07 16:48 ` [U-Boot] [linux-sunxi] " Michal Suchanek
2015-06-08 8:38 ` Roy Spliet
2015-06-08 8:54 ` Michal Suchanek
2015-06-08 9:11 ` Roy Spliet
2015-06-08 10:48 ` Yassin
2015-06-08 11:35 ` Roy Spliet
2015-06-14 11:31 ` Boris Brezillon
2015-06-15 8:00 ` Hans de Goede [this message]
2015-06-08 13:16 ` Hans de Goede
2015-06-08 13:56 ` Roy Spliet
2015-06-14 11:25 ` Boris Brezillon
2015-06-14 11:56 ` Michal Suchanek
2015-06-14 12:18 ` Boris Brezillon
2015-06-14 17:42 ` Michal Suchanek
2015-06-14 19:07 ` Boris Brezillon
2015-06-06 15:09 ` [U-Boot] [RFC] Sunxi NAND support for U-Boot Hans de Goede
2015-06-06 15:11 ` Hans de Goede
2015-06-14 11:13 ` Boris Brezillon
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=557E862E.7050408@redhat.com \
--to=hdegoede@redhat.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox