From: Stefan Roese <sr@denx.de>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH v2 15/21] Refactor the bootm command to reduce code duplication
Date: Thu, 27 Jun 2013 15:40:36 +0200 [thread overview]
Message-ID: <51CC40D4.8070703@denx.de> (raw)
In-Reply-To: <1370974493-21822-16-git-send-email-sjg@chromium.org>
Hi Simon,
On 06/11/2013 08:14 PM, Simon Glass wrote:
> At present the bootm code is mostly duplicated for the plain 'bootm'
> command and its sub-command variant. This makes the code harder to
> maintain and means that changes must be made to several places.
>
> Introduce do_bootm_states() which performs selected portions of the bootm
> work, so that both plain 'bootm' and 'bootm <sub_command>' can use the
> same code.
>
> Additional duplication exists in bootz, so tidy that up as well. This
> is not intended to change behaviour, apart from minor fixes where the
> previously-duplicated code missed some chunks of code.
>
> Signed-off-by: Simon Glass <sjg@chromium.org>
Simon, this patch breaks bootm (at least on powerpc), while booting an
compressed uImage (with DT). It just hangs while decompressing the
kernel image:
## Booting kernel from Legacy Image at 01000000 ...
Image Name: Linux-3.5.7
Image Type: PowerPC Linux Kernel Image (gzip compressed)
Data Size: 2203312 Bytes = 2.1 MiB
Load Address: 00000000
Entry Point: 00000000
## Flattened Device Tree blob at 01800000
Booting using the fdt blob at 0x1800000
Uncompressing Kernel Image ...
I bisected mainline and this patch is the one that breaks booting:
[stefan at ubuntu-2012 u-boot ((983c72f...)|BISECTING)]$ git bisect good
35fc84fa1ff51e15ecd3e464dac87eb105ffed30 is the first bad commit
commit 35fc84fa1ff51e15ecd3e464dac87eb105ffed30
Author: Simon Glass <sjg@chromium.org>
Date: Tue Jun 11 11:14:47 2013 -0700
Refactor the bootm command to reduce code duplication
I already looked what might be wrong but I couldn't find any problem
upon a quick glance. You know this code may better. Perhaps you could
take look at it and give it a try as well.
Thanks,
Stefan
next prev parent reply other threads:[~2013-06-27 13:40 UTC|newest]
Thread overview: 41+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-06-11 18:14 [U-Boot] [PATCH v2 0/21] Add tracing functionality to U-Boot Simon Glass
2013-06-11 18:14 ` [U-Boot] [PATCH v2 01/21] pci: Convert extern inline functions to static inline Simon Glass
2013-06-11 18:14 ` [U-Boot] [PATCH v2 02/21] x86: Correct missing local variable in bootm Simon Glass
2013-06-11 18:14 ` [U-Boot] [PATCH v2 03/21] Fix missing return in do_mem_loop() Simon Glass
2013-06-11 18:14 ` [U-Boot] [PATCH v2 04/21] Show stdout on error in fit-test Simon Glass
2013-06-11 18:14 ` [U-Boot] [PATCH v2 05/21] bootstage: Correct printf types Simon Glass
2013-06-11 18:14 ` [U-Boot] [PATCH v2 06/21] Add function to print a number with grouped digits Simon Glass
2013-06-11 18:14 ` [U-Boot] [PATCH v2 07/21] Add trace library Simon Glass
2013-06-11 18:14 ` [U-Boot] [PATCH v2 08/21] Add a trace command Simon Glass
2013-06-11 18:14 ` [U-Boot] [PATCH v2 09/21] Support tracing in config.mk when enabled Simon Glass
2013-06-11 18:14 ` [U-Boot] [PATCH v2 10/21] Add trace support to generic board Simon Glass
2013-06-11 18:14 ` [U-Boot] [PATCH v2 11/21] Add proftool to decode profile data Simon Glass
2013-06-11 18:14 ` [U-Boot] [PATCH v2 12/21] sandbox: Support trace feature Simon Glass
2013-06-11 18:14 ` [U-Boot] [PATCH v2 13/21] Add a simple test for sandbox trace Simon Glass
2013-06-11 18:14 ` [U-Boot] [PATCH v2 14/21] Clarify bootm OS arguments Simon Glass
2013-06-11 18:14 ` [U-Boot] [PATCH v2 15/21] Refactor the bootm command to reduce code duplication Simon Glass
2013-06-27 13:40 ` Stefan Roese [this message]
2013-06-27 14:40 ` Tom Rini
2013-06-27 15:39 ` Simon Glass
2013-06-27 19:51 ` Simon Glass
2013-06-27 20:07 ` Tom Rini
2013-06-28 2:14 ` Simon Glass
2013-06-28 4:12 ` Stefan Roese
2013-06-28 5:44 ` Simon Glass
2013-06-28 6:58 ` Simon Glass
2013-06-28 7:04 ` Stefan Roese
2013-06-28 7:12 ` Simon Glass
2013-06-28 7:57 ` Simon Glass
2013-06-11 18:14 ` [U-Boot] [PATCH v2 16/21] Add a 'fake' go command to the bootm command Simon Glass
2013-06-11 18:14 ` [U-Boot] [PATCH v2 17/21] arm: Implement the 'fake' go command Simon Glass
2013-06-11 19:59 ` Albert ARIBAUD
2013-06-11 20:02 ` Simon Glass
2013-06-11 20:21 ` Albert ARIBAUD
2013-06-11 21:17 ` Simon Glass
2013-06-11 21:36 ` Albert ARIBAUD
2013-06-20 4:16 ` Simon Glass
2013-06-11 18:14 ` [U-Boot] [PATCH v2 18/21] exynos: Avoid function instrumentation for microsecond timer Simon Glass
2013-06-11 18:14 ` [U-Boot] [PATCH v2 19/21] exynos: config: Add tracing options Simon Glass
2013-06-11 18:14 ` [U-Boot] [PATCH v2 20/21] x86: Support tracing function Simon Glass
2013-06-11 18:14 ` [U-Boot] [PATCH v2 21/21] x86: config: Add tracing options Simon Glass
2013-06-26 20:24 ` [U-Boot] [PATCH v2 0/21] Add tracing functionality to U-Boot 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=51CC40D4.8070703@denx.de \
--to=sr@denx.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.