[Intel-gfx] [PATCH 2/2] drm/i915/lvds: Use i915.lvds_fixed_mode= as a last resort

[Mike Isely]

On Wed, 20 Apr 2011, Chris Wilson wrote:

> On Wed, 20 Apr 2011 14:48:36 -0500 (CDT), Mike Isely <isely at isely.net> wrote:
> > 
> > Chris:
> > 
> > I've tested this patch and it doesn't help us here.  Even if I add 
> > lvds_fixed_mode=<whatever>, the display still comes up with the messed 
> > up configuration from the motherboard's completely ignorant BIOS.  It 
> > appears that lvds_fixed_mode is being ignored by the driver.
> 
> You can test the functionality of the patch by parsing
> i915.lvds_fixed_mode first rather than last.

I will test for that - it was the next thing I was going to dig into.


> 
> Then I just have to weigh up the wishes of Dave who has hordes of users
> desperate to fry their hardware, versus the tiny number who truly need
> an override and know what they are doing...

I understand your sentiment here.  But please also consider that this 
same feature existed on the UMS side for 3 years and I don't remember 
hearing about any flood of fried hardware because of it...

And really, this should be all about making legitimate capabilities 
available, not deliberately blocking them.

A good compromise, if this is really viewed as a legitimate problem 
(which I don't think it should be), would be to add an "I know what I'm 
doing, darnit" switch to the driver which would enable potentially 
hazardous tweaks.  Then one can hang all the "here there be dragons", 
"do not enter", "don't blame me for frying your hardware", etc caveats 
and warnings onto that switch...

  -Mike


-- 

Mike Isely
isely @ isely (dot) net
PGP: 03 54 43 4D 75 E5 CC 92 71 16 01 E2 B5 F5 C1 E8