Re: [Intel-gfx] [PATCH] drm/i915: disable LVDS downclock by default

[Peter Clifton]

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.



--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/