public inbox for linux-arm-kernel@lists.infradead.org
 help / color / mirror / Atom feed
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

  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