[PATCH v2 0/5] omapdrm: fences and zpos

Laurent Pinchart laurent.pinchart at ideasonboard.com
Sat Apr 15 09:16:27 UTC 2017


Hello,

This patch series contains two completely unrelated features that just happen
to have been developed one right after the other.

Patches 1/5 to 3/5 implement explicit fences support. The first patch fixes
event handling in the omapdrm driver as required by the atomic commit helper
(and the DRM atomic update API). The second patch replace the hand-rolled
atomic commit handler with the DRM core atomic commit helper, bringing fences
support as a bonus. The third patch then removes the omapdrm custom sync
support that has no user in the mainline kernel.

Patches 4/5 and 5/5 clean up zpos handling by switching to the standard zpos
property implemented in the DRM core. The omapdrm-specific zorder property is
kept for backward compatibility as an alias and should be removed later after
a long enough grace period for userspace to be updated.

Compared to v1, the patches have been rebased on top of Dave's drm-next branch
which now contains all the dependencies.

On the testing side, I've ran kmstest --flip for 250 frames in a loop for half
an hour, with and without CPU load (with 'stress -c 4'), on both PandaBoard
and AM572x EVM, and couldn't reproduce any of the issues reported against v1.

As fence support is implemented completely inside the atomic commit helper I
have only tested it lightly using the sw-sync driver. No issue has been
noticed.

Laurent Pinchart (5):
  drm: omapdrm: Handle events when enabling/disabling CRTCs
  drm: omapdrm: Use DRM core's atomic commit helper
  drm: omapdrm: Remove legacy buffer synchronization support
  drm: omapdrm: Store the Z order in the plane state zpos field
  drm: omapdrm: Add zpos property

 drivers/gpu/drm/omapdrm/omap_crtc.c  |  30 +++--
 drivers/gpu/drm/omapdrm/omap_drv.c   | 162 ++++----------------------
 drivers/gpu/drm/omapdrm/omap_drv.h   |  14 +--
 drivers/gpu/drm/omapdrm/omap_gem.c   | 214 -----------------------------------
 drivers/gpu/drm/omapdrm/omap_plane.c |  74 ++----------
 include/uapi/drm/omap_drm.h          |   4 +-
 6 files changed, 55 insertions(+), 443 deletions(-)

-- 
Regards,

Laurent Pinchart



More information about the dri-devel mailing list