diff options
| author | Matthias Hopf <mhopf@suse.de> | 2007-04-18 17:36:15 +0200 |
|---|---|---|
| committer | Matthias Hopf <mhopf@suse.de> | 2007-04-18 17:36:15 +0200 |
| commit | 16ef77df4ebaf5ea13baa82972aaf98e71ac32ee (patch) | |
| tree | 4c2796d9539331f4d537dc344719d7a0277b8466 | |
| parent | 0abce69f0d826a7ca1a41d963cd4730b6e01c145 (diff) | |
Set sync polarity restriction flags even for non-"digital separate" monitors.
According to Lisa Wu, this is correct regarding the VESA EEDID standard.
Bugzilla #10620 (2), original patch by Lisa Wu @ATI
| -rw-r--r-- | src/radeon_display.c | 12 | ||||
| -rw-r--r-- | src/radeon_modes.c | 12 |
2 files changed, 10 insertions, 14 deletions
diff --git a/src/radeon_display.c b/src/radeon_display.c index ac05648..57e752e 100644 --- a/src/radeon_display.c +++ b/src/radeon_display.c @@ -755,13 +755,11 @@ static void RADEONUpdatePanelSize(ScrnInfoPtr pScrn) info->VSyncWidth = d_timings->v_sync_width; info->VBlank = d_timings->v_blanking; info->Flags = (d_timings->interlaced ? V_INTERLACE : 0); - if (d_timings->sync == 3) { - switch (d_timings->misc) { - case 0: info->Flags |= V_NHSYNC | V_NVSYNC; break; - case 1: info->Flags |= V_PHSYNC | V_NVSYNC; break; - case 2: info->Flags |= V_NHSYNC | V_PVSYNC; break; - case 3: info->Flags |= V_PHSYNC | V_PVSYNC; break; - } + switch (d_timings->misc) { + case 0: info->Flags |= V_NHSYNC | V_NVSYNC; break; + case 1: info->Flags |= V_PHSYNC | V_NVSYNC; break; + case 2: info->Flags |= V_NHSYNC | V_PVSYNC; break; + case 3: info->Flags |= V_PHSYNC | V_PVSYNC; break; } xf86DrvMsg(pScrn->scrnIndex, X_INFO, "Panel infos found from DDC detailed: %dx%d\n", info->PanelXRes, info->PanelYRes); diff --git a/src/radeon_modes.c b/src/radeon_modes.c index 6b8577e..3d147e2 100644 --- a/src/radeon_modes.c +++ b/src/radeon_modes.c @@ -190,13 +190,11 @@ static DisplayModePtr RADEONDDCModes(ScrnInfoPtr pScrn, xf86MonPtr ddc) #endif new->type |= M_T_DRIVER; - if (d_timings->sync == 3) { - switch (d_timings->misc) { - case 0: new->Flags |= V_NHSYNC | V_NVSYNC; break; - case 1: new->Flags |= V_PHSYNC | V_NVSYNC; break; - case 2: new->Flags |= V_NHSYNC | V_PVSYNC; break; - case 3: new->Flags |= V_PHSYNC | V_PVSYNC; break; - } + switch (d_timings->misc) { + case 0: new->Flags |= V_NHSYNC | V_NVSYNC; break; + case 1: new->Flags |= V_PHSYNC | V_NVSYNC; break; + case 2: new->Flags |= V_NHSYNC | V_PVSYNC; break; + case 3: new->Flags |= V_PHSYNC | V_PVSYNC; break; } count++; |