Sorry this took so long. The vmwgfx part: Reviewed-by: Sinclair Yeh <syeh@xxxxxxxxxx> I've done some testing and the vmwgfx part looks good. Has Daniel already taken these or should I put them in my next request? Sinclair On Wed, Nov 01, 2017 at 08:29:15PM +0200, Ville Syrjala wrote: > From: Ville Syrjälä <ville.syrjala@xxxxxxxxxxxxxxx> > > While trawling the tree I spotted some issues with the way vmwgfx > uses drm_plane_helper_check_state(). Here's my attempt at fixing it. > Do note that I haven't actually tested the resulting code at all, > but it does build at least. > > And while touching that general area I took up Daniel's suggestion from > long ago that drm_plane_helper_check_state() should be renamed and > relocated to better reflect its status. > > Here's a branch with the entire series: > git://github.com/vsyrjala/linux.git atomic_helper_plane_stuff > > Cc: VMware Graphics <linux-graphics-maintainer@xxxxxxxxxx> > Cc: Sinclair Yeh <syeh@xxxxxxxxxx> > Cc: Thomas Hellstrom <thellstrom@xxxxxxxxxx> > Cc: Daniel Vetter <daniel@xxxxxxxx> > > Ville Syrjälä (5): > drm/vmwgfx: Remove bogus crtc coords vs fb size check > drm/vmwgfx: Use drm_plane_helper_check_state() > drm/vmwgfx: Try to fix plane clipping > drm: Check crtc_state->enable rather than crtc->enabled in > drm_plane_helper_check_state() > drm: Move drm_plane_helper_check_state() into drm_atomic_helper.c > > drivers/gpu/drm/arm/hdlcd_crtc.c | 8 +- > drivers/gpu/drm/arm/malidp_planes.c | 3 +- > drivers/gpu/drm/drm_atomic_helper.c | 95 ++++++++++++++++++++++++ > drivers/gpu/drm/drm_plane_helper.c | 111 +++------------------------- > drivers/gpu/drm/drm_simple_kms_helper.c | 9 ++- > drivers/gpu/drm/i915/intel_display.c | 20 ++--- > drivers/gpu/drm/imx/ipuv3-plane.c | 8 +- > drivers/gpu/drm/mediatek/mtk_drm_plane.c | 8 +- > drivers/gpu/drm/meson/meson_plane.c | 8 +- > drivers/gpu/drm/msm/mdp/mdp5/mdp5_plane.c | 5 +- > drivers/gpu/drm/nouveau/nv50_display.c | 18 +++-- > drivers/gpu/drm/rockchip/rockchip_drm_vop.c | 6 +- > drivers/gpu/drm/tegra/dc.c | 4 +- > drivers/gpu/drm/vmwgfx/vmwgfx_kms.c | 40 ++++------ > drivers/gpu/drm/zte/zx_plane.c | 15 ++-- > include/drm/drm_atomic_helper.h | 7 ++ > include/drm/drm_plane_helper.h | 5 -- > 17 files changed, 187 insertions(+), 183 deletions(-) > > -- > 2.13.6 > _______________________________________________ dri-devel mailing list dri-devel@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/dri-devel
