All of lore.kernel.org
 help / color / mirror / Atom feed
From: Sascha Hauer <s.hauer@pengutronix.de>
To: Orjan Friberg <of@flatfrog.com>
Cc: barebox@lists.infradead.org
Subject: Re: OMAP 3530 arch_shutdown "undefined instruction"
Date: Thu, 27 May 2010 11:31:32 +0200	[thread overview]
Message-ID: <20100527093132.GC23664@pengutronix.de> (raw)
In-Reply-To: <4BFD34A9.6030008@flatfrog.com>

Hi Orjan,

On Wed, May 26, 2010 at 04:48:09PM +0200, Orjan Friberg wrote:
> Hi,
>
> I'm trying to use barebox as both the 2nd and 3rd stage bootloader (boot  
> ROM being the 1st) on an OMAP 3530 (BeagleBoard).
>
> Every go <address> command in the 2nd stage, regardless of whether I  
> have uploaded anything to this address or not, is met with a
>
> go 0x80e80000
> ## Starting application at 0x80E80000 ...
> undefined instruction
> pc : [<40208b00>]    lr : [<40203a34>]
> sp : 87bff21c  ip : 00000028  fp : 87bffa4d
> r10: 00000000  r9 : 00000000  r8 : 87bff64d
> r7 : 87bffe44  r6 : 00000002  r5 : 4020b1e8  r4 : 80e80000
> r3 : 00000000  r2 : 00000001  r1 : 0000843f  r0 : 4020ae50
> Flags: nZCv  IRQs off  FIQs off  Mode SVC_32
> Resetting CPU ...
>
> which is in arch_shutdown:
>
> 40208af8 <arch_shutdown>:
> 40208af8:       e3a03000        mov     r3, #0  ; 0x0
> 40208afc:       ee073f17        mcr     15, 0, r3, cr7, cr7, {0}
> 40208b00:       e12fff1e        bx      lr
>
> This is from the 2010.05.0 snapshot, though it seems that code hasn't  
> changed in the latest trunk (arch/arm/cpu/cpu.c):
>
> 	/* flush I/D-cache */
> 	i = 0;
> 	asm ("mcr p15, 0, %0, c7, c7, 0": :"r" (i));
>
>

Seems this does not work on Cortex Processors. Can you try replacing
this with the following please:

	asm volatile (
		"bl __mmu_cache_flush;"
		"bl __mmu_cache_off;"
		:
		:
		: "r0", "r1", "r2", "r3", "r6", "r10", "r12", "cc", "memory"
	);

This should select the right cache flush functions.

Sascha

-- 
Pengutronix e.K.                           |                             |
Industrial Linux Solutions                 | http://www.pengutronix.de/  |
Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0    |
Amtsgericht Hildesheim, HRA 2686           | Fax:   +49-5121-206917-5555 |

_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox

  reply	other threads:[~2010-05-27  9:31 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-05-26 14:48 OMAP 3530 arch_shutdown "undefined instruction" Orjan Friberg
2010-05-27  9:31 ` Sascha Hauer [this message]
2010-05-27 11:03   ` Orjan Friberg
  -- strict thread matches above, loose matches on Subject: below --
2010-06-08  8:02 Luca Ceresoli
2010-06-10 12:22 ` Sascha Hauer

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=20100527093132.GC23664@pengutronix.de \
    --to=s.hauer@pengutronix.de \
    --cc=barebox@lists.infradead.org \
    --cc=of@flatfrog.com \
    /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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.