From: Tom Rini Date: Fri, 28 Jun 2013 09:03:22 -0400 Subject: [U-Boot] [PATCH] Fix bootm to work on powerpc again (compressed uImage) In-Reply-To: <1372406523-5726-1-git-send-email-sr@denx.de> References: <1372406523-5726-1-git-send-email-sr@denx.de> Message-ID: <20130628130322.GH28078@bill-the-cat> 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, Jun 28, 2013 at 10:02:03AM +0200, Stefan Roese wrote: > Patch 35fc84fa1 [Refactor the bootm command to reduce code duplication] > breaks booting Linux (compressed uImage with fdt) on powerpc. > > boot_jump_linux() mustn't be called before boot_prep_linux() and > boot_body_linux() have been called. So remove the superfluous call > to boot_jump_linux() in arch/powerpc/lib/bootm.c as its called later on > in this function. > > Signed-off-by: Stefan Roese > Cc: Simon Glass > Cc: Tom Rini > --- > Simon, I'm not 100% sure why this is triggered by your patch. > > As you see, my patch only fixes powerpc. ARM might have the same problem > as it has the same code duplication in bootm.c. Progress! On PowerPC, does bootm work like before? I still have to issue a separate 'bootm go', but that now boots, rather than hangs. -- Tom -------------- next part -------------- A non-text attachment was scrubbed... Name: not available Type: application/pgp-signature Size: 836 bytes Desc: Digital signature URL: