Package: u-boot
Version: 2011.12-1
Severity: important

I tried to install this version of u-boot on my dreamplug and it failed. I
reported this issue upstream[0] but thought it would be worth a bugreport here
too as a heads up and to prevent this version getting into testing.

2011.09-2 works for me fine.

Cheers,
Ian.

[0] It's held for moderation, even though I'm a subscriber, so I can't provide
a link right now, the text follows:

v2011.12 hangs on boot on my dreamplug with:

        U-Boot 2011.12-dirty (Jan 08 2012 - 15:16:44)
        Marvell-DreamPlug
        
        SoC:   Kirkwood 88F6281_A1
        DRAM:  512 MiB
        WARNING: Caches not enabled

I bisected it down to:
        commit ca4b55800ed74207c35271bf7335a092d4955416
        Author: Heiko Schocher <h...@denx.de>
        Date:   Wed Nov 9 20:06:23 2011 +0000
        
            arm, arm926ejs: always do cpu critical inits
            
            always do the cpu critical inits in cpu_init_crit,
            and only jump to lowlevel_init, if CONFIG_SKIP_LOWLEVEL_INIT
            is not defined.
            
            Signed-off-by: Heiko Schocher <h...@denx.de>
            Cc: Albert ARIBAUD <albert.u.b...@aribaud.net>
            Cc: Wolfgang Denk <h...@denx.de>
            Cc: Sandeep Paulraj <s-paul...@ti.com>
            Cc: Tom Rini <tom.r...@gmail.com>
            Cc: Christian Riesch <christian.rie...@omicron.at>

and in particular the line
        orr     r0, r0, #0x00000002     /* set bit 2 (A) Align */

Commenting out just that one line causes things to work. Is the error
setting this bit or is it that there is a bug somewhere else which
prevents us setting this bit?

A different approach to the original patch was suggested in
http://lists.denx.de/pipermail/u-boot/2011-December/112353.html and I
suppose that combined with
s/CONFIG_SKIP_LOWLEVEL_INIT/CONFIG_SKIP_CPU_INIT_CRIT/g in the relevant
config files (at least include/configs/dreamplug.h, but perhaps
everywhere except whichever platform caused this change?) would fix
things.

Thanks,
Ian.

-- System Information:
Debian Release: 6.0.3
  APT prefers stable-updates
  APT policy: (500, 'stable-updates'), (500, 'stable')
Architecture: armel (armv5tel)

Kernel: Linux 2.6.38.8 (PREEMPT)
Locale: LANG=en_GB.UTF-8, LC_CTYPE=en_GB.UTF-8 (charmap=UTF-8)
Shell: /bin/sh linked to /bin/dash

-- 
Ian Campbell


What one believes to be true either is true or becomes true.
                -- John Lilly

Attachment: signature.asc
Description: This is a digitally signed message part

Reply via email to