From: Ladislav Michl <ladis@linux-mips.org>
To: u-boot@lists.denx.de
Subject: [U-Boot] [RFC] omap3: single binary supporting all flash types
Date: Fri, 17 Jun 2016 12:00:13 +0200 [thread overview]
Message-ID: <20160617100013.GA19253@localhost.localdomain> (raw)
Hi there!
currently it is not possible to build single working OMAP3 binary which
enables more than one of these:
CONFIG_NOR
CONFIG_NAND
CONFIG_CMD_ONENAND
first problem lies in gpmc_init which configures CS[0] to one flash type
determined at compile time. Another is that identify_nand_chip hang when
trying to indentify chip which is not present. The same issue is with
OneNAND mtd driver. And finally omap_gpmc:board_nand_init() searches
for NAND specific configuration in gpmc config1 register - but that value
is forced earlier from gpmc_init, making test completely pointless.
Most of these issues are fixed with patches sent as reply to this message,
but:
As we might want to use single binary also to boot from MMC or other
sources, we cannot use boot_device to get a glue about used flash type.
Also U-Boot should not count on being loaded by its own SPL. So, to make
things easier to begin with, do we all agree that omap_gpmc:board_nand_init
as well as its OneNAND counterpart should expect gpmc config1 register
properly set and fail if something unexpected is found? That basically
means if we should rely on CS0 config provided by first stage loader or
try to rediscover flash devices (this problem does not exist when flash
itself is boot source)
Thank you for your attention,
ladis
next reply other threads:[~2016-06-17 10:00 UTC|newest]
Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-06-17 10:00 Ladislav Michl [this message]
2016-06-17 10:02 ` [U-Boot] [PATCH] armv7: add reset timeout to identify_nand_chip Ladislav Michl
2016-06-17 10:04 ` [U-Boot] [PATCH] armv7: make gpmc_cfg const Ladislav Michl
2016-06-17 10:05 ` [U-Boot] [RFC] armv7: Introduce enable_gpmc_cs_config Ladislav Michl
2016-06-17 10:07 ` [U-Boot] (no subject) Ladislav Michl
2016-06-17 10:09 ` Ladislav Michl
2016-06-17 10:17 ` [U-Boot] [PATCH] mtd: OneNAND: allow board init function fail Ladislav Michl
2016-06-18 11:08 ` [U-Boot] [RFC] omap3: single binary supporting all flash types Ladislav Michl
2016-06-18 11:12 ` [U-Boot] [PATCH] armv7: introduce set_gpmc_cs0 Ladislav Michl
2016-06-18 11:13 ` [U-Boot] [PATCH] igep00x0: runtime flash detection Ladislav Michl
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=20160617100013.GA19253@localhost.localdomain \
--to=ladis@linux-mips.org \
--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