From: Stefan Roese <sr@denx.de>
To: u-boot@lists.denx.de
Subject: [U-Boot-Users] Release status - things to be done
Date: Mon, 7 Jul 2008 12:21:34 +0200 [thread overview]
Message-ID: <200807071221.35030.sr@denx.de> (raw)
In-Reply-To: <200807071150.26502.sr@denx.de>
On Monday 07 July 2008, Stefan Roese wrote:
> > > > 3766 06/12 "Brian S. Park" Re: [U-Boot-Users] U-Boot and Linux
> > > > on Walnut based board -> problem confirmed, to be taken care of by
> > > > PPC4xx custodian.
> > >
> > > I just tested with current TOT U-Boot (c956717a) and it works (again).
> > > Not sure what caused this problem though.
> >
> > Well, the bug is bisectable, IIRC.
> >
> > I would like to be sure that the bug ius really fixed now, and will
> > not show up in the next version again.
> >
> > Could you please check?
>
> OK, I'll try to find some time today to do this too.
I found the time. Unfortunately it was not easy to "bisect" since 4xx was not
buildable for quite some time. So I had to manually bisect. The commit that
fixes this issue is:
c821b5f120bedf73867513466412587c6912a8f8
ppc4xx: Enable Primordial Stack for 40x and Unify ECC Handling
This patch (Part 1 of 2):
* Rolls up a suite of changes to enable correct primordial stack and
global data handling when the data cache is used for such a purpose
for PPC40x-variants (i.e. CFG_INIT_DCACHE_CS).
* Related to the first, unifies DDR2 SDRAM and ECC initialization by
eliminating redundant ECC initialization implementations and moving
redundant SDRAM initialization out of board code into shared 4xx
code.
* Enables MCSR visibility on the 405EX(r).
* Enables the use of the data cache for initial RAM on
both AMCC's Kilauea and Makalu and removes a redundant
CFG_POST_MEMORY flag from each board's CONFIG_POST value.
- Removed, per Stefan Roese's request, defunct memory.c file for
Makalu and rolled sdram_init from it into makalu.c.
With respect to the 4xx DDR initialization and ECC unification, there
is certainly more work that can and should be done (file renaming,
etc.). However, that can be handled at a later date on a second or
third pass. As it stands, this patch moves things forward in an
incremental yet positive way for those platforms that utilize this
code and the features associated with it.
Signed-off-by: Grant Erickson <gerickson@nuovations.com>
Signed-off-by: Stefan Roese <sr@denx.de>
So it seems to have been a problem with the INIT RAM in D-cache which got
fixed by Grant's patch here. Thanks again to Grant for all his great work.
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:[~2008-07-07 10:21 UTC|newest]
Thread overview: 25+ messages / expand[flat|nested] mbox.gz Atom feed top
2008-07-05 23:05 [U-Boot-Users] Release status - things to be done Wolfgang Denk
2008-07-06 5:24 ` Shinya Kuribayashi
2008-07-06 7:17 ` Ben Warren
2008-07-06 7:54 ` Wolfgang Denk
2008-07-07 2:54 ` Nobuhiro Iwamatsu
2008-07-07 6:13 ` Ben Warren
2008-07-06 9:00 ` Magnus Lilja
2008-07-06 10:51 ` Jean-Christophe PLAGNIOL-VILLARD
2008-07-06 11:28 ` Nobuhiro Iwamatsu
2008-07-07 9:05 ` Stefan Roese
2008-07-07 9:37 ` Wolfgang Denk
2008-07-07 9:50 ` Stefan Roese
2008-07-07 10:21 ` Stefan Roese [this message]
2008-07-07 18:18 ` Scott Wood
2008-07-07 14:28 ` John Rigby
2008-07-07 15:03 ` Wolfgang Denk
2008-07-07 14:32 ` Scott Wood
2008-07-07 18:23 ` Jon Loeliger
2008-07-07 18:55 ` Kim Phillips
2008-07-07 20:30 ` Rune Torgersen
2008-07-07 20:41 ` Rune Torgersen
2008-07-09 23:34 ` Anatolij Gustschin
2008-07-10 11:12 ` Haavard Skinnemoen
-- strict thread matches above, loose matches on Subject: below --
2008-07-09 19:46 Liew Tsi Chung
2008-07-09 21:02 ` 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=200807071221.35030.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox