Re: [Intel-gfx] [PATCH] drm/edid: Adding common CVT inferred modes when monitor allows range limited ones trough EDID.

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 4/13/12 10:29 AM, Takashi Iwai wrote:
At Fri, 13 Apr 2012 10:14:46 -0400,
Adam Jackson wrote:
Yeah, that's a bug.  That's why I said it should be renamed
drm_dmt_modes_for_range and run unconditionally if we find a range
descriptor.

Yeah, I saw your patches.  Should the further work base on them?

Would be nice.

Yesterday I've read a news reporting that 1366x768 is the most
commonly used panel now, more than 1024x768.  And, 1600x900 is in the
second place of the modern laptop panels.

Windows and others do work with these resolutions on the same
monitor.  Why Linux driver can't?  Everbody (but developers) thinks
like that way.

I think you're trying to make me defend a position I wasn't taking...

If it's not the native panel size and it's not a commonly found size in
the wild, why are we obligated to provide them for every user?  Remember
that userspace has the ability to hand in modes from above.

I don't think we need to support all wild modes, too.  But the _very_
common modes like 1366x768 and 1600x900 should be really supported as
default.

I'm not disagreeing. I think common sizes should be available, and we have code already that's intended to do that.

My issue with the list in the patch is it contains some nonsense. If some of those more weird-looking sizes _do_ exist in the wild they should be already present in EDID as the native size. For panels where they're not native I have difficulty imagining anyone wanting to set that mode intentionally. And for someone who really does want it they have the ability to pass in arbitrary crap from userspace anyway.

1600x900 is reasonable to add to an "extras" list, because it is actually common despite not being in DMT. I'm even willing to take Windows as the example for what modes should be in the extras list. But I'm not willing to take "I wish this was in a preset list" as the sole justification.

- ajax
_______________________________________________
dri-devel mailing list
dri-devel@xxxxxxxxxxxxxxxxxxxxx
http://lists.freedesktop.org/mailman/listinfo/dri-devel


[Index of Archives]     [Linux DRI Users]     [Linux Intel Graphics]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]
  Powered by Linux