diff options
| author | Daniel Vetter <daniel.vetter@ffwll.ch> | 2012-01-27 22:40:51 +0100 |
|---|---|---|
| committer | Dave Airlie <airlied@redhat.com> | 2012-02-13 11:58:53 +0000 |
| commit | 8bf42225dac865665a5a0f8bbe1b62139470ffa0 (patch) | |
| tree | 1f8d5cf4aa5599fb0b4e739279c29e9603a0fc3e /drivers/gpu/drm/drm_modes.c | |
| parent | fb2a99e15ff0d342de4ba58c84a791224a96a01a (diff) | |
| download | blackbird-op-linux-8bf42225dac865665a5a0f8bbe1b62139470ffa0.tar.gz blackbird-op-linux-8bf42225dac865665a5a0f8bbe1b62139470ffa0.zip | |
drm/modes: do not enforce an odd vtotal for interlaced modes
CEA actually specifies an interlaced mode with even vtotal and
supplies a diagram showing how this is supposed to work.
Note that interlaced modes with an even vtotal seem to be a fairly
recent invention. All modelines lore I could dig up with googling says
that vtotal for interlaced modes _needs_ to be odd. But the even
modelines in CEA are not a spec-bug, there's a figure in CEA-861-E
called "Figure 5 Special Interlaced Video Format Timing (Even Vtotal)"
that explains how it's supposed to work. Furthermore intel Bspec
explicitly mentions that both odd and even interlaced vtotal are
supported (VTOTAL register in the south display engine of PCH split
chips).
Acked-by: Adam Jackson <ajax@redhat.com>
Signed-Off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
Signed-off-by: Dave Airlie <airlied@redhat.com>
Diffstat (limited to 'drivers/gpu/drm/drm_modes.c')
| -rw-r--r-- | drivers/gpu/drm/drm_modes.c | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/drivers/gpu/drm/drm_modes.c b/drivers/gpu/drm/drm_modes.c index fb8e46b4e8bc..7ff13bc47ca2 100644 --- a/drivers/gpu/drm/drm_modes.c +++ b/drivers/gpu/drm/drm_modes.c @@ -686,8 +686,6 @@ void drm_mode_set_crtcinfo(struct drm_display_mode *p, int adjust_flags) p->crtc_vsync_end /= 2; p->crtc_vtotal /= 2; } - - p->crtc_vtotal |= 1; } if (p->flags & DRM_MODE_FLAG_DBLSCAN) { |
