Enabling runtime P2V by default (Re: [PATCH 3/5] mach-u300: patch physoffset by default)

On Wed, Aug 10, 2011 at 10:22:06AM +0100, Will Deacon wrote:

On Wed, Aug 10, 2011 at 10:16:35AM +0100, Russell King - ARM Linux wrote:

quoted

On Tue, Aug 09, 2011 at 09:41:38PM +0200, Linus Walleij wrote:

quoted

From: Linus Walleij <redacted>

This works like a charm so I'll just default-select it.

Well, we can remove the EXPERIMENTAL status of this option now.  This
raises the question is whether we should now default it to 'y' - I
think we should.  Anyone have any objections?

I've been running with this option enabled for the collection of ARM boards
I have and the only problem I have encountered was related to u-boot loading
at the wrong address.

So I'm all for enabling it by default, especially since it will force out
any remaining issues for boards where this hasn't been used extensively.

Maybe also making the option hidden depending on EXPERT, or even EMBEDDED
would be a good idea too.  I think it falls into at least the same class
as UID16, sysctl, hotplug, printk, etc. which are all EXPERT options.