On Mon, Oct 21, 2024 at 04:54:16PM +0300, Jani Nikula wrote: > Switch to using the new display->platform.<platform> members for > platform identification in display code. > > Signed-off-by: Jani Nikula <[email protected]> > --- > drivers/gpu/drm/i915/display/intel_vblank.c | 11 +++++------ > 1 file changed, 5 insertions(+), 6 deletions(-) > > diff --git a/drivers/gpu/drm/i915/display/intel_vblank.c > b/drivers/gpu/drm/i915/display/intel_vblank.c > index d18b8292be49..a95fb3349eba 100644 > --- a/drivers/gpu/drm/i915/display/intel_vblank.c > +++ b/drivers/gpu/drm/i915/display/intel_vblank.c > @@ -195,7 +195,6 @@ static u32 > __intel_get_crtc_scanline_from_timestamp(struct intel_crtc *crtc) > 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); > > /* > * The scanline counter increments at the leading edge of hsync. > @@ -225,7 +224,7 @@ int intel_crtc_scanline_offset(const struct > intel_crtc_state *crtc_state) > */ > if (DISPLAY_VER(display) == 2) > return -1; > - else if (HAS_DDI(i915) && intel_crtc_has_type(crtc_state, > INTEL_OUTPUT_HDMI)) > + else if (HAS_DDI(display) && intel_crtc_has_type(crtc_state, > INTEL_OUTPUT_HDMI))
I believe this first chunk should be in a separate patch. but if so Reviewed-by: Rodrigo Vivi <[email protected]> in both anyways... > return 2; > else > return 1; > @@ -327,14 +326,13 @@ static bool i915_get_crtc_scanoutpos(struct drm_crtc > *_crtc, > const struct drm_display_mode *mode) > { > struct intel_display *display = to_intel_display(_crtc->dev); > - struct drm_i915_private *dev_priv = to_i915(display->drm); > struct intel_crtc *crtc = to_intel_crtc(_crtc); > enum pipe pipe = crtc->pipe; > int position; > int vbl_start, vbl_end, hsync_start, htotal, vtotal; > unsigned long irqflags; > bool use_scanline_counter = DISPLAY_VER(display) >= 5 || > - IS_G4X(dev_priv) || DISPLAY_VER(display) == 2 || > + display->platform.g4x || DISPLAY_VER(display) == 2 || > crtc->mode_flags & I915_MODE_FLAG_USE_SCANLINE_COUNTER; > > if (drm_WARN_ON(display->drm, !mode->crtc_clock)) { > @@ -603,14 +601,15 @@ void intel_vblank_evade_init(const struct > intel_crtc_state *old_crtc_state, > const struct intel_crtc_state *new_crtc_state, > struct intel_vblank_evade_ctx *evade) > { > + struct intel_display *display = to_intel_display(new_crtc_state); > struct intel_crtc *crtc = to_intel_crtc(new_crtc_state->uapi.crtc); > - struct drm_i915_private *i915 = to_i915(crtc->base.dev); > const struct intel_crtc_state *crtc_state; > const struct drm_display_mode *adjusted_mode; > > evade->crtc = crtc; > > - evade->need_vlv_dsi_wa = (IS_VALLEYVIEW(i915) || IS_CHERRYVIEW(i915)) && > + evade->need_vlv_dsi_wa = (display->platform.valleyview || > + display->platform.cherryview) && > intel_crtc_has_type(new_crtc_state, INTEL_OUTPUT_DSI); > > /* > -- > 2.39.5 >
