From: Andre Schwarz <andre.schwarz@matrix-vision.de>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH] powerpc: do not fixup NULL ptrs
Date: Thu, 04 Nov 2010 17:53:30 +0100 [thread overview]
Message-ID: <4CD2E50A.9090407@matrix-vision.de> (raw)
In-Reply-To: <OFFD615B82.B27415C2-ONC12577D1.00518553-C12577D1.0053539B@transmode.se>
Jocke,
>> ok - works for me, i.e. no quad-nop needed anymore.
>>
> Does both your boards work now?
>
MPC8343 @ 400MHz never had any issues - it's still working with your
patch applied.
MPC8377 works fine up to 533MHz ... 600MHz+ still hangs.
Looks like there are more sync missing.
>>> I am guessing it takes a while for the stw r4, LBLAWAR1(r3)
>>> to hit the HW so one must wait for it, not sure what is
>>> best though, sync or isync?
>>>
>>>
>> If it is a timing issue why should have the nops influenced this ?
>> I still wonder if this is the real problem and whether we might need
>> more (i)syncs elsewhere ...
>>
> You can try replacing the isync with 4 nops. That works
> for me. moving the 4 nops after the blr doesn't work.
>
> I think it worked earlier by chance but the removal of
> the flags changed timing, probably a cache line crossing
> at the wrong place.
>
"works by chance" is probably not what we want.
Anyway - good catch. Thanks again.
Cheers,
Andr?
MATRIX VISION GmbH, Talstrasse 16, DE-71570 Oppenweiler
Registergericht: Amtsgericht Stuttgart, HRB 271090
Geschaeftsfuehrer: Gerhard Thullner, Werner Armingeon, Uwe Furtner
next prev parent reply other threads:[~2010-11-04 16:53 UTC|newest]
Thread overview: 43+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-10-14 9:51 [U-Boot] [PATCH] powerpc: do not fixup NULL ptrs Joakim Tjernlund
2010-10-18 20:39 ` Wolfgang Denk
2010-10-24 20:08 ` Schwarz, Andre
2010-10-24 20:18 ` Wolfgang Denk
2010-10-24 20:33 ` Schwarz, Andre
2010-10-24 20:48 ` Joakim Tjernlund
2010-10-25 8:24 ` Schwarz, Andre
2010-10-25 13:50 ` Andre Schwarz
2010-10-25 14:32 ` Joakim Tjernlund
2010-10-25 14:46 ` Andre Schwarz
2010-10-25 15:36 ` Joakim Tjernlund
2010-10-25 17:16 ` Wolfgang Denk
2010-10-25 17:45 ` Andre Schwarz
2010-10-25 19:14 ` Wolfgang Denk
2010-10-26 10:28 ` Andre Schwarz
2010-10-26 12:34 ` Andre Schwarz
2010-10-26 12:49 ` Joakim Tjernlund
2010-11-04 9:10 ` Joakim Tjernlund
2010-11-04 9:57 ` Wolfgang Denk
2010-11-04 10:47 ` Joakim Tjernlund
2010-11-04 10:54 ` Joakim Tjernlund
2010-11-04 11:16 ` Wolfgang Denk
2010-11-04 12:19 ` Joakim Tjernlund
2010-11-04 12:46 ` Wolfgang Denk
2010-11-04 12:58 ` Joakim Tjernlund
2010-11-04 13:07 ` Wolfgang Denk
2010-11-04 13:11 ` Joakim Tjernlund
2010-11-04 10:58 ` Andy Pont
2010-11-04 11:04 ` Joakim Tjernlund
2010-11-04 12:14 ` Andre Schwarz
2010-11-04 12:18 ` Joakim Tjernlund
2010-11-04 12:28 ` Andre Schwarz
[not found] ` <OF350A930F.C39462D9-ONC <4CD2B5DF.2080400@matrix-vision.de>
2010-11-04 13:01 ` Joakim Tjernlund
2010-11-04 13:32 ` Andre Schwarz
2010-11-04 14:06 ` Joakim Tjernlund
2010-11-04 14:49 ` Andre Schwarz
2010-11-04 15:10 ` Joakim Tjernlund
2010-11-04 16:53 ` Andre Schwarz [this message]
2010-11-04 10:59 ` Reinhard Meyer
2010-11-04 10:00 ` Andre Schwarz
2010-11-04 9:50 ` Wolfgang Denk
2010-11-04 10:21 ` Joakim Tjernlund
2010-10-24 20:44 ` Joakim Tjernlund
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=4CD2E50A.9090407@matrix-vision.de \
--to=andre.schwarz@matrix-vision.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 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.