From 592d814ee09e86e283116a7a1052762c8398e8e5 Mon Sep 17 00:00:00 2001 From: Adam Jackson Date: Fri, 28 Dec 2007 16:37:23 -0500 Subject: [PATCH] EDID 1.4: Additional semantics for display feature bits. First mode is _always_ preferred in 1.4; the bit that used to mean this now means that the preferred mode is also the native pixel format. The old "is GTF" bit now means "is continuous-frequency" instead. Section 3.6.4, Table 3.14: Feature Support, Notes 4 and 5. --- hw/xfree86/ddc/print_edid.c | 23 +++++++++++++++++------ 1 file changed, 17 insertions(+), 6 deletions(-) diff --git a/hw/xfree86/ddc/print_edid.c b/hw/xfree86/ddc/print_edid.c index 30cd17588..a55c46523 100644 --- a/hw/xfree86/ddc/print_edid.c +++ b/hw/xfree86/ddc/print_edid.c @@ -154,16 +154,27 @@ print_dpms_features(int scrnIndex, struct disp_features *c, if (STD_COLOR_SPACE(c->msc)) xf86DrvMsg(scrnIndex,X_INFO, "Default color space is primary color space\n"); - if (PREFERRED_TIMING_MODE(c->msc)) - xf86DrvMsg(scrnIndex,X_INFO, + + if (PREFERRED_TIMING_MODE(c->msc) || v->revision >= 4) { + xf86DrvMsg(scrnIndex, X_INFO, "First detailed timing is preferred mode\n"); - else if (v->version == 1 && v->revision >= 3) + if (v->revision >= 4) + xf86DrvMsg(scrnIndex, X_INFO, + "Preferred mode is native pixel format and refresh rate\n"); + } else if (v->revision == 3) { xf86DrvMsg(scrnIndex,X_INFO, "First detailed timing not preferred " "mode in violation of standard!"); - if (GFT_SUPPORTED(c->msc)) - xf86DrvMsg(scrnIndex,X_INFO, - "GTF timings supported\n"); + } + + if (v->revision >= 4) { + if (GFT_SUPPORTED(c->msc)) { + xf86DrvMsg(scrnIndex, X_INFO, "Display is continuous-frequency\n"); + } + } else { + if (GFT_SUPPORTED(c->msc)) + xf86DrvMsg(scrnIndex, X_INFO, "GTF timings supported\n"); + } } static void