From: Wolfgang Wegner <wolfgang@leila.ping.de>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH RFC] add CONFIG_BOOTCOUNT_LIMIT for MCF5445x
Date: Fri, 23 Apr 2010 21:43:41 +0200 [thread overview]
Message-ID: <20100423194341.GS20047@leila.ping.de> (raw)
In-Reply-To: <20100423192535.23CF519F60@gemini.denx.de>
Dear Wolfgang Denk,
On Fri, Apr 23, 2010 at 09:25:35PM +0200, Wolfgang Denk wrote:
> 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.
sorry, my wording was not correct.
What I meant was that no device driver in stock kernel uses these
registers, so the only possibility for them to get used is if the
init system or application loads some non-standard drivers.
Regards,
Wolfgang
next prev parent reply other threads:[~2010-04-23 19:43 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
2010-04-23 19:43 ` Wolfgang Wegner [this message]
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=20100423194341.GS20047@leila.ping.de \
--to=wolfgang@leila.ping.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox