linux-arm-kernel.lists.infradead.org archive mirror
 help / color / mirror / Atom feed
From: shawn.guo@linaro.org (Shawn Guo)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH 0/4] Support uncompress debug message for multiplatform build
Date: Wed, 12 Dec 2012 21:45:41 +0800	[thread overview]
Message-ID: <1355319945-3199-1-git-send-email-shawn.guo@linaro.org> (raw)

After moving i.MX to multiplatform build, I hear people complaining
that it's bad to lose the uncompress debug message, because not only
does it mean that we will never see message "Uncompressing Linux...
done, booting the kernel.", but also that the debug facility in
arch/arm/boot/compressed/head.S outputs nothing even DEBUG is turned on.

The series follows the approach handling DEBUG_LL to create a Kconfig
symbol CONFIG_UNCOMPRESS_INCLUDE for choosing the correct uncompress
header.  For traditional build, mach/uncompress.h will be included in
arch/arm/boot/compressed/misc.c, and nothing changes than before. For
multiplatform build with none DEBUG_LL port enabled, a set of empty
functions defined in debug/uncompress/dummy.h will be used.  And for
multiplatform build with a DEBUG_LL port, platform specific
uncompress header like debug/uncompress/imx.h will come to play.

Different from traditional build that uncompress output and DEBUG_LL
are two separate pieces, for multiplatform build uncompress output
becomes a subset of DEBUG_LL and relies on DEBUG_LL to select the uart
port, so it only works when DEBUG_LL is enabled.  I think it's an
acceptable compromise to have the uncompress debug facility back.

Shawn Guo (4):
  ARM: imx: use separated debug uart symbol for imx31 and imx35
  ARM: imx: support DEBUG_LL uart port selection for all i.MX SoCs
  ARM: imx: restore uncompress.h to a common place
  ARM: uncompress: support uncompress debug for multiplatform build

 arch/arm/Kconfig.debug                    |   44 ++++++++++++---
 arch/arm/boot/compressed/misc.c           |    8 +--
 arch/arm/include/debug/imx-uart.h         |   88 +++++++++++++++++++++++++++++
 arch/arm/include/debug/imx.S              |   29 +---------
 arch/arm/include/debug/uncompress/dummy.h |    3 +
 arch/arm/include/debug/uncompress/imx.h   |   63 +++++++++++++++++++++
 6 files changed, 191 insertions(+), 44 deletions(-)
 create mode 100644 arch/arm/include/debug/imx-uart.h
 create mode 100644 arch/arm/include/debug/uncompress/dummy.h
 create mode 100644 arch/arm/include/debug/uncompress/imx.h

-- 
1.7.9.5

             reply	other threads:[~2012-12-12 13:45 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-12-12 13:45 Shawn Guo [this message]
2012-12-12 13:45 ` [PATCH 1/4] ARM: imx: use separated debug uart symbol for imx31 and imx35 Shawn Guo
2012-12-12 13:45 ` [PATCH 2/4] ARM: imx: support DEBUG_LL uart port selection for all i.MX SoCs Shawn Guo
2013-03-12 14:35   ` Karl Beldan
2013-03-13  1:52     ` Shawn Guo
2012-12-12 13:45 ` [PATCH 3/4] ARM: imx: restore uncompress.h to a common place Shawn Guo
2013-01-12  6:25   ` Dirk Behme
2013-01-12 11:04     ` Shawn Guo
2013-01-12 11:58       ` Arnd Bergmann
2012-12-12 13:45 ` [PATCH 4/4] ARM: uncompress: support uncompress debug for multiplatform build Shawn Guo
2013-01-11 19:11   ` Olof Johansson
2013-01-12  3:29     ` Shawn Guo
2013-01-15 19:04       ` Olof Johansson
2013-01-16 14:04         ` Shawn Guo

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=1355319945-3199-1-git-send-email-shawn.guo@linaro.org \
    --to=shawn.guo@linaro.org \
    --cc=linux-arm-kernel@lists.infradead.org \
    /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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).