public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
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
=====================================================================

  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