Messages in this thread | | | Subject | Re: [Intel-gfx] [PATCH] drm/i915: disable LVDS downclock by default | From | Peter Clifton <> | Date | Thu, 14 Jan 2010 20:58:31 +0000 |
| |
On Thu, 2010-01-14 at 12:48 -0800, Jesse Barnes wrote: > Many platform support this feature, and it can provide significant > power savings when the reduced refresh rate is low. However, on some > platforms a secondary (reduced) timing is provided but not actually > supported by the hardware. This results in undesirable flicker at > runtime. > > So disable the feature by default, but allow users to opt-in to the > reduced clock behavior with a new module parameter, lvds_downclock, > that can be set to 1 to enable the feature.
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?
Requiring special module parameters to enable the feature, almost guarantees that no normal end-users will end up benefiting from the feature. Many of whom will have bought machines which don't have screwey BIOS implementations.
I think (on a general note) that vendors supplying defective BIOSen or config should be "named and shamed" in quirk tables - so eventually they will get something done about the problems for future models.
| |