Messages in this thread | | | Date | Thu, 14 Jan 2010 13:21:38 -0800 | From | Jesse Barnes <> | Subject | Re: [Intel-gfx] [PATCH] drm/i915: disable LVDS downclock by default |
| |
On Thu, 14 Jan 2010 13:05:17 -0800 (PST) Linus Torvalds <torvalds@linux-foundation.org> wrote:
> > > On Thu, 14 Jan 2010, Peter Clifton wrote: > > > > Would it not be a better idea to turn this feature on by default, > > then use quirks to disable it on the afflicted borken machines? > > No. > > "Working" is good. "Broken" is bad. > > We don't default to clearly unsafe models. > > Maybe a few years from now, when people have learnt to do power > management correctly and BIOSes fill in the fields right, we can > reconsider. But right now, it's _way_ more important that things work > reliably. > > That said, I think a module parameter is the wrong thing. If this can > be done dynamically with a sysfs value, do it that way instead (yes, > I realize that module parameters end up being also visible in /sys, > but I think Jesse's patch doesn't allow a person to set the value - > and make it change the behavior - _while_ the display is all up and > running).
Having a runtime option would be good, but we'd have to do some extra code for that since the feature is either enabled or disabled at LVDS and VBIOS parse time only.
With a sysfs interface we could allow the user to select the timing for their reduced mode though, which would give them more power saving potentially (if their panel worked with a very low refresh).
I can do that, but it seems more appropriate for 2.6.34? I'd just remove the module param at that time I guess.
-- Jesse Barnes, Intel Open Source Technology Center
| |