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

[Russell King - ARM Linux]

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:
> > On Tue, Aug 09, 2011 at 09:41:38PM +0200, Linus Walleij wrote:
> > > From: Linus Walleij <linus.walleij at linaro.org>
> > > 
> > > 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.