From: arnd@arndb.de (Arnd Bergmann)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH] ARM: Allow MULTIPLATFORM to select XIP
Date: Fri, 19 Feb 2016 16:06:40 +0100 [thread overview]
Message-ID: <3699765.F7sfuMCyKT@wuerfel> (raw)
In-Reply-To: <HK2PR06MB05613CF6C996BF8BD459842E8AA00@HK2PR06MB0561.apcprd06.prod.outlook.com>
On Friday 19 February 2016 14:46:36 Chris Brandt wrote:
> On 19 Feb 2016, Arnd Bergmann wrote:
>
> > > A car with the best seat belts and airbags still won't help you
> > > if you drive it off a cliff.
> >
> > That was my point about DEBUG_LL: We already allow picking a DEBUG_LL
> > option that immediately breaks booting on any other platform, and
> > (worse) breaks even printing any useful output about that fact.
>
> Ya, the DEBUG_LL issue seems like it could be an endless hole of options
> all throughout the Kconfigs.
>
> DT was supposed to allow everything to be configured at boot time, but
> it looks like we still need an early-Device-Tree for settings before DT
> is available. That would solve your DEBUG_LL and PHYS_OFFSET issue.
Not really: for DEBUG_LL, it doesn't work because DEBUG_LL is literally
meant to work from the first instruction in the kernel, so there can't
be any code to configure it. If you don't need to go that low-level,
you should use "earlycon", which can parse the DT just fine.
For PHYS_OFFSET, parsing the DT doesn't help at all because we only
need it for XIP_KERNEL (more or less at least) which cannot patch
the kernel image at boot time, so knowing that the address is wrong
also doesn't help you.
Arnd
next prev parent reply other threads:[~2016-02-19 15:06 UTC|newest]
Thread overview: 18+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-02-18 17:25 [PATCH] ARM: Allow MULTIPLATFORM to select XIP Chris Brandt
2016-02-18 17:48 ` Nicolas Pitre
2016-02-18 18:18 ` Chris Brandt
2016-02-18 18:31 ` Nicolas Pitre
2016-02-18 19:42 ` Chris Brandt
2016-02-18 22:33 ` Russell King - ARM Linux
2016-02-19 3:39 ` Chris Brandt
2016-02-19 11:11 ` Arnd Bergmann
2016-02-19 13:49 ` Chris Brandt
2016-02-19 14:21 ` Arnd Bergmann
2016-02-19 14:46 ` Chris Brandt
2016-02-19 15:06 ` Arnd Bergmann [this message]
2016-02-19 15:36 ` Chris Brandt
2016-02-19 15:59 ` Arnd Bergmann
2016-02-19 16:28 ` Chris Brandt
2016-02-19 16:39 ` Arnd Bergmann
2016-02-19 16:47 ` Russell King - ARM Linux
2016-02-23 19:15 ` [PATCH v2] " Chris Brandt
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=3699765.F7sfuMCyKT@wuerfel \
--to=arnd@arndb.de \
--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