From: Wolfgang Denk <wd@denx.de>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH 0/1] Fix hang trying to protect flash sectors
Date: Thu, 20 May 2010 10:35:17 +0200 [thread overview]
Message-ID: <20100520083517.9F927CCF026@gemini.denx.de> (raw)
In-Reply-To: <loom.20100520T005209-871@post.gmane.org>
Dear Chris Packham,
In message <loom.20100520T005209-871@post.gmane.org> you wrote:
>
> While it would be possible to shuffle the memory map around there is one
> problem with the hardware design that I don't think can be overcome (I'd
> love to be proven wrong). The boot chip select is mapped to the _bottom_
> of the first flash chip. It was done this way so that we could expand the
> flash in the future as a rolling production change (we're now shipping
> units with 64MB fitted). i.e. we knew we could rely on a fixed base
> address so thats where we pointed the boot chip select.
I don't see why this should be relevant. Usually you can set up nearly
any memory map in software, independent of the CPU state at boot time.
Which exact processor family are we talking about?
> I think in hindsight we could have modified our flash detection code to
> start at the top and jump backwards looking for extra chips. Unfortunately
What do you mean by "our flash detection code"? Are you using any
private code for that? Why? U-Boot already has all the needed stuff,
just use it.
> we're not able to change the hardware design for this product but we can
> take this into account on future designs.
I'm not convinced that any hardware changes would be needed.
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
Don't panic.
next prev parent reply other threads:[~2010-05-20 8:35 UTC|newest]
Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-05-18 5:26 [U-Boot] [PATCH 0/1] Fix hang trying to protect flash sectors Mark Tomlinson
2010-05-18 5:26 ` [U-Boot] [PATCH 1/1] flash: Check info pointer in flash_protect() Mark Tomlinson
2010-05-19 22:22 ` Mike Frysinger
2010-05-20 8:38 ` Wolfgang Denk
2010-05-18 8:20 ` [U-Boot] [PATCH 0/1] Fix hang trying to protect flash sectors Stefan Roese
2010-05-18 20:10 ` mark tomlinson
2010-05-19 9:44 ` Stefan Roese
2010-05-19 21:09 ` mark tomlinson
2010-05-19 21:59 ` Wolfgang Denk
2010-05-19 23:08 ` Chris Packham
2010-05-20 8:35 ` Wolfgang Denk [this message]
2010-05-20 18:59 ` Chris Packham
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=20100520083517.9F927CCF026@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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox