Dear Gerd,

In message <1413894504-26255-1-git-send-email-kra...@redhat.com> you wrote:
> base[0] is saved, but only restored in case the ram test failed.

When posting a new version of a patch please always make sure to add a
change log!

I'm afraid your patch breaks at least some boards;  tested on TQM5200
board...

Before (v2014.10):

        U-Boot 2014.10 (Oct 21 2014 - 15:59:47)

        CPU:   MPC5200B v2.2, Core v1.4 at 396 MHz
               Bus 132 MHz, IPB 132 MHz, PCI 66 MHz
        Board: TQM5200S (TQ-Components GmbH)
               on a STK52xx carrier board
        I2C:   85 kHz, ready
        DRAM:  64 MiB
        Flash: 32 MiB
        In:    serial
        Out:   serial
        Err:   serial
        Net:   FEC
        IDE:   Bus 0: OK 
        ...

With your patch:

        U-Boot 2014.10-00001-g27030e7 (Oct 21 2014 - 15:45:12)

        CPU:   MPC5200B v2.2, Core v1.4 at 396 MHz
               Bus 132 MHz, IPB 132 MHz, PCI 66 MHz
        Board: TQM5200S (TQ-Components GmbH)
               on a STK52xx carrier board
        I2C:   85 kHz, ready
        DRAM:  1 GiB
[board hangs here]

Note also the incorrect RAM size.


Sorry, but this is not a correct fix.  NAK.

Best regards,

Wolfgang Denk

-- 
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: w...@denx.de
"It's when they say 2 + 2 = 5 that I begin to argue."    - Eric Pepke
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Reply via email to