On Wed, Feb 08, 2017 at 08:10:04PM +0200, Laurent Pinchart wrote: > Hi Tomi, > > On Wednesday 08 Feb 2017 13:30:51 Tomi Valkeinen wrote: > > On 07/02/17 11:16, Shawn Guo wrote: > > > From: Shawn Guo <shawn....@linaro.org> > > > > > > The vblank is mostly CRTC specific and implemented as part of CRTC > > > driver. The first patch adds 3 vblank core<->driver hooks into struct > > > drm_crtc_funcs, and plug them into core by adding wrapper functions for > > > vblank handling code. We effectively make the .get_vblank_counter hook > > > optional by providing drm_vblank_no_hw_counter() as the default fallback > > > in the wrapper function. > > > > > > Patch #2 and #3 unexport function drm_vblank_no_hw_counter() by cleaning > > > up its use, since it's already the default implememention for > > > .get_vblank_counter hook anyway. > > > > > > The rest of the series is trying to do a massive conversion to the new > > > hooks for DRIVER_MODESET drivers. But it only handles low-hanging > > > fruit, and leaves out the ones that need a bit surgery, like gma500, > > > i915, msm etc. Most of conversion get done by simply moving code and > > > making functions static, but imx and rockchip are great examples showing > > > how driver code can be cleaned up with these new hooks. > > > > > > The series is generated against branch drm-next. > > > > Thanks for the series. I've attached a patch for omapdrm, in case you're > > sending v4 and want to include it in the series. > > You can add my > > Reviewed-by: Laurent Pinchart <laurent.pinch...@ideasonboard.com>
To simplify the flow a bit I've pushed this into drm-misc-next, with Tomi's ack. -Daniel -- Daniel Vetter Software Engineer, Intel Corporation http://blog.ffwll.ch _______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel
