public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
From: Boris Brezillon <boris.brezillon@free-electrons.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [linux-sunxi] [RFC 11/11] mtd/nand: Sunxi NAND boot partition definitions
Date: Sun, 14 Jun 2015 13:31:13 +0200	[thread overview]
Message-ID: <20150614133113.47b29959@bbrezillon> (raw)
In-Reply-To: <55757DE8.6070708@ultimaker.com>

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.

Best Regards,

Boris

-- 
Boris Brezillon, Free Electrons
Embedded Linux and Kernel engineering
http://free-electrons.com

  reply	other threads:[~2015-06-14 11:31 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 [this message]
2015-06-15  8:00                 ` Hans de Goede
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=20150614133113.47b29959@bbrezillon \
    --to=boris.brezillon@free-electrons.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