On Mon, Jul 29, 2013 at 05:48:27PM -0300, Paulo Zanoni wrote: > From: Paulo Zanoni <paulo.r.zanoni@xxxxxxxxx> > > Things should be working, so enable it by default. > > Signed-off-by: Paulo Zanoni <paulo.r.zanoni@xxxxxxxxx> > --- > drivers/gpu/drm/i915/i915_drv.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/drivers/gpu/drm/i915/i915_drv.c b/drivers/gpu/drm/i915/i915_drv.c > index 6df516b..8f6e685 100644 > --- a/drivers/gpu/drm/i915/i915_drv.c > +++ b/drivers/gpu/drm/i915/i915_drv.c > @@ -141,9 +141,9 @@ module_param_named(fastboot, i915_fastboot, bool, 0600); > MODULE_PARM_DESC(fastboot, "Try to skip unnecessary mode sets at boot time " > "(default: false)"); > > -int i915_allow_pc8 __read_mostly = 0; > +int i915_allow_pc8 __read_mostly = 1; > module_param_named(allow_pc8, i915_allow_pc8, int, 0600); > -MODULE_PARM_DESC(allow_pc8, "Allow package states > 8 (default: false)"); > +MODULE_PARM_DESC(allow_pc8, "Allow package states > 8 (default: true)"); How about (after changing the variable name as earlier): "Enable support for low power package sleep states (pc8) [default: true])" Hmm, I thought we were consistent in using [default], alas not. -Chris -- Chris Wilson, Intel Open Source Technology Centre _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/intel-gfx
