Vertical blanking needs to be halved on interlace modes. WBDELAYCOUNT
was calculated without such halving, resulting in WBUNCOMPLETE errors.

Signed-off-by: Tomi Valkeinen <[email protected]>
Acked-by: Benoit Parrot <[email protected]>
---
 drivers/gpu/drm/omapdrm/dss/dispc.c | 14 +++++++++-----
 1 file changed, 9 insertions(+), 5 deletions(-)

diff --git a/drivers/gpu/drm/omapdrm/dss/dispc.c 
b/drivers/gpu/drm/omapdrm/dss/dispc.c
index 7398039954a5..000a3d4a27bf 100644
--- a/drivers/gpu/drm/omapdrm/dss/dispc.c
+++ b/drivers/gpu/drm/omapdrm/dss/dispc.c
@@ -2814,14 +2814,18 @@ int dispc_wb_setup(struct dispc_device *dispc,
                /* WBDELAYCOUNT */
                REG_FLD_MOD(dispc, DISPC_OVL_ATTRIBUTES2(plane), 0, 7, 0);
        } else {
-               int wbdelay;
+               u32 wbdelay;
 
                if (channel_in == DSS_WB_TV_MGR)
-                       wbdelay = min(vm->vsync_len + vm->vback_porch,
-                               (u32)255);
+                       wbdelay = vm->vsync_len + vm->vback_porch;
                else
-                       wbdelay = min(vm->vfront_porch +
-                               vm->vsync_len + vm->vback_porch, (u32)255);
+                       wbdelay = vm->vfront_porch + vm->vsync_len +
+                               vm->vback_porch;
+
+               if (vm->flags & DISPLAY_FLAGS_INTERLACED)
+                       wbdelay /= 2;
+
+               wbdelay = min(wbdelay, 255u);
 
                /* WBDELAYCOUNT */
                REG_FLD_MOD(dispc, DISPC_OVL_ATTRIBUTES2(plane), wbdelay, 7, 0);
-- 
Texas Instruments Finland Oy, Porkkalankatu 22, 00180 Helsinki.
Y-tunnus/Business ID: 0615521-4. Kotipaikka/Domicile: Helsinki

_______________________________________________
dri-devel mailing list
[email protected]
https://lists.freedesktop.org/mailman/listinfo/dri-devel

Reply via email to