From: pawel.moll@arm.com (Pawel Moll)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH 9/9] ARM: vexpress: Config option for early printk console
Date: Wed, 11 Jul 2012 15:16:21 +0100 [thread overview]
Message-ID: <1342016181.30755.2.camel@hornet> (raw)
In-Reply-To: <20120711140912.GC13498@mudshark.cambridge.arm.com>
On Wed, 2012-07-11 at 15:09 +0100, Will Deacon wrote:
> > +#else /* CONFIG_DEBUG_LL_UART_NONE */
> > +
> > + .macro addruart, rp, rv, tmp
> > + /*
> > + * Have to provide reasonable dummy values,
> > + * otherwise code in head.S crashes badly...
> > + */
> > + mov \rp, #0
> > + mov \rv, #0
> > + .endm
>
> Eek, this means we take out an I/O section mapping at zero, doesn't it?
> Is that really a good idea? It scares me because (a) it's below PAGE_OFFSET
> and (b) we don't know consistently what's at physical address 0.
>
> Perhaps we could change head.S to avoid the mapping if addruart returns a
> NULL virtual address?
Sure (care to prepare a patch? ;-), but this will have to go through
Russell not arm-soc, so it won't be a part of this update. And I'll
remove the comment...
Cheers!
Pawe?
next prev parent reply other threads:[~2012-07-11 14:16 UTC|newest]
Thread overview: 32+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-07-11 13:36 [PATCH 0/9] Versatile Express updates for 3.6 Pawel Moll
2012-07-11 13:36 ` [PATCH 1/9] ARM: vexpress: remove automatic errata workaround selection Pawel Moll
2012-07-11 15:02 ` Rob Herring
2012-07-11 15:18 ` Will Deacon
2012-07-11 15:45 ` Rob Herring
2012-07-11 15:53 ` Will Deacon
2012-07-11 18:06 ` Rob Herring
2012-07-11 18:14 ` Will Deacon
2012-07-12 10:17 ` [PATCH v2] " Pawel Moll
2012-07-12 14:14 ` Will Deacon
2012-07-12 14:39 ` Rob Herring
2012-07-11 13:36 ` [PATCH 2/9] ARM: vexpress: Check master site in daughterboard's sysctl operations Pawel Moll
2012-07-11 13:36 ` [PATCH 3/9] ARM: versatile: Make plat-versatile clock optional Pawel Moll
2012-07-11 16:43 ` Linus Walleij
2012-07-11 16:58 ` Pawel Moll
2012-07-12 0:11 ` Mike Turquette
2012-07-11 13:36 ` [PATCH 4/9] ARM: vexpress: Initial common clock support Pawel Moll
2012-07-11 16:51 ` Linus Walleij
2012-07-11 16:57 ` Pawel Moll
2012-07-16 20:15 ` Turquette, Mike
2012-07-17 9:48 ` Pawel Moll
2012-07-11 13:36 ` [PATCH 5/9] ARM: vexpress: Add missing SP804 interrupt in motherboard's DTS files Pawel Moll
2012-07-11 13:36 ` [PATCH 6/9] ARM: vexpress: Add fixed regulator for SMSC Pawel Moll
2012-07-11 13:36 ` [PATCH 7/9] ARM: vexpress: Convert V2P-CA15 Device Tree to 64 bit addresses Pawel Moll
2012-07-11 13:36 ` [PATCH 8/9] ARM: vexpress: Add Device Tree for V2P-CA15_CA7 core tile Pawel Moll
2012-07-11 13:36 ` [PATCH 9/9] ARM: vexpress: Config option for early printk console Pawel Moll
2012-07-11 14:09 ` Will Deacon
2012-07-11 14:16 ` Pawel Moll [this message]
2012-07-11 14:23 ` Pawel Moll
2012-07-11 14:23 ` Jonathan Austin
2012-07-11 14:24 ` Pawel Moll
2012-07-11 16:21 ` [PATCH v2] " Pawel Moll
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=1342016181.30755.2.camel@hornet \
--to=pawel.moll@arm.com \
--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).