All of lore.kernel.org
 help / color / mirror / Atom feed
From: Wolfgang Denk <wd@denx.de>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH RFC] add CONFIG_BOOTCOUNT_LIMIT for MCF5445x
Date: Fri, 23 Apr 2010 21:25:35 +0200	[thread overview]
Message-ID: <20100423192535.23CF519F60@gemini.denx.de> (raw)
In-Reply-To: <1272026324-18566-1-git-send-email-w.wegner@astro-kom.de>

Dear Wolfgang Wegner,

In message <1272026324-18566-1-git-send-email-w.wegner@astro-kom.de> you wrote:
> This patch adds bootcount for Freescale MCF5445x. Two registers of
> eDMA transfer control descriptors (TCD[1]) are used because these
> are unused by linux kernel (freescale LTIB linux-2.6.25) and were
> tested to keep their contents across resets.
...
> +#ifdef CONFIG_BOOTCOUNT_LIMIT
> +/*
> + * We use transfer descriptor registers as a persistent storage
> + * across resets. This was tested on a MCF54455.
> + * Neither U-Boot nor the stock LTIB kernel seem to use
> + * TCD[1], so it should be safe at least until application
> + * start.

This is not sufficent, then. The bootcounter requires to be reset from
User Space, i. e. _after_ the application has been succesfully
started. If application start messes with the value, this fails to
work.

Best regards,

Wolfgang Denk

-- 
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
"The combination of a number of things to make existence worthwhile."
"Yes, the philosophy of 'none,' meaning 'all.'"
	-- Spock and Lincoln, "The Savage Curtain", stardate 5906.4

  reply	other threads:[~2010-04-23 19:25 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-04-23 12:38 [U-Boot] [PATCH RFC] add CONFIG_BOOTCOUNT_LIMIT for MCF5445x Wolfgang Wegner
2010-04-23 19:25 ` Wolfgang Denk [this message]
2010-04-23 19:43   ` Wolfgang Wegner
2010-04-23 20:03     ` Wolfgang Denk
2010-04-23 20:26       ` [U-Boot] [PATCH RFC v2] " Wolfgang Wegner
2010-04-23 20:47         ` Wolfgang Denk
2010-04-23 20:59           ` [U-Boot] [PATCH RFC v3] " Wolfgang Wegner
2010-04-23 22:27             ` Wolfgang Denk
2010-04-24  9:45               ` Wolfgang Wegner
2010-04-23 21:00           ` [U-Boot] [PATCH RFC v2] " Wolfgang Wegner

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=20100423192535.23CF519F60@gemini.denx.de \
    --to=wd@denx.de \
    --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.