From: Stefan Roese <sr@denx.de>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH v2] ppc4xx: Add Sequoia RAM-booting target
Date: Mon, 11 May 2009 15:31:11 +0200 [thread overview]
Message-ID: <200905111531.11474.sr@denx.de> (raw)
In-Reply-To: <20090511132010.86983832E416@gemini.denx.de>
On Monday 11 May 2009 15:20:10 Wolfgang Denk wrote:
> > > And what happens when the TLB is not correctly set? Will we get a
> > > useful error message then?
> >
> > No. SDRAM needs to work correctly so that this RAM-booting version can
> > function. And for this the MMU needs to be configured and the SDRAM
> > controller setup correctly. If this is not the case and SDRAM is not
> > working from the debugger then we have no chance to load this image
> > without problems (verify will most likely fail) or run it.
>
> Hm... if the image cannot run anyway without initialized SDRAM TLB,
> then why do you scan for it? My understanding is that the result
> wouldbe the same even if you omitted this test: without initialized
> SDRAM TLB the system would just hang / crash without error message.
> So this test is dead code.
>
> Or am I missing something?
I'm "scanning" for the SDRAM TLB, because it can be on any TLB location. In my
first implementation I assumed that the SDRAM TLB was TLB #0 (configured via
BDI2000). This worked just fine. But then I loaded this RAM-booting image via
U-Boot and started it here (via go ...). And it crashed since U-Boot uses a
different TLB #. So now I'm scanning/detecting the SDRAM TLB to be independent
from the originator of the TLB.
Best regards,
Stefan
=====================================================================
DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-0 Fax: +49-8142-66989-80 Email: office at denx.de
=====================================================================
next prev parent reply other threads:[~2009-05-11 13:31 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2009-05-11 9:47 [U-Boot] [PATCH v2] ppc4xx: Add Sequoia RAM-booting target Stefan Roese
2009-05-11 10:21 ` Wolfgang Denk
2009-05-11 11:39 ` Stefan Roese
2009-05-11 13:20 ` Wolfgang Denk
2009-05-11 13:31 ` Stefan Roese [this message]
2009-05-11 15:57 ` Wolfgang Denk
2009-05-11 16:46 ` Stefan Roese
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=200905111531.11474.sr@denx.de \
--to=sr@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.