Hi all, Somewhat motivated (but only really tangentially) by the dirtyfb discussion with Rob and Thomas I started digging around in the various driver implementations for implicit vs. explicit fencing. There's definitely a huge pile of drivers which don't do any implicit fencing at all - not sure that's good or not. And for some of the drivers with more history I think they don't correctly overwrite implicit fencing when explicit fencing is present. At least I've gotten lost in the mazes before I found positive proof. So this is just the lower hanging stuff, plus a doc patch to hopefully clarify this all better. Comments and review and especially in the case of the msm/vc4 patches, also testing, very much welcome. Thanks, Daniel Daniel Vetter (9): drm/vmwgfx: Remove no-op prepare/cleanup_fb callbacks drm: Move simple_display_pipe prepare_fb helper into gem fb helpers drm/tve200: Use simple_display_pipe prepare_fb helper drm/pl111: Use simple_display_pipe prepare_fb helper drm/mxsfb: Use simple_display_pipe prepare_fb helper drm/atomic: better doc for implicit vs explicit fencing drm/gem-fb-helper: Always do implicit sync drm/vc4: Always obey implicit sync drm/msm: Always obey implicit fencing drivers/gpu/drm/drm_atomic.c | 8 +++++++ drivers/gpu/drm/drm_gem_framebuffer_helper.c | 21 ++++++++++++++++- drivers/gpu/drm/msm/msm_atomic.c | 2 +- drivers/gpu/drm/mxsfb/mxsfb_drv.c | 8 +------ drivers/gpu/drm/pl111/pl111_display.c | 8 +------ drivers/gpu/drm/tinydrm/core/tinydrm-pipe.c | 17 -------------- drivers/gpu/drm/tinydrm/ili9225.c | 2 +- drivers/gpu/drm/tinydrm/mi0283qt.c | 3 ++- drivers/gpu/drm/tinydrm/repaper.c | 2 +- drivers/gpu/drm/tinydrm/st7586.c | 2 +- drivers/gpu/drm/tinydrm/st7735r.c | 2 +- drivers/gpu/drm/tve200/tve200_display.c | 8 +------ drivers/gpu/drm/vc4/vc4_plane.c | 11 +++++---- drivers/gpu/drm/vmwgfx/vmwgfx_ldu.c | 35 ---------------------------- include/drm/drm_gem_framebuffer_helper.h | 3 +++ include/drm/drm_modeset_helper_vtables.h | 5 +++- include/drm/drm_plane.h | 7 +++++- include/drm/drm_simple_kms_helper.h | 3 +++ include/drm/tinydrm/tinydrm.h | 2 -- 19 files changed, 61 insertions(+), 88 deletions(-) -- 2.16.2 _______________________________________________ dri-devel mailing list dri-devel@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/dri-devel