From: Nikita Kiryanov <nikita@compulab.co.il>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH 00/12] SPL mmc refactor and alternate boot device feature
Date: Fri, 23 Oct 2015 12:46:56 +0300 [thread overview]
Message-ID: <20151023094655.GB11008@skynet> (raw)
In-Reply-To: <CAP9ODKpKf+Ur90Jv4DBDUzG571yVkmUaM3mn1nJ22w36ONeODg@mail.gmail.com>
Hi Otavio,
On Thu, Oct 22, 2015 at 10:24:57AM -0200, Otavio Salvador wrote:
> On Thu, Oct 22, 2015 at 10:01 AM, Nikita Kiryanov <nikita@compulab.co.il> wrote:
> > This series has two parts: patches 1-7 perform refactors aimed at reducing the
> > ifdef complexity of SPL mmc code (and some nand as well). This refactor also
> > addresses a few design issues I noticed while working on the refactor.
> >
> > The rest of the series introduces a new SPL feature that allows board code to
> > define a list of boot devices that SPL will try before failing (instead of the
> > only one device it attempts now). This feature is useful for implementing
> > fallbacks, as well as reacting to bootROM sequences. For example:
> >
> > On CM-FX6, if boot from the alternate boot device (MMC) fails, the bootROM
> > proceeds to try boot from SPI flash. If the SPI flash boot is succesful, SPL
> > will still try to load U-Boot from MMC, instead of from the actual boot device
> > (SPI flash), and probably fail and hang. The alternate boot feature makes it
> > possible for SPL to follow the MMC boot attempt with boot from the SPI flash.
> > The CM-FX6 based miniature PC Utilite depends on this capability for its SPI
> > flash boot to work, since SPI flash boot is only attempted if MMC boot fails.
> >
> > This series was tested on CM-FX6 and compile tested for arm and powerpc.
>
> Thanks a lot for working on this; it does improves the SPL framework
> and extends its feature set. One thing I missed is the patches to
> rework the current boards removing the defines the serie drops and a
> proper documentation in the README files.
I'll make the additions once it's clear what part of the spl_mmc
refactor is accepted. There shouldn't be much changes for the board
files though, since the #defines themselves are not eliminated, just
that their checks are moved to a place where they will not make it
difficult to read the source code.
>
> --
> Otavio Salvador O.S. Systems
> http://www.ossystems.com.br http://code.ossystems.com.br
> Mobile: +55 (53) 9981-7854 Mobile: +1 (347) 903-9750
>
prev parent reply other threads:[~2015-10-23 9:46 UTC|newest]
Thread overview: 29+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-10-22 12:01 [U-Boot] [PATCH 00/12] SPL mmc refactor and alternate boot device feature Nikita Kiryanov
2015-10-22 12:01 ` [U-Boot] [PATCH 01/12] spl: nand: remove code duplication Nikita Kiryanov
2015-10-22 18:11 ` Scott Wood
2015-10-22 12:01 ` [U-Boot] [PATCH 02/12] spl: mmc: add break statements in spl_mmc_load_image() Nikita Kiryanov
2015-10-22 12:37 ` Hans de Goede
2015-10-22 13:08 ` Nikita Kiryanov
2015-10-22 13:23 ` Hans de Goede
2015-10-22 13:53 ` Nikita Kiryanov
2015-10-22 12:01 ` [U-Boot] [PATCH 03/12] spl: mmc: refactor device location code to its own function Nikita Kiryanov
2015-10-22 12:42 ` Hans de Goede
2015-10-22 14:15 ` Nikita Kiryanov
2015-10-22 12:01 ` [U-Boot] [PATCH 04/12] spl: mmc: remove #ifdef CONFIG_SPL_OS_BOOT check Nikita Kiryanov
2015-10-22 12:44 ` Hans de Goede
2015-10-22 12:01 ` [U-Boot] [PATCH 05/12] spl: mmc: get rid of #ifdef CONFIG_SYS_MMCSD_RAW_MODE_U_BOOT_PARTITION check Nikita Kiryanov
2015-10-22 12:47 ` Hans de Goede
2015-10-22 13:40 ` Nikita Kiryanov
2015-10-22 13:49 ` Tom Rini
2015-10-23 9:40 ` Nikita Kiryanov
2015-10-23 14:03 ` Hans de Goede
2015-10-23 14:11 ` Tom Rini
2015-10-22 12:01 ` [U-Boot] [PATCH 06/12] spl: mmc: move fs boot into its own function Nikita Kiryanov
2015-10-22 12:01 ` [U-Boot] [PATCH 07/12] spl: mmc: get rid of emmc boot code duplication Nikita Kiryanov
2015-10-22 12:01 ` [U-Boot] [PATCH 08/12] spl: change return values of spl_*_load_image() Nikita Kiryanov
2015-10-22 12:01 ` [U-Boot] [PATCH 09/12] common: spl: move image load to its own function Nikita Kiryanov
2015-10-22 12:01 ` [U-Boot] [PATCH 10/12] spl: add support for alternative boot device Nikita Kiryanov
2015-10-22 12:01 ` [U-Boot] [PATCH 11/12] spl: announce boot devices Nikita Kiryanov
2015-10-22 12:01 ` [U-Boot] [PATCH 12/12] arm: mx6: cm-fx6: define fallback boot devices for spl Nikita Kiryanov
2015-10-22 12:24 ` [U-Boot] [PATCH 00/12] SPL mmc refactor and alternate boot device feature Otavio Salvador
2015-10-23 9:46 ` Nikita Kiryanov [this message]
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=20151023094655.GB11008@skynet \
--to=nikita@compulab.co.il \
--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.