From: David Hawkins <dwh@ovro.caltech.edu>
To: u-boot@lists.denx.de
Subject: [U-Boot-Users] CFG_MONITOR_BASE < CFG_FLASH_BASE
Date: Thu, 17 May 2007 08:55:31 -0700 [thread overview]
Message-ID: <464C7AF3.30500@ovro.caltech.edu> (raw)
In-Reply-To: <464C6CFC.7020602@freescale.com>
Hi Timur,
>> The MPC8349E can be booted such that the core is held in
>> reset, and the processor registers can be configured over
>> PCI by another host computer. Therefore it is conceivable
>> that the host can program the SDRAM controller on the
>> MPC8349E and take the core out of reset. If the core
>> is configured to boot from an address mapped to SDRAM,
>> then U-Boot could have been copied to SDRAM by the
>> host. Once U-Boot boots, it could then use FTP etc
>> to boot the kernel blah blah ...
>
> Ok fine, but you're talking about an 8349 on a different board. I have
> a *board* header file for the MPC8349E-mITX, which comes with 16MB of
> flash and works just fine. I've done the hard work of getting U-Boot
> running on that board with flash. So the question is, am I going to
> upset someone if I remove support for booting from RAM on that board?
In that instance, since its a board you are personally
supporting, I think its your choice, so go ahead and remove it.
If someone decides that they want to add RAM boot support,
then they can supply the patch. If someone asks how
to do it, then point them to the MPC8349E-MDS-PB source
(where I assume it exists). If they get it to work on
the -mITX, then they can submit a patch along with
a clear explanation of why they thought it was
necessary to support it.
Of course the fact that this code sounds like its being copied
from BSP to BSP, when its quite possibly general code, is not
that desirable. So deleting it from this BSP will hopefully
discourage that practice.
Cheers,
Dave
next prev parent reply other threads:[~2007-05-17 15:55 UTC|newest]
Thread overview: 17+ messages / expand[flat|nested] mbox.gz Atom feed top
2007-05-16 20:47 [U-Boot-Users] CFG_MONITOR_BASE < CFG_FLASH_BASE Timur Tabi
2007-05-16 22:04 ` Wolfgang Denk
2007-05-16 22:11 ` Timur Tabi
2007-05-16 23:04 ` Leonid
2007-05-16 23:11 ` Timur Tabi
2007-05-17 2:12 ` Leonid
2007-05-17 15:35 ` Jon Loeliger
2007-05-17 0:11 ` Wolfgang Denk
2007-05-17 1:47 ` Leonid
2007-05-17 10:00 ` Wolfgang Denk
2007-05-16 23:34 ` David Hawkins
2007-05-17 14:55 ` Timur Tabi
2007-05-17 15:55 ` David Hawkins [this message]
2007-05-17 20:29 ` Wolfgang Denk
2007-08-16 11:31 ` Florian Boelstler
2007-08-16 11:51 ` Wolfgang Denk
2007-05-17 0:08 ` Wolfgang Denk
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=464C7AF3.30500@ovro.caltech.edu \
--to=dwh@ovro.caltech.edu \
--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