On Fri, Apr 06, 2018 at 10:38:38AM +0300, Oleksandr Andrushchenko wrote: > Hi, Daniel! > > It seems that this series misses xen-front's > "Use simple_display_pipe prepare_fb helper" change? Hm indeed, I guess I was on an older tree. Will follow up. Care to review the other patches meanwhile? -Daniel > > Thank you, > Oleksandr > > On 04/05/2018 06:44 PM, Daniel Vetter wrote: > > 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(-) > > > -- Daniel Vetter Software Engineer, Intel Corporation http://blog.ffwll.ch _______________________________________________ dri-devel mailing list dri-devel@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/dri-devel
