From: Matthias Kaehlcke <matthias@kaehlcke.net>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH 0/3] Fixes for edb9315
Date: Sat, 6 Feb 2010 23:15:27 +0100 [thread overview]
Message-ID: <20100206221527.GA25569@darwin> (raw)
In-Reply-To: <cover.1265484838.git.rubini@unipv.it>
Hi Alessandro,
El Sat, Feb 06, 2010 at 08:53:30PM +0100 Alessandro Rubini ha dit:
> I'm porting current u-boot to a board similar to EP9315A, so I'm using
> arm/master as a basis, as it includes the patches by Matthias
> Kaehlcke. I'm currently running from RAM (SKIP_LOWLEVEL_INIT), after
> setting up sdram and pll elsewhere (older vendor u-boot code, still to
> be ported).
>
> "fix syscon_regs definition" is needed to access any software-locked
> syscon register from C code (e.g., in reset_cpu() and devicecfg used
> by me in patch 3).
you are right, i missed the gap between ChipID and SysCfg
> "change calculation un early_udelay.h" is needed at least for eldk-4.2
> (gcc-4.2.2), as without this patch it will use the stack before setting SP.
> Actually, we could use the ether buffer as a stack, if needed, but it's not
> really needed here.
i feared this could happen with different compiler versions, your fix
certainly reduces the risk
> "enable the uart in devicecfg register" prevents u-boot from freezing
> at least with SKIP_LOWLEVEL_INIT set, but I'm pretty sure lowlevel_setup
> assembly code doesn't enable the uart, either.
my ep9301 and ep9307 based boards for some reason boot with the uart
enabled, though according to the datasheet it is disabled until enabled
explicitly. therefore your fix is necessary for all cpus behaving as
expected.
thanks a lot for your patches!
--
Matthias Kaehlcke
Embedded Linux Developer
Barcelona
The assumption that what currently exists must necessarily
exist is the acid that corrodes all visionary thinking
.''`.
using free software / Debian GNU/Linux | http://debian.org : :' :
`. `'`
gpg --keyserver pgp.mit.edu --recv-keys 47D8E5D4 `-
prev parent reply other threads:[~2010-02-06 22:15 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-02-06 19:53 [U-Boot] [PATCH 0/3] Fixes for edb9315 Alessandro Rubini
2010-02-06 19:53 ` [U-Boot] [PATCH 1/3] EP93xx: fix syscon_regs definition Alessandro Rubini
2010-02-06 22:17 ` Matthias Kaehlcke
2010-02-06 19:53 ` [U-Boot] [PATCH 2/3] edb93xx: change calculation un early_udelay.h Alessandro Rubini
2010-02-06 22:17 ` Matthias Kaehlcke
2010-02-06 19:54 ` [U-Boot] [PATCH 3/3] edb93xx: enable the uart in devicecfg register Alessandro Rubini
2010-02-06 22:24 ` Matthias Kaehlcke
2010-02-07 23:08 ` Tom
2010-02-06 22:15 ` Matthias Kaehlcke [this message]
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=20100206221527.GA25569@darwin \
--to=matthias@kaehlcke.net \
--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