Re: [PATCH 4/6] drm/tidss: use drm_crtc_vblank_crtc()

["Jyri Sarha" <jyri.sarha@iki.fi>]

[-- Attachment #1: Type: text/plain, Size: 1245 bytes --]

November 7, 2025 at 1:04 PM, "Jani Nikula" <jani.nikula@intel.com mailto:jani.nikula@intel.com?to=%22Jani%20Nikula%22%20%3Cjani.nikula%40intel.com%3E > wrote:


> 
> We have drm_crtc_vblank_crtc() to get the struct drm_vblank_crtc pointer
> for a crtc. Use it instead of poking at dev->vblank[] directly.
> 
> Cc: Jyri Sarha <jyri.sarha@iki.fi>
> Cc: Tomi Valkeinen <tomi.valkeinen@ideasonboard.com>
> Signed-off-by: Jani Nikula <jani.nikula@intel.com>
> 
Acked-by: Jyri Sarha <jyri.sarha@iki.fi mailto:jyri.sarha@iki.fi >

Thanks,.
Jyri
---
 drivers/gpu/drm/tidss/tidss_crtc.c | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/drivers/gpu/drm/tidss/tidss_crtc.c b/drivers/gpu/drm/tidss/tidss_crtc.c
index 411b1a25e29c..8f81eb560b9e 100644
--- a/drivers/gpu/drm/tidss/tidss_crtc.c
+++ b/drivers/gpu/drm/tidss/tidss_crtc.c
@@ -248,8 +248,7 @@ static void tidss_crtc_atomic_enable(struct drm_crtc *crtc,
 dispc_vp_enable(tidss->dispc, tcrtc->hw_videoport);
 
 if (crtc->state->event) {
- unsigned int pipe = drm_crtc_index(crtc);
- struct drm_vblank_crtc *vblank = &ddev->vblank[pipe];
+ struct drm_vblank_crtc *vblank = drm_crtc_vblank_crtc(crtc);
 
 vblank->time = ktime_get();
 
-- 
2.47.3

[-- Attachment #2: Type: text/html, Size: 1629 bytes --]