Re: [PATCH] drm/i915: Extend i915_powersave parameter.

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Wed, Jul 17, 2013 at 02:45:29PM -0300, Rodrigo Vivi wrote:
> So if I understood correctly you suggest that if we have
> i915_powersave=0 and i915_enable_rc6=1 we should enable rc6?
> This is not how it is implemented nowadays on fbc right now... And
> this lead me to use pwoesave as an umbrella for force disable/enable
> ignoring individual parameters. But I'm open to do the other way
> around as long as we have a standardized behavior.

Just depends on whether powersave=0 is default and powersave=-1 is
force off, with powersave=1 as force on.
 
> So, we can either respect individual parameters when they are set or
> completely ignore. In the way it is now it doesn't respect individual
> fbc parameter for powersave=0.

My opinion is that we respect the specific module parameters, and if
they are left to default values, then apply the global powersave
parameter. If that too is default, then we apply the module default.
-Chris

-- 
Chris Wilson, Intel Open Source Technology Centre
_______________________________________________
Intel-gfx mailing list
Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
http://lists.freedesktop.org/mailman/listinfo/intel-gfx




[Index of Archives]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]
  Powered by Linux