On Tue, Aug 20, 2013 at 01:08:22AM +0100, Damien Lespiau wrote: > On Mon, Aug 19, 2013 at 04:20:40PM -0700, Ben Widawsky wrote: > > We generally don't want people or distros to use this option unless they > > know what they're doing. I missed the initial conversation but it's > > likely a way for people who have a built-in i915.ko and have no other > > way to change the behavior. > > It's to be able to have a config file to generate kernels supporting > preliminary hardware, without having to always add the command line > parameter or carry a patch or changing the bootloader configuration. Exactly. > > As such: > > Set default to n > > Display message for what users should select (N) > > and while there, a small whitespace fix. > > > > Cc Josh Triplett <josh@xxxxxxxxxxxxxxxx> > > Missing the ':' so you didn't end up Ccing Josh. Thanks for fixing that. :) > > Signed-off-by: Ben Widawsky <ben@xxxxxxxxxxxx> > > --- > > drivers/gpu/drm/Kconfig | 5 ++++- > > 1 file changed, 4 insertions(+), 1 deletion(-) > > > > diff --git a/drivers/gpu/drm/Kconfig b/drivers/gpu/drm/Kconfig > > index 62a06c7..ad4e369 100644 > > --- a/drivers/gpu/drm/Kconfig > > +++ b/drivers/gpu/drm/Kconfig > > @@ -171,12 +171,15 @@ config DRM_I915_KMS > > config DRM_I915_PRELIMINARY_HW_SUPPORT > > bool "Enable preliminary support for prerelease Intel hardware by default" > > depends on DRM_I915 > > + default n > > That's the default, you don't have to make it explicit (but of course it > doesn't hurt). make oldconfig with this new option gives: > > Enable preliminary support for prerelease Intel hardware by default > (DRM_I915_PRELIMINARY_HW_SUPPORT) [N/y/?] (NEW) Right. I definitely would *not* have set this to default y, but default n is the default default, so it doesn't need stating. - Josh Triplett _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/intel-gfx
