public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
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

  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