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.guo@xxxxxxxxxx> > > > > 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.pinchart@xxxxxxxxxxxxxxxx> to the patch. -- Regards, Laurent Pinchart _______________________________________________ dri-devel mailing list dri-devel@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/dri-devel
