From: Tom Rini <trini@ti.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH 0/2] Add bootcount support to SPL
Date: Wed, 2 Oct 2013 11:07:41 -0400 [thread overview]
Message-ID: <20131002150741.GG15917@bill-the-cat> (raw)
In-Reply-To: <CAP9ODKp=W8ZaTFp0WCmUwRJX+MqTkPL3A9J9bapAhwPyyBRX0g@mail.gmail.com>
On Fri, Sep 27, 2013 at 10:20:52PM -0300, Otavio Salvador wrote:
> On Fri, Sep 27, 2013 at 4:26 PM, Tom Rini <trini@ti.com> wrote:
> > The following patches depend on the series I posted earlier that adds
> > environment support to SPL as well as the series that converts the
> > davinci bootcount driver to support am335x as well. The most obvious
> > issue to me as that as Scott pointed out with the last series, this
> > would be clearer Kconfig where we wouldn't need to introduce
> > CONFIG_SPL_BOOTCOUNT_SUPPORT and just re-use CONFIG_BOOTCOUNT_SUPPORT.
> > Other than that, I think things look good. It may be possible to
> > re-jigger bootcount drivers to allow counting for both SPL and full
> > U-Boot to happen, if that's strongly desired but the way I see this
> > being used is to fall-back to a failsafe full U-Boot to do what's needed
> > there.
>
> I think we need to count both; the SPL part solves the U-Boot failsafe
> but the full U-Boot is needed to failsafe the OS.
It's _possible_ we could split the counter up further, 16bit magic, 8
bit SPL count, 8 bit U-Boot count. But I'm not sure if it's useful.
Looking at Stefan's patch, we count in SPL, and use altbootcmd in full
U-Boot to know we have failure rather than just booting fully via
U-Boot.
--
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 836 bytes
Desc: Digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20131002/a321b41c/attachment.pgp>
prev parent reply other threads:[~2013-10-02 15:07 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-09-27 19:26 [U-Boot] [PATCH 0/2] Add bootcount support to SPL Tom Rini
2013-09-27 19:26 ` [U-Boot] [PATCH 1/2] SPL: Add CONFIG_SPL_BOOTCOUNT_SUPPORT Tom Rini
2013-09-30 6:37 ` Stefan Roese
2013-10-02 13:14 ` Tom Rini
2013-09-27 19:26 ` [U-Boot] [PATCH 2/2] am335x: Switch to CONFIG_SPL_BOOTCOUNT_SUPPORT Tom Rini
2013-09-28 1:20 ` [U-Boot] [PATCH 0/2] Add bootcount support to SPL Otavio Salvador
2013-10-02 15:07 ` Tom Rini [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=20131002150741.GG15917@bill-the-cat \
--to=trini@ti.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 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.