public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
From: Matthias Weisser <weisserm@arcor.de>
To: u-boot@lists.denx.de
Subject: [U-Boot] Cache function change breaks zmx25
Date: Sat, 10 Dec 2011 17:36:53 +0100	[thread overview]
Message-ID: <4EE38AA5.4030703@arcor.de> (raw)
In-Reply-To: <4EE22329.5090700@emcraft.com>

Am 09.12.2011 16:03, schrieb Ilya Yanok:
> Hi Matthias,
> 
> On 09.12.2011 18:24, Matthias Wei?er wrote:
>> breaks zmx25 booting with the following command:
>>
>> tftpboot 0x82000000 foo.img; dcache on; bootm 0x82000000
>>
>> It is stuck then in an endless loop after dcache is disabled before
>> jumping to the OS.
>>
>> ------------------------------------------------
>> WARNING: cache operations are not implemented!
>> WARNING: disabling D-Cache now, you can re-enable itlater with 'dcache
>> on' command
>> ------------------------------------------------
> 
> Argh.. That's really bad. May I ask you to debug this a little bit?
>
> What is exact cache function being called? Where from?
> How comes that dcache_status() returns true after dcache_disable()? Or
> does somebody call dcache_enable() in between?

The call trace in may case should be something like (not debuged, only
looked at the code):

cmd_elf.c    : do_bootelf()
cmd_elf.c    : do_bootelf_exec()
cache-cp15.c : dcache_disable()
cache-cp15.c : cache_disable(CR_C); -> Cache is not disabled in this
                                     function before
cache.c      : flush_dcache_all();   is called
cache.c      : dcache_noop();
cache-cp15.c : dcache_status() -> returns true
cache-cp15.c : dcache_disable()
cache-cp15.c : cache_disable(CR_C)
cache.c      : flush_dcache_all();
....

And we have the endless loop. I think this impacts not only zmx25 but
other boards enabling dcache.

If you need additional informations: I will have access to the board on
monday again.

-- 
Matthias

  reply	other threads:[~2011-12-10 16:36 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-12-09 14:24 [U-Boot] Cache function change breaks zmx25 Matthias Weißer
2011-12-09 15:03 ` Ilya Yanok
2011-12-10 16:36   ` Matthias Weisser [this message]
2011-12-11 23:09     ` [U-Boot] [PATCH] arm926ejs: remove noop flush_dcache_all function Ilya Yanok
2011-12-12  9:20       ` Matthias Weißer
2011-12-13 18:43         ` Tom Rini

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=4EE38AA5.4030703@arcor.de \
    --to=weisserm@arcor.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