From: Russell King - ARM Linux <linux@arm.linux.org.uk>
To: Tony Lindgren <tony@atomide.com>, arm@kernel.org
Cc: linux-omap@vger.kernel.org, linux-arm-kernel@lists.infradead.org
Subject: Re: [PATCH 1/6] ARM: OMAP2+: Remove board-4430sdp.c
Date: Sat, 6 Jul 2013 14:36:27 +0100 [thread overview]
Message-ID: <20130706133627.GV21614@n2100.arm.linux.org.uk> (raw)
In-Reply-To: <20130706131057.GU21614@n2100.arm.linux.org.uk>
On Sat, Jul 06, 2013 at 02:10:57PM +0100, Russell King - ARM Linux wrote:
> Well, this doesn't work. I've tried several things, and I think I'm
> generating a correct image, but it doesn't boot.
>
> This could be that the current Linus tip plus my stuff plus arm-soc
> is broken, but as it boots on the 3430LDP I suspect that isn't the
> case.
>
> I've tried with omap4-sdp.dtb and the other omap4-sdp DTB file with no
> improvement. No idea how to even begin to debug this, as there's no
> way for the decompressor to produce output, and I've no idea which
> OMAP uart to use for debug output (because that information has been
> removed from the kernel source.)
>
> And... WTF is the OMAP debug uart selection in its own separate menu
> from the main "choice" statement? This is totally unnecessary complexity
> and is just making things more complicated for the hell of it. Look at
> what every other platform does - they put their stuff in the main choice
> statement even if they have multiple UARTs. The hint there is that these
> depend on the appropriate support being selected, so in the case of a
> kernel only targetting OMAP, as things stand you'll end up with a menu
> which lists the OMAP2PLUS, none, icedcc, and semihosting entries followed
> by another menu to select which OMAP port you want. That's utterly
> rediculous and broken. And just think about the two titles for the
> menus:
>
> prompt "Kernel low-level debugging port"
> vs
> prompt "Low-level debug console UART"
>
> Oh wait, why don't I get to choose my "debug console UART" on an AT91?
> Maybe AT91 should move their debug uart selection into this menu as well,
> and maybe the Versatile Express options too, because they're all to do
> with selecting the UART to be used. Please... some sane *thought* would
> be *really* good here.
>
> Oh my god, you're not the only ones. Arnd/Olof, who started this madness
> and _why_ haven't you already stepped on it? Right, I'm fixing this in
> this merge window. Everything is moving under the original choice menu
> as it was intended to be.
>
> The attempts are all on the builder website against the (disabled)
> omap4430-sdp entry.
Okay, I guessed that the OMAP4430SDP was "blaze" (it's not obvious to
use internal codenames for boards when they're known as "SDP" etc -
especially when they have stickers on them saying that they're "SDP".)
With that worked out, throwing my standard printascii() hack into the
kernel results in boot messages... up to the point where the timer is
calibrated. So, it looks like either interrupts, clocks, or the OMAP
timers are non-functional with DT based kernels on the SDP board.
Any ideas?
next prev parent reply other threads:[~2013-07-06 13:36 UTC|newest]
Thread overview: 50+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-05-17 19:17 [PATCH 0/6] Drop legacy booting for omap4 for v3.11, boot with device tree only Tony Lindgren
2013-05-17 19:17 ` [PATCH 1/6] ARM: OMAP2+: Remove board-4430sdp.c Tony Lindgren
2013-05-20 9:54 ` Russell King - ARM Linux
2013-05-20 17:10 ` Tony Lindgren
2013-07-06 13:10 ` Russell King - ARM Linux
2013-07-06 13:36 ` Russell King - ARM Linux [this message]
2013-07-08 9:34 ` Tony Lindgren
2013-07-08 14:21 ` Russell King - ARM Linux
2013-07-09 8:23 ` Tony Lindgren
2013-07-12 9:09 ` Peter Ujfalusi
2013-07-22 9:40 ` Tomi Valkeinen
2013-10-27 13:26 ` Javier Martinez Canillas
2013-10-28 13:32 ` Tomi Valkeinen
2013-10-28 14:25 ` Javier Martinez Canillas
2013-07-13 2:31 ` Olof Johansson
2013-07-13 3:10 ` Olof Johansson
2013-07-15 6:55 ` Tony Lindgren
2013-07-06 21:36 ` Arnd Bergmann
2013-07-06 23:37 ` debugging options (was: Re: [PATCH 1/6] ARM: OMAP2+: Remove board-4430sdp.c) Russell King - ARM Linux
2013-05-17 19:17 ` [PATCH 2/6] ARM: OMAP2+: Remove board-omap4panda.c Tony Lindgren
2013-06-03 15:28 ` Ming Lei
2013-06-12 16:31 ` Tony Lindgren
2013-06-12 17:14 ` Tony Lindgren
2013-06-13 9:21 ` Sricharan R
2013-06-13 10:12 ` Sricharan R
2013-06-13 11:05 ` Tony Lindgren
2013-06-13 13:36 ` Ming Lei
2013-06-14 13:31 ` Ming Lei
2013-06-14 13:58 ` Ming Lei
2013-06-17 7:42 ` Sricharan R
2013-06-17 8:06 ` Arnaud Patard
2013-06-17 8:16 ` Ming Lei
2013-06-17 8:27 ` Tony Lindgren
2013-06-17 8:41 ` Ming Lei
2013-06-17 8:49 ` Tony Lindgren
2013-06-17 8:24 ` Tony Lindgren
2013-06-17 9:46 ` Arnaud Patard
2013-06-17 10:26 ` Tony Lindgren
2013-06-18 11:27 ` Tony Lindgren
2013-06-19 9:45 ` Arnaud Patard
2013-06-19 9:58 ` Tony Lindgren
2013-06-20 7:29 ` Arnaud Patard
2013-06-21 6:01 ` Tony Lindgren
2013-05-17 19:17 ` [PATCH 3/6] ARM: OMAP2+: Remove legacy mux data for omap4 Tony Lindgren
2013-05-17 19:17 ` [PATCH 4/6] ARM: OMAP2+: Remove omap4 pdata from hsmmc.c Tony Lindgren
2013-05-17 19:18 ` [PATCH 5/6] ARM: OMAP2+: Remove omap4 pdata for USB Tony Lindgren
2013-05-17 19:18 ` [PATCH 6/6] ARM: OMAP2+: Remove omap4 ocp2scp pdata Tony Lindgren
2013-05-17 20:05 ` Felipe Balbi
2013-05-21 11:53 ` Kishon Vijay Abraham I
2013-06-07 10:05 ` [PATCH 0/6] Drop legacy booting for omap4 for v3.11, boot with device tree only Tomi Valkeinen
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=20130706133627.GV21614@n2100.arm.linux.org.uk \
--to=linux@arm.linux.org.uk \
--cc=arm@kernel.org \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-omap@vger.kernel.org \
--cc=tony@atomide.com \
/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).