niansa-misc/linux
1
0
Fork 0
mirror of synced 2025-03-06 20:59:54 +01:00
Code Issues Projects Releases Packages Wiki Activity

drm/i915/vblank: convert to struct intel_display

Browse source 
Going forward, struct intel_display shall replace struct
drm_i915_private as the main display device data pointer type. Convert
intel_vblank.[ch] to struct intel_display.

Some stragglers are left behind where needed.

Reviewed-by: Rodrigo Vivi <rodrigo.vivi@intel.com>
Link: https://patchwork.freedesktop.org/patch/msgid/40430651a45ddd9e350a1fd7938fe4054492f6ea.1724342644.git.jani.nikula@intel.com
Signed-off-by: Jani Nikula <jani.nikula@intel.com>
This commit is contained in:
Jani Nikula 2024-08-22 19:04:51 +03:00
parent 58cfea648f
commit aa451ae76f
1 changed files with 50 additions and 47 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

97
drivers/gpu/drm/i915/display/intel_vblank.c
View file

@ -67,7 +67,7 @@
*/ */
u32 i915_get_vblank_counter(struct drm_crtc *crtc) u32 i915_get_vblank_counter(struct drm_crtc *crtc)
{ {
struct drm_i915_private *dev_priv = to_i915(crtc->dev); struct intel_display *display = to_intel_display(crtc->dev);
struct drm_vblank_crtc *vblank = drm_crtc_vblank_crtc(crtc); struct drm_vblank_crtc *vblank = drm_crtc_vblank_crtc(crtc);
const struct drm_display_mode *mode = &vblank->hwmode; const struct drm_display_mode *mode = &vblank->hwmode;
enum pipe pipe = to_intel_crtc(crtc)->pipe; enum pipe pipe = to_intel_crtc(crtc)->pipe;
@ -103,8 +103,8 @@ u32 i915_get_vblank_counter(struct drm_crtc *crtc)
* we get a low value that's stable across two reads of the high * we get a low value that's stable across two reads of the high
* register. * register.
*/ */
frame = intel_de_read64_2x32(dev_priv, PIPEFRAMEPIXEL(dev_priv, pipe), frame = intel_de_read64_2x32(display, PIPEFRAMEPIXEL(display, pipe),
PIPEFRAME(dev_priv, pipe)); PIPEFRAME(display, pipe));
pixel = frame & PIPE_PIXEL_MASK; pixel = frame & PIPE_PIXEL_MASK;
frame = (frame >> PIPE_FRAME_LOW_SHIFT) & 0xffffff; frame = (frame >> PIPE_FRAME_LOW_SHIFT) & 0xffffff;
@ -119,19 +119,19 @@ u32 i915_get_vblank_counter(struct drm_crtc *crtc)
u32 g4x_get_vblank_counter(struct drm_crtc *crtc) u32 g4x_get_vblank_counter(struct drm_crtc *crtc)
{ {
struct drm_i915_private *dev_priv = to_i915(crtc->dev); struct intel_display *display = to_intel_display(crtc->dev);
struct drm_vblank_crtc *vblank = drm_crtc_vblank_crtc(crtc); struct drm_vblank_crtc *vblank = drm_crtc_vblank_crtc(crtc);
enum pipe pipe = to_intel_crtc(crtc)->pipe; enum pipe pipe = to_intel_crtc(crtc)->pipe;
if (!vblank->max_vblank_count) if (!vblank->max_vblank_count)
return 0; return 0;
return intel_de_read(dev_priv, PIPE_FRMCOUNT_G4X(dev_priv, pipe)); return intel_de_read(display, PIPE_FRMCOUNT_G4X(display, pipe));
} }
static u32 intel_crtc_scanlines_since_frame_timestamp(struct intel_crtc *crtc) static u32 intel_crtc_scanlines_since_frame_timestamp(struct intel_crtc *crtc)
{ {
struct drm_i915_private *dev_priv = to_i915(crtc->base.dev); struct intel_display *display = to_intel_display(crtc);
struct drm_vblank_crtc *vblank = drm_crtc_vblank_crtc(&crtc->base); struct drm_vblank_crtc *vblank = drm_crtc_vblank_crtc(&crtc->base);
const struct drm_display_mode *mode = &vblank->hwmode; const struct drm_display_mode *mode = &vblank->hwmode;
u32 htotal = mode->crtc_htotal; u32 htotal = mode->crtc_htotal;
@ -150,16 +150,16 @@ static u32 intel_crtc_scanlines_since_frame_timestamp(struct intel_crtc *crtc)
* pipe frame time stamp. The time stamp value * pipe frame time stamp. The time stamp value
* is sampled at every start of vertical blank. * is sampled at every start of vertical blank.
*/ */
scan_prev_time = intel_de_read_fw(dev_priv, scan_prev_time = intel_de_read_fw(display,
PIPE_FRMTMSTMP(crtc->pipe)); PIPE_FRMTMSTMP(crtc->pipe));
/* /*
* The TIMESTAMP_CTR register has the current * The TIMESTAMP_CTR register has the current
* time stamp value. * time stamp value.
*/ */
scan_curr_time = intel_de_read_fw(dev_priv, IVB_TIMESTAMP_CTR); scan_curr_time = intel_de_read_fw(display, IVB_TIMESTAMP_CTR);
scan_post_time = intel_de_read_fw(dev_priv, scan_post_time = intel_de_read_fw(display,
PIPE_FRMTMSTMP(crtc->pipe)); PIPE_FRMTMSTMP(crtc->pipe));
} while (scan_post_time != scan_prev_time); } while (scan_post_time != scan_prev_time);
@ -192,6 +192,7 @@ static u32 __intel_get_crtc_scanline_from_timestamp(struct intel_crtc *crtc)
static int intel_crtc_scanline_offset(const struct intel_crtc_state *crtc_state) static int intel_crtc_scanline_offset(const struct intel_crtc_state *crtc_state)
{ {
struct intel_display *display = to_intel_display(crtc_state);
struct drm_i915_private *i915 = to_i915(crtc_state->uapi.crtc->dev); struct drm_i915_private *i915 = to_i915(crtc_state->uapi.crtc->dev);
/* /*
@ -220,7 +221,7 @@ static int intel_crtc_scanline_offset(const struct intel_crtc_state *crtc_state)
* However if queried just before the start of vblank we'll get an * However if queried just before the start of vblank we'll get an
* answer that's slightly in the future. * answer that's slightly in the future.
*/ */
if (DISPLAY_VER(i915) == 2) if (DISPLAY_VER(display) == 2)
return -1; return -1;
else if (HAS_DDI(i915) && intel_crtc_has_type(crtc_state, INTEL_OUTPUT_HDMI)) else if (HAS_DDI(i915) && intel_crtc_has_type(crtc_state, INTEL_OUTPUT_HDMI))
return 2; return 2;
@ -234,8 +235,7 @@ static int intel_crtc_scanline_offset(const struct intel_crtc_state *crtc_state)
*/ */
static int __intel_get_crtc_scanline(struct intel_crtc *crtc) static int __intel_get_crtc_scanline(struct intel_crtc *crtc)
{ {
struct drm_device *dev = crtc->base.dev; struct intel_display *display = to_intel_display(crtc);
struct drm_i915_private *dev_priv = to_i915(dev);
struct drm_vblank_crtc *vblank = drm_crtc_vblank_crtc(&crtc->base); struct drm_vblank_crtc *vblank = drm_crtc_vblank_crtc(&crtc->base);
const struct drm_display_mode *mode = &vblank->hwmode; const struct drm_display_mode *mode = &vblank->hwmode;
enum pipe pipe = crtc->pipe; enum pipe pipe = crtc->pipe;
@ -249,7 +249,7 @@ static int __intel_get_crtc_scanline(struct intel_crtc *crtc)
vtotal = intel_mode_vtotal(mode); vtotal = intel_mode_vtotal(mode);
position = intel_de_read_fw(dev_priv, PIPEDSL(dev_priv, pipe)) & PIPEDSL_LINE_MASK; position = intel_de_read_fw(display, PIPEDSL(display, pipe)) & PIPEDSL_LINE_MASK;
/* /*
* On HSW, the DSL reg (0x70000) appears to return 0 if we * On HSW, the DSL reg (0x70000) appears to return 0 if we
@ -263,13 +263,13 @@ static int __intel_get_crtc_scanline(struct intel_crtc *crtc)
* problem. We may need to extend this to include other platforms, * problem. We may need to extend this to include other platforms,
* but so far testing only shows the problem on HSW. * but so far testing only shows the problem on HSW.
*/ */
if (HAS_DDI(dev_priv) && !position) { if (HAS_DDI(display) && !position) {
int i, temp; int i, temp;
for (i = 0; i < 100; i++) { for (i = 0; i < 100; i++) {
udelay(1); udelay(1);
temp = intel_de_read_fw(dev_priv, temp = intel_de_read_fw(display,
PIPEDSL(dev_priv, pipe)) & PIPEDSL_LINE_MASK; PIPEDSL(display, pipe)) & PIPEDSL_LINE_MASK;
if (temp != position) { if (temp != position) {
position = temp; position = temp;
break; break;
@ -304,23 +304,25 @@ int intel_crtc_scanline_to_hw(struct intel_crtc *crtc, int scanline)
* otherwise they may hang. * otherwise they may hang.
*/ */
#ifdef I915 #ifdef I915
static void intel_vblank_section_enter(struct drm_i915_private *i915) static void intel_vblank_section_enter(struct intel_display *display)
__acquires(i915->uncore.lock) __acquires(i915->uncore.lock)
{ {
struct drm_i915_private *i915 = to_i915(display->drm);
spin_lock(&i915->uncore.lock); spin_lock(&i915->uncore.lock);
} }
static void intel_vblank_section_exit(struct drm_i915_private *i915) static void intel_vblank_section_exit(struct intel_display *display)
__releases(i915->uncore.lock) __releases(i915->uncore.lock)
{ {
struct drm_i915_private *i915 = to_i915(display->drm);
spin_unlock(&i915->uncore.lock); spin_unlock(&i915->uncore.lock);
} }
#else #else
static void intel_vblank_section_enter(struct drm_i915_private *i915) static void intel_vblank_section_enter(struct intel_display *display)
{ {
} }
static void intel_vblank_section_exit(struct drm_i915_private *i915) static void intel_vblank_section_exit(struct intel_display *display)
{ {
} }
#endif #endif
@ -331,19 +333,19 @@ static bool i915_get_crtc_scanoutpos(struct drm_crtc *_crtc,
ktime_t *stime, ktime_t *etime, ktime_t *stime, ktime_t *etime,
const struct drm_display_mode *mode) const struct drm_display_mode *mode)
{ {
struct drm_device *dev = _crtc->dev; struct intel_display *display = to_intel_display(_crtc->dev);
struct drm_i915_private *dev_priv = to_i915(dev); struct drm_i915_private *dev_priv = to_i915(display->drm);
struct intel_crtc *crtc = to_intel_crtc(_crtc); struct intel_crtc *crtc = to_intel_crtc(_crtc);
enum pipe pipe = crtc->pipe; enum pipe pipe = crtc->pipe;
int position; int position;
int vbl_start, vbl_end, hsync_start, htotal, vtotal; int vbl_start, vbl_end, hsync_start, htotal, vtotal;
unsigned long irqflags; unsigned long irqflags;
bool use_scanline_counter = DISPLAY_VER(dev_priv) >= 5 || bool use_scanline_counter = DISPLAY_VER(display) >= 5 ||
IS_G4X(dev_priv) || DISPLAY_VER(dev_priv) == 2 || IS_G4X(dev_priv) || DISPLAY_VER(display) == 2 ||
crtc->mode_flags & I915_MODE_FLAG_USE_SCANLINE_COUNTER; crtc->mode_flags & I915_MODE_FLAG_USE_SCANLINE_COUNTER;
if (drm_WARN_ON(&dev_priv->drm, !mode->crtc_clock)) { if (drm_WARN_ON(display->drm, !mode->crtc_clock)) {
drm_dbg(&dev_priv->drm, drm_dbg(display->drm,
"trying to get scanoutpos for disabled pipe %c\n", "trying to get scanoutpos for disabled pipe %c\n",
pipe_name(pipe)); pipe_name(pipe));
return false; return false;
@ -361,7 +363,7 @@ static bool i915_get_crtc_scanoutpos(struct drm_crtc *_crtc,
* preemption disabled, so the following code must not block. * preemption disabled, so the following code must not block.
*/ */
local_irq_save(irqflags); local_irq_save(irqflags);
intel_vblank_section_enter(dev_priv); intel_vblank_section_enter(display);
/* preempt_disable_rt() should go right here in PREEMPT_RT patchset. */ /* preempt_disable_rt() should go right here in PREEMPT_RT patchset. */
@ -393,7 +395,7 @@ static bool i915_get_crtc_scanoutpos(struct drm_crtc *_crtc,
* We can split this into vertical and horizontal * We can split this into vertical and horizontal
* scanout position. * scanout position.
*/ */
position = (intel_de_read_fw(dev_priv, PIPEFRAMEPIXEL(dev_priv, pipe)) & PIPE_PIXEL_MASK) >> PIPE_PIXEL_SHIFT; position = (intel_de_read_fw(display, PIPEFRAMEPIXEL(display, pipe)) & PIPE_PIXEL_MASK) >> PIPE_PIXEL_SHIFT;
/* convert to pixel counts */ /* convert to pixel counts */
vbl_start *= htotal; vbl_start *= htotal;
@ -429,7 +431,7 @@ static bool i915_get_crtc_scanoutpos(struct drm_crtc *_crtc,
/* preempt_enable_rt() should go right here in PREEMPT_RT patchset. */ /* preempt_enable_rt() should go right here in PREEMPT_RT patchset. */
intel_vblank_section_exit(dev_priv); intel_vblank_section_exit(display);
local_irq_restore(irqflags); local_irq_restore(irqflags);
/* /*
@ -464,42 +466,42 @@ bool intel_crtc_get_vblank_timestamp(struct drm_crtc *crtc, int *max_error,
int intel_get_crtc_scanline(struct intel_crtc *crtc) int intel_get_crtc_scanline(struct intel_crtc *crtc)
{ {
struct drm_i915_private *dev_priv = to_i915(crtc->base.dev); struct intel_display *display = to_intel_display(crtc);
unsigned long irqflags; unsigned long irqflags;
int position; int position;
local_irq_save(irqflags); local_irq_save(irqflags);
intel_vblank_section_enter(dev_priv); intel_vblank_section_enter(display);
position = __intel_get_crtc_scanline(crtc); position = __intel_get_crtc_scanline(crtc);
intel_vblank_section_exit(dev_priv); intel_vblank_section_exit(display);
local_irq_restore(irqflags); local_irq_restore(irqflags);
return position; return position;
} }
static bool pipe_scanline_is_moving(struct drm_i915_private *dev_priv, static bool pipe_scanline_is_moving(struct intel_display *display,
enum pipe pipe) enum pipe pipe)
{ {
i915_reg_t reg = PIPEDSL(dev_priv, pipe); i915_reg_t reg = PIPEDSL(display, pipe);
u32 line1, line2; u32 line1, line2;
line1 = intel_de_read(dev_priv, reg) & PIPEDSL_LINE_MASK; line1 = intel_de_read(display, reg) & PIPEDSL_LINE_MASK;
msleep(5); msleep(5);
line2 = intel_de_read(dev_priv, reg) & PIPEDSL_LINE_MASK; line2 = intel_de_read(display, reg) & PIPEDSL_LINE_MASK;
return line1 != line2; return line1 != line2;
} }
static void wait_for_pipe_scanline_moving(struct intel_crtc *crtc, bool state) static void wait_for_pipe_scanline_moving(struct intel_crtc *crtc, bool state)
{ {
struct drm_i915_private *dev_priv = to_i915(crtc->base.dev); struct intel_display *display = to_intel_display(crtc);
enum pipe pipe = crtc->pipe; enum pipe pipe = crtc->pipe;
/* Wait for the display line to settle/start moving */ /* Wait for the display line to settle/start moving */
if (wait_for(pipe_scanline_is_moving(dev_priv, pipe) == state, 100)) if (wait_for(pipe_scanline_is_moving(display, pipe) == state, 100))
drm_err(&dev_priv->drm, drm_err(display->drm,
"pipe %c scanline %s wait timed out\n", "pipe %c scanline %s wait timed out\n",
pipe_name(pipe), str_on_off(state)); pipe_name(pipe), str_on_off(state));
} }
@ -517,8 +519,8 @@ void intel_wait_for_pipe_scanline_moving(struct intel_crtc *crtc)
void intel_crtc_update_active_timings(const struct intel_crtc_state *crtc_state, void intel_crtc_update_active_timings(const struct intel_crtc_state *crtc_state,
bool vrr_enable) bool vrr_enable)
{ {
struct intel_display *display = to_intel_display(crtc_state);
struct intel_crtc *crtc = to_intel_crtc(crtc_state->uapi.crtc); struct intel_crtc *crtc = to_intel_crtc(crtc_state->uapi.crtc);
struct drm_i915_private *i915 = to_i915(crtc->base.dev);
u8 mode_flags = crtc_state->mode_flags; u8 mode_flags = crtc_state->mode_flags;
struct drm_display_mode adjusted_mode; struct drm_display_mode adjusted_mode;
int vmax_vblank_start = 0; int vmax_vblank_start = 0;
@ -527,7 +529,8 @@ void intel_crtc_update_active_timings(const struct intel_crtc_state *crtc_state,
drm_mode_init(&adjusted_mode, &crtc_state->hw.adjusted_mode); drm_mode_init(&adjusted_mode, &crtc_state->hw.adjusted_mode);
if (vrr_enable) { if (vrr_enable) {
drm_WARN_ON(&i915->drm, (mode_flags & I915_MODE_FLAG_VRR) == 0); drm_WARN_ON(display->drm,
(mode_flags & I915_MODE_FLAG_VRR) == 0);
adjusted_mode.crtc_vtotal = crtc_state->vrr.vmax; adjusted_mode.crtc_vtotal = crtc_state->vrr.vmax;
adjusted_mode.crtc_vblank_end = crtc_state->vrr.vmax; adjusted_mode.crtc_vblank_end = crtc_state->vrr.vmax;
@ -549,8 +552,8 @@ void intel_crtc_update_active_timings(const struct intel_crtc_state *crtc_state,
* __intel_get_crtc_scanline()) with vblank_time_lock? * __intel_get_crtc_scanline()) with vblank_time_lock?
* Need to audit everything to make sure it's safe. * Need to audit everything to make sure it's safe.
*/ */
spin_lock_irqsave(&i915->drm.vblank_time_lock, irqflags); spin_lock_irqsave(&display->drm->vblank_time_lock, irqflags);
intel_vblank_section_enter(i915); intel_vblank_section_enter(display);
drm_calc_timestamping_constants(&crtc->base, &adjusted_mode); drm_calc_timestamping_constants(&crtc->base, &adjusted_mode);
@ -559,8 +562,8 @@ void intel_crtc_update_active_timings(const struct intel_crtc_state *crtc_state,
crtc->mode_flags = mode_flags; crtc->mode_flags = mode_flags;
crtc->scanline_offset = intel_crtc_scanline_offset(crtc_state); crtc->scanline_offset = intel_crtc_scanline_offset(crtc_state);
intel_vblank_section_exit(i915); intel_vblank_section_exit(display);
spin_unlock_irqrestore(&i915->drm.vblank_time_lock, irqflags); spin_unlock_irqrestore(&display->drm->vblank_time_lock, irqflags);
} }
int intel_mode_vdisplay(const struct drm_display_mode *mode) int intel_mode_vdisplay(const struct drm_display_mode *mode)
@ -666,7 +669,7 @@ void intel_vblank_evade_init(const struct intel_crtc_state *old_crtc_state,
int intel_vblank_evade(struct intel_vblank_evade_ctx *evade) int intel_vblank_evade(struct intel_vblank_evade_ctx *evade)
{ {
struct intel_crtc *crtc = evade->crtc; struct intel_crtc *crtc = evade->crtc;
struct drm_i915_private *i915 = to_i915(crtc->base.dev); struct intel_display *display = to_intel_display(crtc);
long timeout = msecs_to_jiffies_timeout(1); long timeout = msecs_to_jiffies_timeout(1);
wait_queue_head_t *wq = drm_crtc_vblank_waitqueue(&crtc->base); wait_queue_head_t *wq = drm_crtc_vblank_waitqueue(&crtc->base);
DEFINE_WAIT(wait); DEFINE_WAIT(wait);
@ -688,7 +691,7 @@ int intel_vblank_evade(struct intel_vblank_evade_ctx *evade)
break; break;
if (!timeout) { if (!timeout) {
drm_err(&i915->drm, drm_err(display->drm,
"Potential atomic update failure on pipe %c\n", "Potential atomic update failure on pipe %c\n",
pipe_name(crtc->pipe)); pipe_name(crtc->pipe));
break; break;