[U-Boot-Users] Release status - things to be done

[Stefan Roese]

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 at nuovations.com>
    Signed-off-by: Stefan Roese <sr at 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
=====================================================================