All of lore.kernel.org
 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 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.