We have accessors for dev->vblank[], and poking at that directly should be reserved for drm_vblank.c. This series along with [1] switches to use the accessors.
I've got a whole another series brewing refactoring drm_vblank.c, but wanted to split this out of the series to keep it manageable. None of the patches here depend on each other, and the refactoring doesn't depend on it, so the patches to drivers can be merged every which way, just let me know. BR, Jani. [1] https://lore.kernel.org/r/[email protected] Jani Nikula (6): drm/vblank: use drm_crtc_vblank_crtc() in workers drm/atomic: use drm_crtc_vblank_waitqueue() drm/msm: use drm_crtc_vblank_waitqueue() drm/tidss: use drm_crtc_vblank_crtc() drm/vmwgfx: use drm_crtc_vblank_crtc() drm/gma500: use drm_crtc_vblank_crtc() drivers/gpu/drm/drm_atomic_helper.c | 4 ++- drivers/gpu/drm/drm_vblank_work.c | 2 +- drivers/gpu/drm/gma500/psb_irq.c | 36 +++++++++++++++-------- drivers/gpu/drm/msm/disp/mdp4/mdp4_crtc.c | 3 +- drivers/gpu/drm/msm/disp/mdp5/mdp5_crtc.c | 3 +- drivers/gpu/drm/tidss/tidss_crtc.c | 3 +- drivers/gpu/drm/vmwgfx/vmwgfx_vkms.c | 6 ++-- 7 files changed, 34 insertions(+), 23 deletions(-) -- 2.47.3
