On Tue, 2010-04-27 at 15:01 +1000, Dave Airlie wrote: > On Tue, Apr 27, 2010 at 5:03 AM, Adam Jackson <ajax@xxxxxxxxxx> wrote: > > Signed-off-by: Adam Jackson <ajax@xxxxxxxxxx> > > not sure about the kernel-parameters additions, since the option would > be drm.edid_threshhold= and its a module option, though now that we > can accept module options on the command line, we could start adding > them to parameters file. However you'd need to add the drm. bit to the > front of the option. The doc already kind of mentions that, but it's kind of wrong. It only talks about "passing options with modprobe" and "passing options to built-in modules on kcmdline", but not about "passing options to loadable modules on kcmdline". I'll follow up on that to lkml. > How much broken hw we seeing that this fixes? maybe some bug > references, as I hate having special options that people have to know > just to see somehting on screen. I've seen at least two cases in bug reports. I could dig up a reference I suppose. That's a valid point though: we currently don't distinguish between "connected but with broken EDID" and "disconnected". I think it sounds reasonable to treat the former as connector_status_unknown instead. That way, if nothing else is connected, X will still light you up at the fallback size. Sound right? - ajax
Attachment:
signature.asc
Description: This is a digitally signed message part
_______________________________________________ dri-devel mailing list dri-devel@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/dri-devel
