linux-arm-kernel.lists.infradead.org archive mirror
 help / color / mirror / Atom feed
From: hjk@linutronix.de (Hans J. Koch)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH v2] mx35: Fix boot ROM hang in internal boot mode
Date: Fri, 24 Sep 2010 12:10:06 +0200	[thread overview]
Message-ID: <20100924101006.GB1819@silverbox.local> (raw)
In-Reply-To: <80aan7sgnj.fsf@merkur.tec.linutronix.de>

On Fri, Sep 24, 2010 at 10:13:36AM +0200, John Ogness wrote:
> On 2010-08-13, Uwe Kleine-K?nig <u.kleine-koenig@pengutronix.de> wrote:
> >> In my tests on an mx35pdk board, I found these three clocks being the
> >> minimum set of additional clocks that need to be turned on. That
> >> means, if you turn off any of the three, it won't boot anymore.
> >> 
> >> As a sidenote, it should be clear that this patch is a workaround for
> >> a serious chip bug of the MX35.
> >
> > If you're conviced this is a chip problem, did you consider to contact
> > FSL about it?
> 
> Freescale has responded to my inqueries about the boot ROM issue:
> 
> MX51EVK, MX508ARM2, MX25PDK boards also do this. Freescale has no plans
> to change the MX35 ROM code.

Sad thing. A hardware watchdog is there to protect you in situations where
things go wrong, so it must never assume that certain registers have certain
values.

If a hardware developer chose one of these CPUs for the following reasons

1) he needs internal bootmode to verify the bootloader
2) he needs the hardware watchdog to ensure reliable operation

he can simply not use these SoCs, because there are cases where the boot ROM
will hang until the next hardware reset or powercycle. If he uses such a CPU,
he needs additional hardware for an external watchdog.

Thanks,
Hans

  reply	other threads:[~2010-09-24 10:10 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-09-24  8:13 [PATCH v2] mx35: Fix boot ROM hang in internal boot mode John Ogness
2010-09-24 10:10 ` Hans J. Koch [this message]
  -- strict thread matches above, loose matches on Subject: below --
2010-08-11 23:37 Hans J. Koch
2010-08-12  5:57 ` Uwe Kleine-König
2010-08-12 11:29   ` Hans J. Koch
2010-08-12 12:15     ` Hans J. Koch
2010-08-13  8:17       ` Uwe Kleine-König
2010-08-13  8:58         ` Eric Bénard
2010-08-13 10:04         ` Hans J. Koch
2010-08-13 10:25           ` Uwe Kleine-König

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=20100924101006.GB1819@silverbox.local \
    --to=hjk@linutronix.de \
    --cc=linux-arm-kernel@lists.infradead.org \
    /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;
as well as URLs for NNTP newsgroup(s).