From: linux@arm.linux.org.uk (Russell King - ARM Linux)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH 1/3] ARM: debug: use kconfig choice for selecting DEBUG_LL UART
Date: Tue, 6 Sep 2011 11:52:06 +0100 [thread overview]
Message-ID: <20110906105206.GL6619@n2100.arm.linux.org.uk> (raw)
In-Reply-To: <20110906102703.GA1895@atomide.com>
On Tue, Sep 06, 2011 at 03:27:03AM -0700, Tony Lindgren wrote:
> * Russell King - ARM Linux <linux@arm.linux.org.uk> [110906 02:04]:
> > On Tue, Sep 06, 2011 at 12:28:17PM +0300, Tony Lindgren wrote:
> > > Probably the best long term solution is to set up the debug uart based
> > > on DT data and initialize it in the uncompress code. Anybody debugging
> > > lower level stuff can certainly patch in the ucompress debug_ll code.
> >
> > How do you do that before the MMU has been setup? Are you going to
> > write a DT data parser in pure assembly to extract this information?
>
> Don't we already have that with ATAGs to DT support in the DT append
> patches? At least it calls fdt_setprop so calling fdt_getprop should
> also work. The patch I'm talking about is:
>
> http://permalink.gmane.org/gmane.linux.drivers.devicetree/5938
Let me reiterate. How are you going to do that before the MMU has been
setup.
I'll give you a hint: in the kernel (not the boot loader) you can't
call *ANY* C code until you have the MMU enabled. One of the points
of the *LOW LEVEL* debug code is so you can debug the low level
assembly in the head*.S files before the MMU has been enabled.
That means you can't call the C function "fdt_getprop" to get the
parameters.
So, should we have a chunk of assembly code to enable the MMU so that
we can run fdt_getprop() to get the parameters for the LL debug code,
turn the MMU off, and then run through the head.S code to enable the
MMU for the kernel proper? If you think that's silly, you're starting
to get the picture I've had for years about all this bastardization of
the LL debug code to solve problems beyond what it was originally
intended for - which, again, was to debug the head.S code.
Personally, I think this whole thing is getting impractical for the
use which people are trying to stretch it too - it's trying to be used
to solve the "how do we get console output before the console is up"
problem as well as the "how do we debug the low level assembly code".
With the advent of multi-platform kernels, the two requirements have
been well proving to be mutually exclusive.
next prev parent reply other threads:[~2011-09-06 10:52 UTC|newest]
Thread overview: 73+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-08-16 21:41 [PATCH 1/3] ARM: debug: use kconfig choice for selecting DEBUG_LL UART Will Deacon
2011-08-16 21:41 ` [PATCH 2/3] ARM: plat-samsung: use Kconfig choice for debug UART selection Will Deacon
2011-10-10 11:56 ` Thomas Abraham
2011-10-10 12:23 ` Will Deacon
2011-10-10 12:35 ` Thomas Abraham
2011-10-10 13:34 ` Will Deacon
2011-10-10 13:38 ` Thomas Abraham
2011-08-16 21:41 ` [PATCH 3/3] ARM: realview: " Will Deacon
2011-08-18 4:06 ` [PATCH 1/3] ARM: debug: use kconfig choice for selecting DEBUG_LL UART Nicolas Pitre
2011-08-18 9:33 ` Will Deacon
2011-08-18 16:11 ` Shawn Guo
2011-08-18 16:07 ` Will Deacon
2011-08-19 4:59 ` Shawn Guo
2011-08-19 11:08 ` Will Deacon
2011-08-19 11:37 ` Shawn Guo
2011-08-19 12:32 ` Will Deacon
2011-08-19 15:49 ` Nicolas Pitre
2011-08-21 9:14 ` Russell King - ARM Linux
2011-08-21 17:35 ` Nicolas Pitre
2011-08-21 18:26 ` Russell King - ARM Linux
2011-08-21 19:02 ` Nicolas Pitre
2011-08-21 19:18 ` Russell King - ARM Linux
2011-08-21 19:22 ` Russell King - ARM Linux
2011-08-21 20:07 ` Nicolas Pitre
2011-08-21 20:54 ` Russell King - ARM Linux
2011-08-21 21:00 ` Nicolas Pitre
2011-08-21 21:29 ` Russell King - ARM Linux
2011-08-21 22:00 ` Nicolas Pitre
2011-08-21 19:53 ` Nicolas Pitre
2011-09-06 9:28 ` Tony Lindgren
2011-09-06 9:37 ` Russell King - ARM Linux
2011-09-06 10:27 ` Tony Lindgren
2011-09-06 10:52 ` Russell King - ARM Linux [this message]
2011-09-06 11:01 ` Tony Lindgren
2011-09-06 11:07 ` Russell King - ARM Linux
2011-09-06 19:45 ` Uwe Kleine-König
2011-08-19 14:54 ` Nicolas Pitre
2011-08-19 4:56 ` [PATCH] arm/imx: use Kconfig choice for low-level debug UART selection Shawn Guo
2011-08-19 6:35 ` Sascha Hauer
2011-08-19 7:00 ` Shawn Guo
2011-08-19 11:09 ` Will Deacon
2011-08-19 11:39 ` Sascha Hauer
2011-08-19 12:35 ` Will Deacon
2011-08-19 17:15 ` Sascha Hauer
2011-08-21 9:18 ` Russell King - ARM Linux
2011-08-21 11:25 ` Will Deacon
2011-08-21 17:59 ` Nicolas Pitre
2011-08-21 18:17 ` Russell King - ARM Linux
2011-08-21 18:28 ` Nicolas Pitre
2011-08-21 18:33 ` Russell King - ARM Linux
2011-11-22 8:58 ` Uwe Kleine-König
2011-11-22 11:50 ` Will Deacon
2011-11-22 13:02 ` Uwe Kleine-König
2011-11-22 13:20 ` Will Deacon
2011-11-22 13:30 ` Uwe Kleine-König
2011-11-22 15:48 ` Russell King - ARM Linux
2011-11-22 16:38 ` Uwe Kleine-König
2011-11-22 16:47 ` Mark Brown
2011-11-22 17:13 ` Uwe Kleine-König
2011-11-22 20:24 ` Russell King - ARM Linux
2011-11-22 21:19 ` Arnd Bergmann
2011-11-22 23:00 ` Mark Brown
2011-11-23 10:57 ` Will Deacon
2011-09-15 17:34 ` [PATCH 1/3] ARM: debug: use kconfig choice for selecting DEBUG_LL UART Stephen Boyd
2011-09-16 11:41 ` Will Deacon
2011-09-19 18:01 ` [PATCH 1/2] ARM: debug: Add CLSP711X_UART1 config choice Stephen Boyd
2011-09-19 18:01 ` [PATCH 2/2] ARM: debug: Move DEBUG_ICEDCC into the DEBUG_LL choice Stephen Boyd
2011-09-19 21:25 ` [PATCH 1/2] ARM: debug: Add CLSP711X_UART1 config choice Will Deacon
2011-09-19 22:12 ` Stephen Boyd
2011-09-19 22:41 ` Russell King - ARM Linux
2011-09-19 22:55 ` Stephen Boyd
2011-11-22 20:42 ` Russell King - ARM Linux
2011-09-19 23:14 ` Will Deacon
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=20110906105206.GL6619@n2100.arm.linux.org.uk \
--to=linux@arm.linux.org.uk \
--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).