[Intel-gfx] [PATCH 8/8] drm/i915: allow Package C8+ by default

[Chris Wilson]

On Mon, Jul 29, 2013 at 05:48:27PM -0300, Paulo Zanoni wrote:
> From: Paulo Zanoni <paulo.r.zanoni at intel.com>
> 
> Things should be working, so enable it by default.
> 
> Signed-off-by: Paulo Zanoni <paulo.r.zanoni at intel.com>
> ---
>  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