public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
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
> 

      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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox