intel-xe.lists.freedesktop.org archive mirror
 help / color / mirror / Atom feed
* [PATCH 0/6] drm: avoid poking at dev->vblank[] directly
@ 2025-11-07 11:04 Jani Nikula
  2025-11-07 11:04 ` [PATCH 1/6] drm/vblank: use drm_crtc_vblank_crtc() in workers Jani Nikula
                   ` (9 more replies)
  0 siblings, 10 replies; 20+ messages in thread
From: Jani Nikula @ 2025-11-07 11:04 UTC (permalink / raw)
  To: dri-devel; +Cc: intel-gfx, intel-xe, jani.nikula, ville.syrjala

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/20251106200000.1455164-1-jani.nikula@intel.com


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


^ permalink raw reply	[flat|nested] 20+ messages in thread

end of thread, other threads:[~2025-11-10 13:42 UTC | newest]

Thread overview: 20+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2025-11-07 11:04 [PATCH 0/6] drm: avoid poking at dev->vblank[] directly Jani Nikula
2025-11-07 11:04 ` [PATCH 1/6] drm/vblank: use drm_crtc_vblank_crtc() in workers Jani Nikula
2025-11-10  9:44   ` Thomas Zimmermann
2025-11-07 11:04 ` [PATCH 2/6] drm/atomic: use drm_crtc_vblank_waitqueue() Jani Nikula
2025-11-10  9:57   ` Thomas Zimmermann
2025-11-10 12:51     ` Jani Nikula
2025-11-10 13:18       ` Thomas Zimmermann
2025-11-10 13:42         ` Jani Nikula
2025-11-07 11:04 ` [PATCH 3/6] drm/msm: " Jani Nikula
2025-11-08 17:00   ` Dmitry Baryshkov
2025-11-07 11:04 ` [PATCH 4/6] drm/tidss: use drm_crtc_vblank_crtc() Jani Nikula
2025-11-08 15:54   ` Jyri Sarha
2025-11-07 11:04 ` [PATCH 5/6] drm/vmwgfx: " Jani Nikula
2025-11-07 16:21   ` Ian Forbes
2025-11-07 11:05 ` [PATCH 6/6] drm/gma500: " Jani Nikula
2025-11-07 13:11   ` Patrik Jakobsson
2025-11-07 11:48 ` ✗ CI.checkpatch: warning for drm: avoid poking at dev->vblank[] directly Patchwork
2025-11-07 11:50 ` ✓ CI.KUnit: success " Patchwork
2025-11-07 12:34 ` ✓ Xe.CI.BAT: " Patchwork
2025-11-09  1:01 ` ✗ Xe.CI.Full: failure " Patchwork

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).