From: Roy Spliet <r.spliet@ultimaker.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [linux-sunxi] Re: [RFC 02/11] mtd/nand: add ONFI timing mode to nand_timings converter
Date: Mon, 08 Jun 2015 10:41:28 +0200 [thread overview]
Message-ID: <55755538.9050606@ultimaker.com> (raw)
In-Reply-To: <CAOMqctRDEmfM0i9wC9D5spH+nqgg45t80aYSvv2LRDcmJNmY1w@mail.gmail.com>
Hello Michal,
Op 08-06-15 om 10:34 schreef Michal Suchanek:
> On 8 June 2015 at 10:11, Roy Spliet <r.spliet@ultimaker.com> wrote:
>> Hello Scott et al.,
>>
>> Op 06-06-15 om 00:02 schreef Scott Wood:
>>
>> On Fri, 2015-06-05 at 13:52 +0200, Roy Spliet wrote:
>>
>> From: yassin <yassinjaffer@gmail.com>
>>
>> Signed-off-by: Roy Spliet <r.spliet@ultimaker.com>
>> ---
>> drivers/mtd/nand/Makefile | 2 +-
>> drivers/mtd/nand/nand_timings.c | 252
>> ++++++++++++++++++++++++++++++++++++++++
>> include/linux/mtd/nand.h | 3 +
>> 3 files changed, 256 insertions(+), 1 deletion(-)
>> create mode 100644 drivers/mtd/nand/nand_timings.c
>>
>> This code comes from Linux and yet I see no acknowledgement of that,
>> much less a statement of which version of Linux this was pulled from.
>>
>> Correct, my apologies as I should have clarified that "work by Boris
>> Brezillon" does not
>> mean "upsteam work". The code comes from Boris' github tree[1] and contains
>> work
>> that was not yet brought upstream, yet is required for NAND on sunxi to
>> work.
>>
>> It would probably be better to handle this as part of a general sync
>> with the Linux mtd code.
>>
>> Ideally yes, but. In upstream Linux MTD we have a few issues to address as
>> shown
>> by Boris' patch-set, most importantly:
>> - NAND chip timings (patch 1 to 3)
>> - Randomisation support (patch 6)
>> - Per-partition settings for ECC and randomisation (left out of my RFC,
>> highly desirable
>> but not strictly required for U-boot booting)
> Hello,
>
> as I understand it the ECC and randomisation settings for the
> bootloader part of the nand are suboptimal or unusable for ubifs so if
> u-boot SPL is to read the u-boot binary and later u-boot the kernel
> from an ubifs volume it has to support non-uniform settings.
> Alternatively the bootloader part can be extended to contain
> partitions for u-boot binary and kernel image written to raw partition
> without filesystem much like what Andriod usually does.
SPL does not read U-boot from UBI. The SPL driver is separate, much
smaller and
only reads in the same way BROM does. U-boot itself is not bound by the
limitations
imposed by BROM, which means proper randomisation and ECC settings can
be used
for "the UBI partition".
Yours,
Roy
>
> Thanks
>
> Michal
--
IMAGINE IT >> MAKE IT
Meet us online at Twitter <http://twitter.com/ultimaker>, Facebook
<http://facebook.com/ultimaker>, Google+ <http://google.com/+Ultimaker>
www.ultimaker.com
next prev parent reply other threads:[~2015-06-08 8:41 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 [this message]
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
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=55755538.9050606@ultimaker.com \
--to=r.spliet@ultimaker.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