public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
From: Holger Brunck <holger.brunck@keymile.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH 1/5] arm/km: adapt bootcounter evaluation
Date: Mon, 31 Oct 2011 09:13:42 +0100	[thread overview]
Message-ID: <4EAE58B6.9040805@keymile.com> (raw)
In-Reply-To: <F766E4F80769BD478052FB6533FA745D1A14DD8804@SC-VEXCH4.marvell.com>

On 10/31/2011 08:29 AM, Prafulla Wadaskar wrote:
> 
> 
>> -----Original Message-----
>> From: Holger Brunck [mailto:holger.brunck at keymile.com]
>> Sent: Friday, October 28, 2011 6:43 PM
>> To: Prafulla Wadaskar
>> Cc: u-boot at lists.denx.de; Valentin Longchamp
>> Subject: Re: [PATCH 1/5] arm/km: adapt bootcounter evaluation
>>
>> Hi Prafulla,
>>
>> On 09/14/2011 10:41 AM, Holger Brunck wrote:
>>> The bootcounter (stored in the RAM) is not enough protected
>> with the 4 Bytes
>>> BOOTCOUNT_MAGIC against bit errors due to short power loss or
>> holding a system
>>> in RESET. It has been seen, that the bootcounter value has
>> been changed due to
>>> a bit flip on a system holding in RESET, but the
>> BOOTCOUNT_MAGIC was still valid.
>>>
>>> A bit pattern with 4000 bytes (after BOOTCOUNT_MAGIC) has
>> been implemented,
>>> which should be enough to detect a bit error.
>>>
>>> Signed-off-by: Holger Brunck <holger.brunck@keymile.com>
>>> Signed-off-by: Valentin Longchamp
>> <valentin.longchamp@keymile.com>
>>> cc: Prafulla Wadaskar <prafulla@marvell.com>
>>> ---
>>>  board/keymile/km_arm/km_arm.c |   32
>> +++++++++++++++++++++++++++-----
>>>  1 files changed, 27 insertions(+), 5 deletions(-)
>>>
>>
>> ...ping...
>>
>> Or should this serie not go through your tree? If so then
>> please let me know.
> 
> Generally not, this was done earlier by Wolfgang or Albert.
> So I was assuming someone will pull.
> 

Hm, in the past the patches for keymile ARM boards go through your tree Prafulla.

Anyway Wolfgang or Albert, who should pick up the patches for our Keymile
Kirkwood boards?

This two series have only board specific patches:
http://lists.denx.de/pipermail/u-boot/2011-September/101328.html
http://lists.denx.de/pipermail/u-boot/2011-September/102534.html

Best regards
Holger

  reply	other threads:[~2011-10-31  8:13 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-09-14  8:41 [U-Boot] [PATCH 1/5] arm/km: adapt bootcounter evaluation Holger Brunck
2011-09-14  8:41 ` [U-Boot] [PATCH 2/5] arm/km: portl2 environment address update to P1B Holger Brunck
2011-09-14  8:41 ` [U-Boot] [PATCH 3/5] arm/km: add boardid and hwkey to kernel command line Holger Brunck
2011-09-14  8:41 ` [U-Boot] [PATCH 4/5] arm/km: trigger reconfiguration for the Xilinx FPGA Holger Brunck
2011-09-14  8:41 ` [U-Boot] [PATCH 5/5] arm/km: enable jffs2 cmds Holger Brunck
2011-10-28 13:12 ` [U-Boot] [PATCH 1/5] arm/km: adapt bootcounter evaluation Holger Brunck
2011-10-31  7:29   ` Prafulla Wadaskar
2011-10-31  8:13     ` Holger Brunck [this message]
2011-10-31  8:23       ` Prafulla Wadaskar
2011-10-31  8:48       ` Prafulla Wadaskar
2011-10-31 13:14         ` Holger Brunck

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=4EAE58B6.9040805@keymile.com \
    --to=holger.brunck@keymile.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox