From mboxrd@z Thu Jan 1 00:00:00 1970 From: =?ISO-8859-1?Q?Andr=E9?= Schwarz Date: Fri, 22 Oct 2010 21:25:34 +0200 Subject: [U-Boot] [MPC83xx] broken patch "do not fixup NULL ptrs" ? In-Reply-To: References: <1287757849.2284.27.camel@swa-m460> Message-ID: <1287775534.2169.3.camel@swa-e6500> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de On Fri, 2010-10-22 at 16:56 +0200, Joakim Tjernlund wrote: > Andr? Schwarz wrote on 2010/10/22 > 16:30:49: > > > > Joakim, > > > > > > your commit d1e0b10accdbac2e0a8b2cbf7c589645442f87c5 renders my MPC8377 > > based board unbootable. > > > > > > Two weeks ago I got this after power on : > > > > > > U-Boot 2010.09-00486-gfda6278-dirty (Oct 22 2010 - 16:19:20) MPC83XX > > > > Reset Status: > > > > CPU: e300c4, MPC8377, Rev: 2.1 at 533.333 MHz, CSB: 266.667 MHz > > Board: Matrix Vision MergerBox > > .... > > > > > > > > After rebasing my work to current master completely silences the serial > > line and the CPU doesn't come up. Bisection leads to your patch. > > > > reverting it makes the board work again. > > > > Any ideas ? > > Yes, the problem is else where. This code runs after relocation to RAM. > I had a similar problem and the reason was the patch: > "powerpc: Remove warm reset entry point" hmm, ok. But what's the root cause ? > > Try sticking 4 nop's right after _start: > in start.S > ... and why should this work ? Has this something to do with trial and error ? Can you shed some light on this ? Regards, Andr? MATRIX VISION GmbH, Talstrasse 16, DE-71570 Oppenweiler Registergericht: Amtsgericht Stuttgart, HRB 271090 Geschaeftsfuehrer: Gerhard Thullner, Werner Armingeon, Uwe Furtner