[PATCH v4 0/2] Enhance EDID quirks to allow forcing a mode

2013-06-18 Thread Dylan Semler
.html [2] http://lists.freedesktop.org/archives/dri-devel/2013-April/036829.html On Mon, Mar 25, 2013 at 5:58 PM, Dylan Semler wrote: > Changes in this version > * rename do_force_quirk_modes() -> do_force_quirk_mode() > * use list_for_each_entry() instead of list_for_eac

Re: [PATCH v4 0/2] Enhance EDID quirks to allow forcing a mode

2013-06-18 Thread Dylan Semler
.html [2] http://lists.freedesktop.org/archives/dri-devel/2013-April/036829.html On Mon, Mar 25, 2013 at 5:58 PM, Dylan Semler dylan.sem...@gmail.comwrote: Changes in this version * rename do_force_quirk_modes() - do_force_quirk_mode() * use list_for_each_entry() instead

[PATCH v4 0/2] Enhance EDID quirks to allow forcing a mode

2013-04-03 Thread Dylan Semler
On Mon, Mar 25, 2013 at 5:58 PM, Dylan Semler wrote: > > Changes in this version > * rename do_force_quirk_modes() -> do_force_quirk_mode() > * use list_for_each_entry() instead of list_for_each_entry_safe() in >do_force_quirk_mode() > * remove num_modes from do_for

Re: [PATCH v4 0/2] Enhance EDID quirks to allow forcing a mode

2013-04-03 Thread Dylan Semler
On Mon, Mar 25, 2013 at 5:58 PM, Dylan Semler dylan.sem...@gmail.com wrote: Changes in this version * rename do_force_quirk_modes() - do_force_quirk_mode() * use list_for_each_entry() instead of list_for_each_entry_safe() in do_force_quirk_mode() * remove num_modes from

[PATCH v4 2/2] drm: Add EDID force quirk for MMT Monitor2Go HD+

2013-03-25 Thread Dylan Semler
Set the new EDID_QUIRK_FORCE_MODE bit for the MMT Monitor2Go HD+ monitor and add it to the edid_quirk_force_mode_list. Signed-off-by: Dylan Semler --- drivers/gpu/drm/drm_edid.c | 5 + 1 file changed, 5 insertions(+) diff --git a/drivers/gpu/drm/drm_edid.c b/drivers/gpu/drm/drm_edid.c

[PATCH v4 1/2] drm: Enhance EDID quirks to explicitly set a mode

2013-03-25 Thread Dylan Semler
bit needs to be set for the new quirk list discribed above to be checked. Signed-off-by: Dylan Semler --- drivers/gpu/drm/drm_edid.c | 77 ++ 1 file changed, 77 insertions(+) diff --git a/drivers/gpu/drm/drm_edid.c b/drivers/gpu/drm/drm_edid.c ind

[PATCH v4 0/2] Enhance EDID quirks to allow forcing a mode

2013-03-25 Thread Dylan Semler
l other modes. The first patch also defines a new quirk bit: EDID_QUIRK_FORCE_MODE. This bit needs to be set for the new quirk list described above to be checked. The second patch adds the offending monitor to the quirk lists. Dylan Semler (2): drm: Enhance EDID quirks to explicitly set a mode

[PATCH v3 1/2] drm: Enhance EDID quirks to explicitly set a mode

2013-03-25 Thread Dylan Semler
On Mon, Mar 25, 2013 at 9:41 AM, Jani Nikula wrote: > > > Hi, please find some review comments inline. > You make some good points, thanks. > On Sat, 23 Mar 2013, Dylan Semler wrote: > > > > +/* Add an explicit mode based on a quirk > > + */ > > +stat

Re: [PATCH v3 1/2] drm: Enhance EDID quirks to explicitly set a mode

2013-03-25 Thread Dylan Semler
On Mon, Mar 25, 2013 at 9:41 AM, Jani Nikula jani.nik...@linux.intel.com wrote: Hi, please find some review comments inline. You make some good points, thanks. On Sat, 23 Mar 2013, Dylan Semler dylan.sem...@gmail.com wrote: +/* Add an explicit mode based on a quirk + */ +static int

[PATCH v4 0/2] Enhance EDID quirks to allow forcing a mode

2013-03-25 Thread Dylan Semler
. The first patch also defines a new quirk bit: EDID_QUIRK_FORCE_MODE. This bit needs to be set for the new quirk list described above to be checked. The second patch adds the offending monitor to the quirk lists. Dylan Semler (2): drm: Enhance EDID quirks to explicitly set a mode drm: Add EDID

[PATCH v4 1/2] drm: Enhance EDID quirks to explicitly set a mode

2013-03-25 Thread Dylan Semler
to be set for the new quirk list discribed above to be checked. Signed-off-by: Dylan Semler dylan.sem...@gmail.com --- drivers/gpu/drm/drm_edid.c | 77 ++ 1 file changed, 77 insertions(+) diff --git a/drivers/gpu/drm/drm_edid.c b/drivers/gpu/drm/drm_edid.c

[PATCH v3 2/2] drm: Add EDID force quirk for MMT Monitor2Go HD+

2013-03-22 Thread Dylan Semler
Set the new EDID_QUIRK_FORCE_MODE bit for the MMT Monitor2Go HD+ monitor and add it to the edid_quirk_force_mode_list. Signed-off-by: Dylan Semler --- drivers/gpu/drm/drm_edid.c | 5 + 1 file changed, 5 insertions(+) diff --git a/drivers/gpu/drm/drm_edid.c b/drivers/gpu/drm/drm_edid.c

[PATCH v3 1/2] drm: Enhance EDID quirks to explicitly set a mode

2013-03-22 Thread Dylan Semler
bit needs to be set for the new quirk list discribed above to be checked. Signed-off-by: Dylan Semler --- drivers/gpu/drm/drm_edid.c | 78 ++ 1 file changed, 78 insertions(+) diff --git a/drivers/gpu/drm/drm_edid.c b/drivers/gpu/drm/drm_edid.c ind

[PATCH v3 0/2] Enhance EDID quirks to allow forcing a mode

2013-03-22 Thread Dylan Semler
be checked. The second patch adds the offending monitor to the quirk lists. Dylan Semler (2): drm: Enhance EDID quirks to explicitly set a mode drm: Add EDID force quirk for MMT Monitor2Go HD+ drivers/gpu/drm/drm_edid.c | 83 ++ 1 file changed, 83

Enhance EDID quirks to allow forcing a mode

2013-03-22 Thread Dylan Semler
On Fri, Mar 22, 2013 at 10:41 AM, Daniel Vetter wrote: > > On Fri, Mar 22, 2013 at 3:02 PM, Dylan Semler wrote: > > On Fri, Mar 22, 2013 at 9:50 AM, Alex Deucher > > wrote: > >> > >> That's odd. Maybe it's actually in an extension block or something l

Enhance EDID quirks to allow forcing a mode

2013-03-22 Thread Dylan Semler
On Fri, Mar 22, 2013 at 9:50 AM, Alex Deucher wrote: > > On Thu, Mar 21, 2013 at 5:42 PM, Dylan Semler wrote: > > Oops. I neglected to preface this with my motivation: I have a new > > monitor that doesn't report its native resolution in its EDID block. It > >

[PATCH v2 2/2] Adds EDID force mode quirk for MMT Monitor2Go HD+.

2013-03-22 Thread Dylan Semler
Signed-off-by: Dylan Semler --- drivers/gpu/drm/drm_edid.c | 5 + 1 file changed, 5 insertions(+) diff --git a/drivers/gpu/drm/drm_edid.c b/drivers/gpu/drm/drm_edid.c index b1036b5..b400971 100644 --- a/drivers/gpu/drm/drm_edid.c +++ b/drivers/gpu/drm/drm_edid.c @@ -109,6 +109,9 @@ static

[PATCH v2 1/2] Enhances EDID quirks to allow for specifying and preferring a mode not reported in the EDID block.

2013-03-22 Thread Dylan Semler
Signed-off-by: Dylan Semler --- drivers/gpu/drm/drm_edid.c | 72 ++ 1 file changed, 72 insertions(+) diff --git a/drivers/gpu/drm/drm_edid.c b/drivers/gpu/drm/drm_edid.c index c194f4e..b1036b5 100644 --- a/drivers/gpu/drm/drm_edid.c +++ b/drivers/gpu

[PATCH v2 0/2] Enhance EDID quirks to allow forcing a mode

2013-03-22 Thread Dylan Semler
above to be checked. Dylan Semler (2): Enhances EDID quirks to allow for specifying and preferring a mode not reported in the EDID block. Adds EDID force mode quirk for MMT Monitor2Go HD+. drivers/gpu/drm/drm_edid.c | 77 ++ 1 file changed, 77

Enhance EDID quirks to allow forcing a mode

2013-03-22 Thread Dylan Semler
On Fri, Mar 22, 2013 at 4:48 AM, Daniel Vetter wrote: > > I think it'd be good to shovel these text blocks into the (currently rather > empty) commit messages of the patches. Since when reading old commits with > e.g. git blame that's what people will read. Yeah, I just noticed that. For some

Re: Enhance EDID quirks to allow forcing a mode

2013-03-22 Thread Dylan Semler
On Fri, Mar 22, 2013 at 4:48 AM, Daniel Vetter dan...@ffwll.ch wrote: I think it'd be good to shovel these text blocks into the (currently rather empty) commit messages of the patches. Since when reading old commits with e.g. git blame that's what people will read. Yeah, I just noticed that.

[PATCH v2 0/2] Enhance EDID quirks to allow forcing a mode

2013-03-22 Thread Dylan Semler
above to be checked. Dylan Semler (2): Enhances EDID quirks to allow for specifying and preferring a mode not reported in the EDID block. Adds EDID force mode quirk for MMT Monitor2Go HD+. drivers/gpu/drm/drm_edid.c | 77 ++ 1 file changed, 77

[PATCH v2 1/2] Enhances EDID quirks to allow for specifying and preferring a mode not reported in the EDID block.

2013-03-22 Thread Dylan Semler
Signed-off-by: Dylan Semler dylan.sem...@gmail.com --- drivers/gpu/drm/drm_edid.c | 72 ++ 1 file changed, 72 insertions(+) diff --git a/drivers/gpu/drm/drm_edid.c b/drivers/gpu/drm/drm_edid.c index c194f4e..b1036b5 100644 --- a/drivers/gpu/drm

[PATCH v2 2/2] Adds EDID force mode quirk for MMT Monitor2Go HD+.

2013-03-22 Thread Dylan Semler
Signed-off-by: Dylan Semler dylan.sem...@gmail.com --- drivers/gpu/drm/drm_edid.c | 5 + 1 file changed, 5 insertions(+) diff --git a/drivers/gpu/drm/drm_edid.c b/drivers/gpu/drm/drm_edid.c index b1036b5..b400971 100644 --- a/drivers/gpu/drm/drm_edid.c +++ b/drivers/gpu/drm/drm_edid.c

Re: Enhance EDID quirks to allow forcing a mode

2013-03-22 Thread Dylan Semler
On Fri, Mar 22, 2013 at 9:50 AM, Alex Deucher alexdeuc...@gmail.com wrote: On Thu, Mar 21, 2013 at 5:42 PM, Dylan Semler dylan.sem...@gmail.com wrote: Oops. I neglected to preface this with my motivation: I have a new monitor that doesn't report its native resolution in its EDID block

Re: Enhance EDID quirks to allow forcing a mode

2013-03-22 Thread Dylan Semler
On Fri, Mar 22, 2013 at 10:41 AM, Daniel Vetter dan...@ffwll.ch wrote: On Fri, Mar 22, 2013 at 3:02 PM, Dylan Semler dylan.sem...@gmail.com wrote: On Fri, Mar 22, 2013 at 9:50 AM, Alex Deucher alexdeuc...@gmail.com wrote: That's odd. Maybe it's actually in an extension block

[PATCH v3 0/2] Enhance EDID quirks to allow forcing a mode

2013-03-22 Thread Dylan Semler
patch adds the offending monitor to the quirk lists. Dylan Semler (2): drm: Enhance EDID quirks to explicitly set a mode drm: Add EDID force quirk for MMT Monitor2Go HD+ drivers/gpu/drm/drm_edid.c | 83 ++ 1 file changed, 83 insertions

[PATCH v3 1/2] drm: Enhance EDID quirks to explicitly set a mode

2013-03-22 Thread Dylan Semler
to be set for the new quirk list discribed above to be checked. Signed-off-by: Dylan Semler dylan.sem...@gmail.com --- drivers/gpu/drm/drm_edid.c | 78 ++ 1 file changed, 78 insertions(+) diff --git a/drivers/gpu/drm/drm_edid.c b/drivers/gpu/drm/drm_edid.c

[PATCH v3 2/2] drm: Add EDID force quirk for MMT Monitor2Go HD+

2013-03-22 Thread Dylan Semler
Set the new EDID_QUIRK_FORCE_MODE bit for the MMT Monitor2Go HD+ monitor and add it to the edid_quirk_force_mode_list. Signed-off-by: Dylan Semler dylan.sem...@gmail.com --- drivers/gpu/drm/drm_edid.c | 5 + 1 file changed, 5 insertions(+) diff --git a/drivers/gpu/drm/drm_edid.c b/drivers

Enhance EDID quirks to allow forcing a mode

2013-03-21 Thread Dylan Semler
enhancement to allow specifying a new mode and then setting it as preferred. On Thu, Mar 21, 2013 at 5:36 PM, Dylan Semler wrote: > The idea is setup a new quirk list with width, height, and refresh rates > for > each monitor that needs this quirk. If a monitor is attached that matc

[PATCH 2/2] Adds EDID force mode quirk for MMT Monitor2Go HD+.

2013-03-21 Thread Dylan Semler
Signed-off-by: Dylan Semler --- drivers/gpu/drm/drm_edid.c | 5 + 1 file changed, 5 insertions(+) diff --git a/drivers/gpu/drm/drm_edid.c b/drivers/gpu/drm/drm_edid.c index b1036b5..b400971 100644 --- a/drivers/gpu/drm/drm_edid.c +++ b/drivers/gpu/drm/drm_edid.c @@ -109,6 +109,9 @@ static

[PATCH 1/2] Enhances EDID quirks to allow for specifying and preferring a mode not reported in the EDID block.

2013-03-21 Thread Dylan Semler
Signed-off-by: Dylan Semler --- drivers/gpu/drm/drm_edid.c | 72 ++ 1 file changed, 72 insertions(+) diff --git a/drivers/gpu/drm/drm_edid.c b/drivers/gpu/drm/drm_edid.c index c194f4e..b1036b5 100644 --- a/drivers/gpu/drm/drm_edid.c +++ b/drivers/gpu

Enhance EDID quirks to allow forcing a mode

2013-03-21 Thread Dylan Semler
The idea is setup a new quirk list with width, height, and refresh rates for each monitor that needs this quirk. If a monitor is attached that matches one in this list, the full modeline is calculated with drm_gtf_mode, the DRM_MODE_TYPE_PREFERRED bit is set, and the new mode is added to the

[PATCH 1/2] Enhances EDID quirks to allow for specifying and preferring a mode not reported in the EDID block.

2013-03-21 Thread Dylan Semler
Signed-off-by: Dylan Semler dylan.sem...@gmail.com --- drivers/gpu/drm/drm_edid.c | 72 ++ 1 file changed, 72 insertions(+) diff --git a/drivers/gpu/drm/drm_edid.c b/drivers/gpu/drm/drm_edid.c index c194f4e..b1036b5 100644 --- a/drivers/gpu/drm

[PATCH 2/2] Adds EDID force mode quirk for MMT Monitor2Go HD+.

2013-03-21 Thread Dylan Semler
Signed-off-by: Dylan Semler dylan.sem...@gmail.com --- drivers/gpu/drm/drm_edid.c | 5 + 1 file changed, 5 insertions(+) diff --git a/drivers/gpu/drm/drm_edid.c b/drivers/gpu/drm/drm_edid.c index b1036b5..b400971 100644 --- a/drivers/gpu/drm/drm_edid.c +++ b/drivers/gpu/drm/drm_edid.c

Re: Enhance EDID quirks to allow forcing a mode

2013-03-21 Thread Dylan Semler
enhancement to allow specifying a new mode and then setting it as preferred. On Thu, Mar 21, 2013 at 5:36 PM, Dylan Semler dylan.sem...@gmail.comwrote: The idea is setup a new quirk list with width, height, and refresh rates for each monitor that needs this quirk. If a monitor is attached