]> git.ipfire.org Git - thirdparty/linux.git/commitdiff
drm/i915/display: pass struct intel_display to PCH macros
authorJani Nikula <jani.nikula@intel.com>
Thu, 17 Apr 2025 09:10:36 +0000 (12:10 +0300)
committerJani Nikula <jani.nikula@intel.com>
Tue, 22 Apr 2025 12:46:29 +0000 (15:46 +0300)
Now that INTEL_PCH_TYPE() and HAS_PCH_*() macros are under display, and
accept a struct intel_display pointer, use that instead of struct
drm_i915_private pointer in display code.

This is done naively by running:

$ sed -i 's/\(INTEL_PCH_TYPE\|HAS_PCH_[A-Z0-9_-]*\)([^)]*)/\1(display)/g' \
  $(find drivers/gpu/drm/i915/display -name "*.c")

and fixing the fallout, i.e. removing unused local i915 variables and
adding display variables where needed.

v2: Rebase

Reviewed-by: Chaitanya Kumar Borah <chaitanya.kumar.borah@intel.com>
Link: https://lore.kernel.org/r/999f4d7b8ed11739b1c5ec8d6408fc39d5e3776b.1744880985.git.jani.nikula@intel.com
Signed-off-by: Jani Nikula <jani.nikula@intel.com>
28 files changed:
drivers/gpu/drm/i915/display/g4x_dp.c
drivers/gpu/drm/i915/display/g4x_hdmi.c
drivers/gpu/drm/i915/display/i9xx_wm.c
drivers/gpu/drm/i915/display/intel_audio.c
drivers/gpu/drm/i915/display/intel_backlight.c
drivers/gpu/drm/i915/display/intel_bios.c
drivers/gpu/drm/i915/display/intel_cdclk.c
drivers/gpu/drm/i915/display/intel_crt.c
drivers/gpu/drm/i915/display/intel_ddi.c
drivers/gpu/drm/i915/display/intel_display.c
drivers/gpu/drm/i915/display/intel_display_debugfs.c
drivers/gpu/drm/i915/display/intel_display_device.c
drivers/gpu/drm/i915/display/intel_display_irq.c
drivers/gpu/drm/i915/display/intel_display_power.c
drivers/gpu/drm/i915/display/intel_dp.c
drivers/gpu/drm/i915/display/intel_dp_aux.c
drivers/gpu/drm/i915/display/intel_dpll.c
drivers/gpu/drm/i915/display/intel_dpll_mgr.c
drivers/gpu/drm/i915/display/intel_fdi.c
drivers/gpu/drm/i915/display/intel_fifo_underrun.c
drivers/gpu/drm/i915/display/intel_gmbus.c
drivers/gpu/drm/i915/display/intel_hdmi.c
drivers/gpu/drm/i915/display/intel_hotplug_irq.c
drivers/gpu/drm/i915/display/intel_lvds.c
drivers/gpu/drm/i915/display/intel_pch_display.c
drivers/gpu/drm/i915/display/intel_pch_refclk.c
drivers/gpu/drm/i915/display/intel_pps.c
drivers/gpu/drm/i915/display/intel_sdvo.c

index 18e51799d2a6773a6f2d373eb234feafd30ab9ab..9a8c42d6aea908d83ad428d1df52e77f7b47b0fd 100644 (file)
@@ -59,14 +59,13 @@ static void g4x_dp_set_clock(struct intel_encoder *encoder,
                             struct intel_crtc_state *pipe_config)
 {
        struct intel_display *display = to_intel_display(encoder);
-       struct drm_i915_private *dev_priv = to_i915(encoder->base.dev);
        const struct dpll *divisor = NULL;
        int i, count = 0;
 
        if (display->platform.g4x) {
                divisor = g4x_dpll;
                count = ARRAY_SIZE(g4x_dpll);
-       } else if (HAS_PCH_SPLIT(dev_priv)) {
+       } else if (HAS_PCH_SPLIT(display)) {
                divisor = pch_dpll;
                count = ARRAY_SIZE(pch_dpll);
        } else if (display->platform.cherryview) {
@@ -92,7 +91,6 @@ static void intel_dp_prepare(struct intel_encoder *encoder,
                             const struct intel_crtc_state *pipe_config)
 {
        struct intel_display *display = to_intel_display(encoder);
-       struct drm_i915_private *dev_priv = to_i915(encoder->base.dev);
        struct intel_dp *intel_dp = enc_to_intel_dp(encoder);
        enum port port = encoder->port;
        struct intel_crtc *crtc = to_intel_crtc(pipe_config->uapi.crtc);
@@ -140,7 +138,7 @@ static void intel_dp_prepare(struct intel_encoder *encoder,
                        intel_dp->DP |= DP_ENHANCED_FRAMING;
 
                intel_dp->DP |= DP_PIPE_SEL_IVB(crtc->pipe);
-       } else if (HAS_PCH_CPT(dev_priv) && port != PORT_A) {
+       } else if (HAS_PCH_CPT(display) && port != PORT_A) {
                intel_dp->DP |= DP_LINK_TRAIN_OFF_CPT;
 
                intel_de_rmw(display, TRANS_DP_CTL(crtc->pipe),
@@ -276,7 +274,6 @@ bool g4x_dp_port_enabled(struct intel_display *display,
                         i915_reg_t dp_reg, enum port port,
                         enum pipe *pipe)
 {
-       struct drm_i915_private *dev_priv = to_i915(display->drm);
        bool ret;
        u32 val;
 
@@ -287,7 +284,7 @@ bool g4x_dp_port_enabled(struct intel_display *display,
        /* asserts want to know the pipe even if the port is disabled */
        if (display->platform.ivybridge && port == PORT_A)
                *pipe = (val & DP_PIPE_SEL_MASK_IVB) >> DP_PIPE_SEL_SHIFT_IVB;
-       else if (HAS_PCH_CPT(dev_priv) && port != PORT_A)
+       else if (HAS_PCH_CPT(display) && port != PORT_A)
                ret &= cpt_dp_port_selected(display, port, pipe);
        else if (display->platform.cherryview)
                *pipe = (val & DP_PIPE_SEL_MASK_CHV) >> DP_PIPE_SEL_SHIFT_CHV;
@@ -337,7 +334,6 @@ static void intel_dp_get_config(struct intel_encoder *encoder,
                                struct intel_crtc_state *pipe_config)
 {
        struct intel_display *display = to_intel_display(encoder);
-       struct drm_i915_private *dev_priv = to_i915(encoder->base.dev);
        struct intel_dp *intel_dp = enc_to_intel_dp(encoder);
        u32 tmp, flags = 0;
        enum port port = encoder->port;
@@ -352,7 +348,7 @@ static void intel_dp_get_config(struct intel_encoder *encoder,
 
        pipe_config->has_audio = tmp & DP_AUDIO_OUTPUT_ENABLE && port != PORT_A;
 
-       if (HAS_PCH_CPT(dev_priv) && port != PORT_A) {
+       if (HAS_PCH_CPT(display) && port != PORT_A) {
                u32 trans_dp = intel_de_read(display,
                                             TRANS_DP_CTL(crtc->pipe));
 
@@ -415,7 +411,6 @@ intel_dp_link_down(struct intel_encoder *encoder,
                   const struct intel_crtc_state *old_crtc_state)
 {
        struct intel_display *display = to_intel_display(encoder);
-       struct drm_i915_private *dev_priv = to_i915(encoder->base.dev);
        struct intel_dp *intel_dp = enc_to_intel_dp(encoder);
        struct intel_crtc *crtc = to_intel_crtc(old_crtc_state->uapi.crtc);
        enum port port = encoder->port;
@@ -428,7 +423,7 @@ intel_dp_link_down(struct intel_encoder *encoder,
        drm_dbg_kms(display->drm, "\n");
 
        if ((display->platform.ivybridge && port == PORT_A) ||
-           (HAS_PCH_CPT(dev_priv) && port != PORT_A)) {
+           (HAS_PCH_CPT(display) && port != PORT_A)) {
                intel_dp->DP &= ~DP_LINK_TRAIN_MASK_CPT;
                intel_dp->DP |= DP_LINK_TRAIN_PAT_IDLE_CPT;
        } else {
@@ -447,7 +442,7 @@ intel_dp_link_down(struct intel_encoder *encoder,
         * to transcoder A after disabling it to allow the
         * matching HDMI port to be enabled on transcoder A.
         */
-       if (HAS_PCH_IBX(dev_priv) && crtc->pipe == PIPE_B && port != PORT_A) {
+       if (HAS_PCH_IBX(display) && crtc->pipe == PIPE_B && port != PORT_A) {
                /*
                 * We get CPU/PCH FIFO underruns on the other pipe when
                 * doing the workaround. Sweep them under the rug.
@@ -1216,10 +1211,10 @@ static int g4x_dp_compute_config(struct intel_encoder *encoder,
                                 struct intel_crtc_state *crtc_state,
                                 struct drm_connector_state *conn_state)
 {
-       struct drm_i915_private *i915 = to_i915(encoder->base.dev);
+       struct intel_display *display = to_intel_display(encoder);
        int ret;
 
-       if (HAS_PCH_SPLIT(i915) && encoder->port != PORT_A)
+       if (HAS_PCH_SPLIT(display) && encoder->port != PORT_A)
                crtc_state->has_pch_encoder = true;
 
        ret = intel_dp_compute_config(encoder, crtc_state, conn_state);
@@ -1272,7 +1267,6 @@ static const struct drm_encoder_funcs intel_dp_enc_funcs = {
 bool g4x_dp_init(struct intel_display *display,
                 i915_reg_t output_reg, enum port port)
 {
-       struct drm_i915_private *dev_priv = to_i915(display->drm);
        const struct intel_bios_encoder_data *devdata;
        struct intel_digital_port *dig_port;
        struct intel_encoder *intel_encoder;
@@ -1346,7 +1340,7 @@ bool g4x_dp_init(struct intel_display *display,
        intel_encoder->audio_disable = g4x_dp_audio_disable;
 
        if ((display->platform.ivybridge && port == PORT_A) ||
-           (HAS_PCH_CPT(dev_priv) && port != PORT_A))
+           (HAS_PCH_CPT(display) && port != PORT_A))
                dig_port->dp.set_link_train = cpt_set_link_train;
        else
                dig_port->dp.set_link_train = g4x_set_link_train;
@@ -1363,7 +1357,7 @@ bool g4x_dp_init(struct intel_display *display,
                intel_encoder->set_signal_levels = g4x_set_signal_levels;
 
        if (display->platform.valleyview || display->platform.cherryview ||
-           (HAS_PCH_SPLIT(dev_priv) && port != PORT_A)) {
+           (HAS_PCH_SPLIT(display) && port != PORT_A)) {
                dig_port->dp.preemph_max = intel_dp_preemph_max_3;
                dig_port->dp.voltage_max = intel_dp_voltage_max_3;
        } else {
index 21b5db2fa203f2753029376f85b47e82c273d4dd..4e8b13e9ddd8fecb6fc1d6a20287cc0619a4ae51 100644 (file)
@@ -27,7 +27,6 @@ static void intel_hdmi_prepare(struct intel_encoder *encoder,
                               const struct intel_crtc_state *crtc_state)
 {
        struct intel_display *display = to_intel_display(encoder);
-       struct drm_i915_private *dev_priv = to_i915(display->drm);
        struct intel_crtc *crtc = to_intel_crtc(crtc_state->uapi.crtc);
        struct intel_hdmi *intel_hdmi = enc_to_intel_hdmi(encoder);
        const struct drm_display_mode *adjusted_mode = &crtc_state->hw.adjusted_mode;
@@ -36,7 +35,7 @@ static void intel_hdmi_prepare(struct intel_encoder *encoder,
        intel_dp_dual_mode_set_tmds_output(intel_hdmi, true);
 
        hdmi_val = SDVO_ENCODING_HDMI;
-       if (!HAS_PCH_SPLIT(dev_priv) && crtc_state->limited_color_range)
+       if (!HAS_PCH_SPLIT(display) && crtc_state->limited_color_range)
                hdmi_val |= HDMI_COLOR_RANGE_16_235;
        if (adjusted_mode->flags & DRM_MODE_FLAG_PVSYNC)
                hdmi_val |= SDVO_VSYNC_ACTIVE_HIGH;
@@ -51,7 +50,7 @@ static void intel_hdmi_prepare(struct intel_encoder *encoder,
        if (crtc_state->has_hdmi_sink)
                hdmi_val |= HDMI_MODE_SELECT_HDMI;
 
-       if (HAS_PCH_CPT(dev_priv))
+       if (HAS_PCH_CPT(display))
                hdmi_val |= SDVO_PIPE_SEL_CPT(crtc->pipe);
        else if (display->platform.cherryview)
                hdmi_val |= SDVO_PIPE_SEL_CHV(crtc->pipe);
@@ -133,9 +132,8 @@ static int g4x_hdmi_compute_config(struct intel_encoder *encoder,
        struct intel_display *display = to_intel_display(encoder);
        struct intel_atomic_state *state = to_intel_atomic_state(crtc_state->uapi.state);
        struct intel_crtc *crtc = to_intel_crtc(crtc_state->uapi.crtc);
-       struct drm_i915_private *i915 = to_i915(encoder->base.dev);
 
-       if (HAS_PCH_SPLIT(i915)) {
+       if (HAS_PCH_SPLIT(display)) {
                crtc_state->has_pch_encoder = true;
                if (!intel_fdi_compute_pipe_bpp(crtc_state))
                        return -EINVAL;
@@ -154,7 +152,6 @@ static void intel_hdmi_get_config(struct intel_encoder *encoder,
                                  struct intel_crtc_state *pipe_config)
 {
        struct intel_display *display = to_intel_display(encoder);
-       struct drm_i915_private *dev_priv = to_i915(display->drm);
        struct intel_hdmi *intel_hdmi = enc_to_intel_hdmi(encoder);
        u32 tmp, flags = 0;
        int dotclock;
@@ -185,7 +182,7 @@ static void intel_hdmi_get_config(struct intel_encoder *encoder,
        if (tmp & HDMI_AUDIO_ENABLE)
                pipe_config->has_audio = true;
 
-       if (!HAS_PCH_SPLIT(dev_priv) &&
+       if (!HAS_PCH_SPLIT(display) &&
            tmp & HDMI_COLOR_RANGE_16_235)
                pipe_config->limited_color_range = true;
 
@@ -382,7 +379,6 @@ static void intel_disable_hdmi(struct intel_atomic_state *state,
                               const struct drm_connector_state *old_conn_state)
 {
        struct intel_display *display = to_intel_display(encoder);
-       struct drm_i915_private *dev_priv = to_i915(display->drm);
        struct intel_hdmi *intel_hdmi = enc_to_intel_hdmi(encoder);
        struct intel_digital_port *dig_port =
                hdmi_to_dig_port(intel_hdmi);
@@ -400,7 +396,7 @@ static void intel_disable_hdmi(struct intel_atomic_state *state,
         * to transcoder A after disabling it to allow the
         * matching DP port to be enabled on transcoder A.
         */
-       if (HAS_PCH_IBX(dev_priv) && crtc->pipe == PIPE_B) {
+       if (HAS_PCH_IBX(display) && crtc->pipe == PIPE_B) {
                /*
                 * We get CPU/PCH FIFO underruns on the other pipe when
                 * doing the workaround. Sweep them under the rug.
@@ -674,7 +670,6 @@ static bool assert_hdmi_port_valid(struct intel_display *display, enum port port
 bool g4x_hdmi_init(struct intel_display *display,
                   i915_reg_t hdmi_reg, enum port port)
 {
-       struct drm_i915_private *dev_priv = to_i915(display->drm);
        const struct intel_bios_encoder_data *devdata;
        struct intel_digital_port *dig_port;
        struct intel_encoder *intel_encoder;
@@ -716,7 +711,7 @@ bool g4x_hdmi_init(struct intel_display *display,
 
        intel_encoder->hotplug = intel_hdmi_hotplug;
        intel_encoder->compute_config = g4x_hdmi_compute_config;
-       if (HAS_PCH_SPLIT(dev_priv)) {
+       if (HAS_PCH_SPLIT(display)) {
                intel_encoder->disable = pch_disable_hdmi;
                intel_encoder->post_disable = pch_post_disable_hdmi;
        } else {
@@ -737,9 +732,9 @@ bool g4x_hdmi_init(struct intel_display *display,
                intel_encoder->post_disable = vlv_hdmi_post_disable;
        } else {
                intel_encoder->pre_enable = intel_hdmi_pre_enable;
-               if (HAS_PCH_CPT(dev_priv))
+               if (HAS_PCH_CPT(display))
                        intel_encoder->enable = cpt_enable_hdmi;
-               else if (HAS_PCH_IBX(dev_priv))
+               else if (HAS_PCH_IBX(display))
                        intel_encoder->enable = ibx_enable_hdmi;
                else
                        intel_encoder->enable = g4x_enable_hdmi;
index 40751f1547b76ef05af62f8c7292ca53a8880810..77876ef735b746b486e2efcb492d8b4752522ba2 100644 (file)
@@ -4143,10 +4143,8 @@ static const struct intel_wm_funcs nop_funcs = {
 
 void i9xx_wm_init(struct intel_display *display)
 {
-       struct drm_i915_private *dev_priv = to_i915(display->drm);
-
        /* For FIFO watermark updates */
-       if (HAS_PCH_SPLIT(dev_priv)) {
+       if (HAS_PCH_SPLIT(display)) {
                ilk_setup_wm_latency(display);
                display->funcs.wm = &ilk_wm_funcs;
        } else if (display->platform.valleyview || display->platform.cherryview) {
index ea935a5d94c87202a68f84b28b0152835f47fe73..ef07b548826191929eff8fd68493c87c79c2913f 100644 (file)
@@ -587,19 +587,17 @@ static void ibx_audio_regs_init(struct intel_display *display,
                                enum pipe pipe,
                                struct ibx_audio_regs *regs)
 {
-       struct drm_i915_private *i915 = to_i915(display->drm);
-
        if (display->platform.valleyview || display->platform.cherryview) {
                regs->hdmiw_hdmiedid = VLV_HDMIW_HDMIEDID(pipe);
                regs->aud_config = VLV_AUD_CFG(pipe);
                regs->aud_cntl_st = VLV_AUD_CNTL_ST(pipe);
                regs->aud_cntrl_st2 = VLV_AUD_CNTL_ST2;
-       } else if (HAS_PCH_CPT(i915)) {
+       } else if (HAS_PCH_CPT(display)) {
                regs->hdmiw_hdmiedid = CPT_HDMIW_HDMIEDID(pipe);
                regs->aud_config = CPT_AUD_CFG(pipe);
                regs->aud_cntl_st = CPT_AUD_CNTL_ST(pipe);
                regs->aud_cntrl_st2 = CPT_AUD_CNTRL_ST2;
-       } else if (HAS_PCH_IBX(i915)) {
+       } else if (HAS_PCH_IBX(display)) {
                regs->hdmiw_hdmiedid = IBX_HDMIW_HDMIEDID(pipe);
                regs->aud_config = IBX_AUD_CFG(pipe);
                regs->aud_cntl_st = IBX_AUD_CNTL_ST(pipe);
@@ -889,12 +887,10 @@ static const struct intel_audio_funcs hsw_audio_funcs = {
  */
 void intel_audio_hooks_init(struct intel_display *display)
 {
-       struct drm_i915_private *i915 = to_i915(display->drm);
-
        if (display->platform.g4x)
                display->funcs.audio = &g4x_audio_funcs;
        else if (display->platform.valleyview || display->platform.cherryview ||
-                HAS_PCH_CPT(i915) || HAS_PCH_IBX(i915))
+                HAS_PCH_CPT(display) || HAS_PCH_IBX(display))
                display->funcs.audio = &ibx_audio_funcs;
        else if (display->platform.haswell || DISPLAY_VER(display) >= 8)
                display->funcs.audio = &hsw_audio_funcs;
index 4f3fa966c5372ff5162fe364ec6c1c14bffd914c..5470fee7a707ef71ba8fdacbbf27eaf4ebfece9f 100644 (file)
@@ -473,7 +473,6 @@ static void lpt_enable_backlight(const struct intel_crtc_state *crtc_state,
 {
        struct intel_connector *connector = to_intel_connector(conn_state->connector);
        struct intel_display *display = to_intel_display(connector);
-       struct drm_i915_private *i915 = to_i915(connector->base.dev);
        struct intel_panel *panel = &connector->panel;
        u32 pch_ctl1, pch_ctl2;
 
@@ -486,7 +485,7 @@ static void lpt_enable_backlight(const struct intel_crtc_state *crtc_state,
                intel_de_write(display, BLC_PWM_PCH_CTL1, pch_ctl1);
        }
 
-       if (HAS_PCH_LPT(i915))
+       if (HAS_PCH_LPT(display))
                intel_de_rmw(display, SOUTH_CHICKEN2, LPT_PWM_GRANULARITY,
                             panel->backlight.alternate_pwm_increment ?
                             LPT_PWM_GRANULARITY : 0);
@@ -503,7 +502,7 @@ static void lpt_enable_backlight(const struct intel_crtc_state *crtc_state,
                pch_ctl1 |= BLM_PCH_POLARITY;
 
        /* After LPT, override is the default. */
-       if (HAS_PCH_LPT(i915))
+       if (HAS_PCH_LPT(display))
                pch_ctl1 |= BLM_PCH_OVERRIDE_ENABLE;
 
        intel_de_write(display, BLC_PWM_PCH_CTL1, pch_ctl1);
@@ -1064,7 +1063,7 @@ static u32 spt_hz_to_pwm(struct intel_connector *connector, u32 pwm_freq_hz)
  */
 static u32 lpt_hz_to_pwm(struct intel_connector *connector, u32 pwm_freq_hz)
 {
-       struct drm_i915_private *i915 = to_i915(connector->base.dev);
+       struct intel_display *display = to_intel_display(connector);
        struct intel_panel *panel = &connector->panel;
        u32 mul, clock;
 
@@ -1073,7 +1072,7 @@ static u32 lpt_hz_to_pwm(struct intel_connector *connector, u32 pwm_freq_hz)
        else
                mul = 128;
 
-       if (HAS_PCH_LPT_H(i915))
+       if (HAS_PCH_LPT_H(display))
                clock = MHz(135); /* LPT:H */
        else
                clock = MHz(24); /* LPT:LP */
@@ -1230,12 +1229,11 @@ static u32 get_backlight_min_vbt(struct intel_connector *connector)
 static int lpt_setup_backlight(struct intel_connector *connector, enum pipe unused)
 {
        struct intel_display *display = to_intel_display(connector);
-       struct drm_i915_private *i915 = to_i915(connector->base.dev);
        struct intel_panel *panel = &connector->panel;
        u32 cpu_ctl2, pch_ctl1, pch_ctl2, val;
        bool alt, cpu_mode;
 
-       if (HAS_PCH_LPT(i915))
+       if (HAS_PCH_LPT(display))
                alt = intel_de_read(display, SOUTH_CHICKEN2) & LPT_PWM_GRANULARITY;
        else
                alt = intel_de_read(display, SOUTH_CHICKEN1) & SPT_PWM_GRANULARITY;
@@ -1259,7 +1257,7 @@ static int lpt_setup_backlight(struct intel_connector *connector, enum pipe unus
 
        panel->backlight.pwm_enabled = pch_ctl1 & BLM_PCH_PWM_ENABLE;
 
-       cpu_mode = panel->backlight.pwm_enabled && HAS_PCH_LPT(i915) &&
+       cpu_mode = panel->backlight.pwm_enabled && HAS_PCH_LPT(display) &&
                   !(pch_ctl1 & BLM_PCH_OVERRIDE_ENABLE) &&
                   (cpu_ctl2 & BLM_PWM_ENABLE);
 
@@ -1466,15 +1464,13 @@ bxt_setup_backlight(struct intel_connector *connector, enum pipe unused)
 
 static int cnp_num_backlight_controllers(struct intel_display *display)
 {
-       struct drm_i915_private *i915 = to_i915(display->drm);
-
-       if (INTEL_PCH_TYPE(i915) >= PCH_MTL)
+       if (INTEL_PCH_TYPE(display) >= PCH_MTL)
                return 2;
 
-       if (INTEL_PCH_TYPE(i915) >= PCH_DG1)
+       if (INTEL_PCH_TYPE(display) >= PCH_DG1)
                return 1;
 
-       if (INTEL_PCH_TYPE(i915) >= PCH_ICP)
+       if (INTEL_PCH_TYPE(display) >= PCH_ICP)
                return 2;
 
        return 1;
@@ -1482,14 +1478,12 @@ static int cnp_num_backlight_controllers(struct intel_display *display)
 
 static bool cnp_backlight_controller_is_valid(struct intel_display *display, int controller)
 {
-       struct drm_i915_private *i915 = to_i915(display->drm);
-
        if (controller < 0 || controller >= cnp_num_backlight_controllers(display))
                return false;
 
        if (controller == 1 &&
-           INTEL_PCH_TYPE(i915) >= PCH_ICP &&
-           INTEL_PCH_TYPE(i915) <= PCH_ADP)
+           INTEL_PCH_TYPE(display) >= PCH_ICP &&
+           INTEL_PCH_TYPE(display) <= PCH_ADP)
                return intel_de_read(display, SOUTH_CHICKEN1) & ICP_SECOND_PPS_IO_SELECT;
 
        return true;
@@ -1818,7 +1812,6 @@ void intel_backlight_init_funcs(struct intel_panel *panel)
        struct intel_connector *connector =
                container_of(panel, struct intel_connector, panel);
        struct intel_display *display = to_intel_display(connector);
-       struct drm_i915_private *i915 = to_i915(connector->base.dev);
 
        if (connector->base.connector_type == DRM_MODE_CONNECTOR_DSI &&
            intel_dsi_dcs_init_backlight_funcs(connector) == 0)
@@ -1826,14 +1819,14 @@ void intel_backlight_init_funcs(struct intel_panel *panel)
 
        if (display->platform.geminilake || display->platform.broxton) {
                panel->backlight.pwm_funcs = &bxt_pwm_funcs;
-       } else if (INTEL_PCH_TYPE(i915) >= PCH_CNP) {
+       } else if (INTEL_PCH_TYPE(display) >= PCH_CNP) {
                panel->backlight.pwm_funcs = &cnp_pwm_funcs;
-       } else if (INTEL_PCH_TYPE(i915) >= PCH_LPT_H) {
-               if (HAS_PCH_LPT(i915))
+       } else if (INTEL_PCH_TYPE(display) >= PCH_LPT_H) {
+               if (HAS_PCH_LPT(display))
                        panel->backlight.pwm_funcs = &lpt_pwm_funcs;
                else
                        panel->backlight.pwm_funcs = &spt_pwm_funcs;
-       } else if (HAS_PCH_SPLIT(i915)) {
+       } else if (HAS_PCH_SPLIT(display)) {
                panel->backlight.pwm_funcs = &pch_pwm_funcs;
        } else if (display->platform.valleyview || display->platform.cherryview) {
                if (connector->base.connector_type == DRM_MODE_CONNECTOR_DSI) {
index 8fe854a452437343fb765fe32c3b2d5d6330734d..ba7b8938b17c31eda2be3a73eb199b4f1fb68960 100644 (file)
@@ -2245,28 +2245,27 @@ static const u8 adlp_ddc_pin_map[] = {
 
 static u8 map_ddc_pin(struct intel_display *display, u8 vbt_pin)
 {
-       struct drm_i915_private *i915 = to_i915(display->drm);
        const u8 *ddc_pin_map;
        int i, n_entries;
 
-       if (INTEL_PCH_TYPE(i915) >= PCH_MTL || display->platform.alderlake_p) {
+       if (INTEL_PCH_TYPE(display) >= PCH_MTL || display->platform.alderlake_p) {
                ddc_pin_map = adlp_ddc_pin_map;
                n_entries = ARRAY_SIZE(adlp_ddc_pin_map);
        } else if (display->platform.alderlake_s) {
                ddc_pin_map = adls_ddc_pin_map;
                n_entries = ARRAY_SIZE(adls_ddc_pin_map);
-       } else if (INTEL_PCH_TYPE(i915) >= PCH_DG1) {
+       } else if (INTEL_PCH_TYPE(display) >= PCH_DG1) {
                return vbt_pin;
-       } else if (display->platform.rocketlake && INTEL_PCH_TYPE(i915) == PCH_TGP) {
+       } else if (display->platform.rocketlake && INTEL_PCH_TYPE(display) == PCH_TGP) {
                ddc_pin_map = rkl_pch_tgp_ddc_pin_map;
                n_entries = ARRAY_SIZE(rkl_pch_tgp_ddc_pin_map);
-       } else if (HAS_PCH_TGP(i915) && DISPLAY_VER(display) == 9) {
+       } else if (HAS_PCH_TGP(display) && DISPLAY_VER(display) == 9) {
                ddc_pin_map = gen9bc_tgp_ddc_pin_map;
                n_entries = ARRAY_SIZE(gen9bc_tgp_ddc_pin_map);
-       } else if (INTEL_PCH_TYPE(i915) >= PCH_ICP) {
+       } else if (INTEL_PCH_TYPE(display) >= PCH_ICP) {
                ddc_pin_map = icp_ddc_pin_map;
                n_entries = ARRAY_SIZE(icp_ddc_pin_map);
-       } else if (HAS_PCH_CNP(i915)) {
+       } else if (HAS_PCH_CNP(display)) {
                ddc_pin_map = cnp_ddc_pin_map;
                n_entries = ARRAY_SIZE(cnp_ddc_pin_map);
        } else {
@@ -2865,8 +2864,6 @@ parse_general_definitions(struct intel_display *display)
 static void
 init_vbt_defaults(struct intel_display *display)
 {
-       struct drm_i915_private *i915 = to_i915(display->drm);
-
        display->vbt.crt_ddc_pin = GMBUS_PIN_VGADDC;
 
        /* general features */
@@ -2883,7 +2880,7 @@ init_vbt_defaults(struct intel_display *display)
         * clock for LVDS.
         */
        display->vbt.lvds_ssc_freq = intel_bios_ssc_frequency(display,
-                                                             !HAS_PCH_SPLIT(i915));
+                                                             !HAS_PCH_SPLIT(display));
        drm_dbg_kms(display->drm, "Set default to SSC at %d kHz\n",
                    display->vbt.lvds_ssc_freq);
 }
index 6830950aae3f2379350691353195eb473dcf25c8..b1718b491ffdab596f8313d518fa048243305afb 100644 (file)
@@ -3494,7 +3494,6 @@ static int dg1_rawclk(struct intel_display *display)
 
 static int cnp_rawclk(struct intel_display *display)
 {
-       struct drm_i915_private *dev_priv = to_i915(display->drm);
        int divider, fraction;
        u32 rawclk;
 
@@ -3514,7 +3513,7 @@ static int cnp_rawclk(struct intel_display *display)
 
                rawclk |= CNP_RAWCLK_DEN(DIV_ROUND_CLOSEST(numerator * 1000,
                                                           fraction) - 1);
-               if (INTEL_PCH_TYPE(dev_priv) >= PCH_ICP)
+               if (INTEL_PCH_TYPE(display) >= PCH_ICP)
                        rawclk |= ICP_RAWCLK_NUM(numerator);
        }
 
@@ -3553,21 +3552,20 @@ static int i9xx_hrawclk(struct intel_display *display)
  */
 u32 intel_read_rawclk(struct intel_display *display)
 {
-       struct drm_i915_private *dev_priv = to_i915(display->drm);
        u32 freq;
 
-       if (INTEL_PCH_TYPE(dev_priv) >= PCH_MTL)
+       if (INTEL_PCH_TYPE(display) >= PCH_MTL)
                /*
                 * MTL always uses a 38.4 MHz rawclk.  The bspec tells us
                 * "RAWCLK_FREQ defaults to the values for 38.4 and does
                 * not need to be programmed."
                 */
                freq = 38400;
-       else if (INTEL_PCH_TYPE(dev_priv) >= PCH_DG1)
+       else if (INTEL_PCH_TYPE(display) >= PCH_DG1)
                freq = dg1_rawclk(display);
-       else if (INTEL_PCH_TYPE(dev_priv) >= PCH_CNP)
+       else if (INTEL_PCH_TYPE(display) >= PCH_CNP)
                freq = cnp_rawclk(display);
-       else if (HAS_PCH_SPLIT(dev_priv))
+       else if (HAS_PCH_SPLIT(display))
                freq = pch_rawclk(display);
        else if (display->platform.valleyview || display->platform.cherryview)
                freq = vlv_hrawclk(display);
index 8908b51dc3b93381e3f6f6061a9bc2abe9ec72da..415e594053180945b7c9044958bd71b623dd2a59 100644 (file)
@@ -91,13 +91,12 @@ static struct intel_crt *intel_attached_crt(struct intel_connector *connector)
 bool intel_crt_port_enabled(struct intel_display *display,
                            i915_reg_t adpa_reg, enum pipe *pipe)
 {
-       struct drm_i915_private *dev_priv = to_i915(display->drm);
        u32 val;
 
        val = intel_de_read(display, adpa_reg);
 
        /* asserts want to know the pipe even if the port is disabled */
-       if (HAS_PCH_CPT(dev_priv))
+       if (HAS_PCH_CPT(display))
                *pipe = REG_FIELD_GET(ADPA_PIPE_SEL_MASK_CPT, val);
        else
                *pipe = REG_FIELD_GET(ADPA_PIPE_SEL_MASK, val);
@@ -177,7 +176,6 @@ static void intel_crt_set_dpms(struct intel_encoder *encoder,
                               int mode)
 {
        struct intel_display *display = to_intel_display(encoder);
-       struct drm_i915_private *dev_priv = to_i915(encoder->base.dev);
        struct intel_crt *crt = intel_encoder_to_crt(encoder);
        struct intel_crtc *crtc = to_intel_crtc(crtc_state->uapi.crtc);
        const struct drm_display_mode *adjusted_mode = &crtc_state->hw.adjusted_mode;
@@ -194,14 +192,14 @@ static void intel_crt_set_dpms(struct intel_encoder *encoder,
                adpa |= ADPA_VSYNC_ACTIVE_HIGH;
 
        /* For CPT allow 3 pipe config, for others just use A or B */
-       if (HAS_PCH_LPT(dev_priv))
+       if (HAS_PCH_LPT(display))
                ; /* Those bits don't exist here */
-       else if (HAS_PCH_CPT(dev_priv))
+       else if (HAS_PCH_CPT(display))
                adpa |= ADPA_PIPE_SEL_CPT(crtc->pipe);
        else
                adpa |= ADPA_PIPE_SEL(crtc->pipe);
 
-       if (!HAS_PCH_SPLIT(dev_priv))
+       if (!HAS_PCH_SPLIT(display))
                intel_de_write(display, BCLRPAT(display, crtc->pipe), 0);
 
        switch (mode) {
@@ -356,7 +354,6 @@ intel_crt_mode_valid(struct drm_connector *connector,
                     const struct drm_display_mode *mode)
 {
        struct intel_display *display = to_intel_display(connector->dev);
-       struct drm_i915_private *dev_priv = to_i915(connector->dev);
        int max_dotclk = display->cdclk.max_dotclk_freq;
        enum drm_mode_status status;
        int max_clock;
@@ -368,7 +365,7 @@ intel_crt_mode_valid(struct drm_connector *connector,
        if (mode->clock < 25000)
                return MODE_CLOCK_LOW;
 
-       if (HAS_PCH_LPT(dev_priv))
+       if (HAS_PCH_LPT(display))
                max_clock = 180000;
        else if (display->platform.valleyview)
                /*
@@ -387,7 +384,7 @@ intel_crt_mode_valid(struct drm_connector *connector,
                return MODE_CLOCK_HIGH;
 
        /* The FDI receiver on LPT only supports 8bpc and only has 2 lanes. */
-       if (HAS_PCH_LPT(dev_priv) &&
+       if (HAS_PCH_LPT(display) &&
            ilk_get_lanes_required(mode->clock, 270000, 24) > 2)
                return MODE_CLOCK_HIGH;
 
@@ -438,7 +435,6 @@ static int hsw_crt_compute_config(struct intel_encoder *encoder,
                                  struct drm_connector_state *conn_state)
 {
        struct intel_display *display = to_intel_display(encoder);
-       struct drm_i915_private *dev_priv = to_i915(encoder->base.dev);
        struct drm_display_mode *adjusted_mode =
                &crtc_state->hw.adjusted_mode;
 
@@ -457,7 +453,7 @@ static int hsw_crt_compute_config(struct intel_encoder *encoder,
        crtc_state->output_format = INTEL_OUTPUT_FORMAT_RGB;
 
        /* LPT FDI RX only supports 8bpc. */
-       if (HAS_PCH_LPT(dev_priv)) {
+       if (HAS_PCH_LPT(display)) {
                /* TODO: Check crtc_state->max_link_bpp_x16 instead of bw_constrained */
                if (crtc_state->bw_constrained && crtc_state->pipe_bpp < 24) {
                        drm_dbg_kms(display->drm,
@@ -482,13 +478,12 @@ static bool ilk_crt_detect_hotplug(struct drm_connector *connector)
 {
        struct intel_display *display = to_intel_display(connector->dev);
        struct intel_crt *crt = intel_attached_crt(to_intel_connector(connector));
-       struct drm_i915_private *dev_priv = to_i915(connector->dev);
        u32 adpa;
        bool ret;
 
        /* The first time through, trigger an explicit detection cycle */
        if (crt->force_hotplug_required) {
-               bool turn_off_dac = HAS_PCH_SPLIT(dev_priv);
+               bool turn_off_dac = HAS_PCH_SPLIT(display);
                u32 save_adpa;
 
                crt->force_hotplug_required = false;
@@ -583,12 +578,11 @@ static bool valleyview_crt_detect_hotplug(struct drm_connector *connector)
 static bool intel_crt_detect_hotplug(struct drm_connector *connector)
 {
        struct intel_display *display = to_intel_display(connector->dev);
-       struct drm_i915_private *dev_priv = to_i915(connector->dev);
        u32 stat;
        bool ret = false;
        int i, tries = 0;
 
-       if (HAS_PCH_SPLIT(dev_priv))
+       if (HAS_PCH_SPLIT(display))
                return ilk_crt_detect_hotplug(connector);
 
        if (display->platform.valleyview)
@@ -1011,14 +1005,13 @@ static const struct drm_encoder_funcs intel_crt_enc_funcs = {
 
 void intel_crt_init(struct intel_display *display)
 {
-       struct drm_i915_private *dev_priv = to_i915(display->drm);
        struct intel_connector *connector;
        struct intel_crt *crt;
        i915_reg_t adpa_reg;
        u8 ddc_pin;
        u32 adpa;
 
-       if (HAS_PCH_SPLIT(dev_priv))
+       if (HAS_PCH_SPLIT(display))
                adpa_reg = PCH_ADPA;
        else if (display->platform.valleyview)
                adpa_reg = VLV_ADPA;
@@ -1108,7 +1101,7 @@ void intel_crt_init(struct intel_display *display)
 
                intel_ddi_buf_trans_init(&crt->base);
        } else {
-               if (HAS_PCH_SPLIT(dev_priv)) {
+               if (HAS_PCH_SPLIT(display)) {
                        crt->base.compute_config = pch_crt_compute_config;
                        crt->base.disable = pch_disable_crt;
                        crt->base.post_disable = pch_post_disable_crt;
@@ -1130,7 +1123,7 @@ void intel_crt_init(struct intel_display *display)
         * polarity and link reversal bits or not, instead of relying on the
         * BIOS.
         */
-       if (HAS_PCH_LPT(dev_priv)) {
+       if (HAS_PCH_LPT(display)) {
                u32 fdi_config = FDI_RX_POLARITY_REVERSED_LPT |
                                 FDI_RX_LINK_REVERSAL_OVERRIDE;
 
index b48ed5df7a9671362be0aef9af8ce1198ab51fba..99142b8cf56909aeb7dcc82e2ab8d86ea3684ac4 100644 (file)
@@ -4902,9 +4902,7 @@ static enum hpd_pin tgl_hpd_pin(struct intel_display *display, enum port port)
 
 static enum hpd_pin rkl_hpd_pin(struct intel_display *display, enum port port)
 {
-       struct drm_i915_private *dev_priv = to_i915(display->drm);
-
-       if (HAS_PCH_TGP(dev_priv))
+       if (HAS_PCH_TGP(display))
                return tgl_hpd_pin(display, port);
 
        if (port >= PORT_TC1)
@@ -4923,12 +4921,10 @@ static enum hpd_pin icl_hpd_pin(struct intel_display *display, enum port port)
 
 static enum hpd_pin ehl_hpd_pin(struct intel_display *display, enum port port)
 {
-       struct drm_i915_private *dev_priv = to_i915(display->drm);
-
        if (port == PORT_D)
                return HPD_PORT_A;
 
-       if (HAS_PCH_TGP(dev_priv))
+       if (HAS_PCH_TGP(display))
                return icl_hpd_pin(display, port);
 
        return HPD_PORT_A + port - PORT_A;
@@ -4936,9 +4932,7 @@ static enum hpd_pin ehl_hpd_pin(struct intel_display *display, enum port port)
 
 static enum hpd_pin skl_hpd_pin(struct intel_display *display, enum port port)
 {
-       struct drm_i915_private *dev_priv = to_i915(display->drm);
-
-       if (HAS_PCH_TGP(dev_priv))
+       if (HAS_PCH_TGP(display))
                return icl_hpd_pin(display, port);
 
        return HPD_PORT_A + port - PORT_A;
index 33c09999c42e046c7a8b6e6143be0b81650ee7f7..9c966bbbfd31633321074cd8d3a02668a8745f8d 100644 (file)
@@ -1509,7 +1509,6 @@ static void ilk_crtc_enable(struct intel_atomic_state *state,
        struct intel_display *display = to_intel_display(crtc);
        const struct intel_crtc_state *new_crtc_state =
                intel_atomic_get_new_crtc_state(state, crtc);
-       struct drm_i915_private *dev_priv = to_i915(crtc->base.dev);
        enum pipe pipe = crtc->pipe;
 
        if (drm_WARN_ON(display->drm, crtc->active))
@@ -1561,7 +1560,7 @@ static void ilk_crtc_enable(struct intel_atomic_state *state,
 
        intel_encoders_enable(state, crtc);
 
-       if (HAS_PCH_CPT(dev_priv))
+       if (HAS_PCH_CPT(display))
                intel_wait_for_pipe_scanline_moving(crtc);
 
        /*
@@ -2533,15 +2532,13 @@ intel_link_compute_m_n(u16 bits_per_pixel_x16, int nlanes,
 
 void intel_panel_sanitize_ssc(struct intel_display *display)
 {
-       struct drm_i915_private *dev_priv = to_i915(display->drm);
-
        /*
         * There may be no VBT; and if the BIOS enabled SSC we can
         * just keep using it to avoid unnecessary flicker.  Whereas if the
         * BIOS isn't using it, don't assume it will work even if the VBT
         * indicates as much.
         */
-       if (HAS_PCH_IBX(dev_priv) || HAS_PCH_CPT(dev_priv)) {
+       if (HAS_PCH_IBX(display) || HAS_PCH_CPT(display)) {
                bool bios_lvds_use_ssc = intel_de_read(display,
                                                       PCH_DREF_CONTROL) &
                        DREF_SSC1_ENABLE;
@@ -6546,7 +6543,6 @@ static void intel_pipe_fastset(const struct intel_crtc_state *old_crtc_state,
 {
        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 *dev_priv = to_i915(crtc->base.dev);
 
        /*
         * Update pipe size and adjust fitter if needed: the reason for this is
@@ -6562,7 +6558,7 @@ static void intel_pipe_fastset(const struct intel_crtc_state *old_crtc_state,
        if (DISPLAY_VER(display) >= 9) {
                if (new_crtc_state->pch_pfit.enabled)
                        skl_pfit_enable(new_crtc_state);
-       } else if (HAS_PCH_SPLIT(dev_priv)) {
+       } else if (HAS_PCH_SPLIT(display)) {
                if (new_crtc_state->pch_pfit.enabled)
                        ilk_pfit_enable(new_crtc_state);
                else if (old_crtc_state->pch_pfit.enabled)
@@ -7638,15 +7634,13 @@ static bool ilk_has_edp_a(struct intel_display *display)
 
 static bool intel_ddi_crt_present(struct intel_display *display)
 {
-       struct drm_i915_private *dev_priv = to_i915(display->drm);
-
        if (DISPLAY_VER(display) >= 9)
                return false;
 
        if (display->platform.haswell_ult || display->platform.broadwell_ult)
                return false;
 
-       if (HAS_PCH_LPT_H(dev_priv) &&
+       if (HAS_PCH_LPT_H(display) &&
            intel_de_read(display, SFUSE_STRAP) & SFUSE_STRAP_CRT_DISABLED)
                return false;
 
@@ -7668,7 +7662,6 @@ bool assert_port_valid(struct intel_display *display, enum port port)
 
 void intel_setup_outputs(struct intel_display *display)
 {
-       struct drm_i915_private *dev_priv = to_i915(display->drm);
        struct intel_encoder *encoder;
        bool dpd_is_edp = false;
 
@@ -7685,7 +7678,7 @@ void intel_setup_outputs(struct intel_display *display)
 
                if (display->platform.geminilake || display->platform.broxton)
                        vlv_dsi_init(display);
-       } else if (HAS_PCH_SPLIT(dev_priv)) {
+       } else if (HAS_PCH_SPLIT(display)) {
                int found;
 
                /*
@@ -8053,13 +8046,11 @@ static const struct intel_display_funcs i9xx_display_funcs = {
  */
 void intel_init_display_hooks(struct intel_display *display)
 {
-       struct drm_i915_private *dev_priv = to_i915(display->drm);
-
        if (DISPLAY_VER(display) >= 9) {
                display->funcs.display = &skl_display_funcs;
        } else if (HAS_DDI(display)) {
                display->funcs.display = &ddi_display_funcs;
-       } else if (HAS_PCH_SPLIT(dev_priv)) {
+       } else if (HAS_PCH_SPLIT(display)) {
                display->funcs.display = &pch_split_display_funcs;
        } else if (display->platform.cherryview ||
                   display->platform.valleyview) {
index 3f7c605d47d3a9f20ad092b2701ba88ed5ff36e7..957c9a579a18fe81465bfad3e9a3511f675f4ccd 100644 (file)
@@ -53,10 +53,9 @@ static struct intel_display *node_to_intel_display(struct drm_info_node *node)
 static int intel_display_caps(struct seq_file *m, void *data)
 {
        struct intel_display *display = node_to_intel_display(m->private);
-       struct drm_i915_private *i915 = to_i915(display->drm);
        struct drm_printer p = drm_seq_file_printer(m);
 
-       drm_printf(&p, "PCH type: %d\n", INTEL_PCH_TYPE(i915));
+       drm_printf(&p, "PCH type: %d\n", INTEL_PCH_TYPE(display));
 
        intel_display_device_info_print(DISPLAY_INFO(display),
                                        DISPLAY_RUNTIME_INFO(display), &p);
@@ -85,7 +84,6 @@ static int i915_frontbuffer_tracking(struct seq_file *m, void *unused)
 static int i915_sr_status(struct seq_file *m, void *unused)
 {
        struct intel_display *display = node_to_intel_display(m->private);
-       struct drm_i915_private *dev_priv = to_i915(display->drm);
        intel_wakeref_t wakeref;
        bool sr_enabled = false;
 
@@ -93,7 +91,7 @@ static int i915_sr_status(struct seq_file *m, void *unused)
 
        if (DISPLAY_VER(display) >= 9)
                /* no global SR status; inspect per-plane WM */;
-       else if (HAS_PCH_SPLIT(dev_priv))
+       else if (HAS_PCH_SPLIT(display))
                sr_enabled = intel_de_read(display, WM1_LP_ILK) & WM_LP_ENABLE;
        else if (display->platform.i965gm || display->platform.g4x ||
                 display->platform.i945g || display->platform.i945gm)
index 738ae522c8f4fa234520572cb8f0704111c5b2ba..61f76487397af5f51756645fd1c901fd33de23a5 100644 (file)
@@ -1711,7 +1711,6 @@ void intel_display_device_remove(struct intel_display *display)
 
 static void __intel_display_device_info_runtime_init(struct intel_display *display)
 {
-       struct drm_i915_private *i915 = to_i915(display->drm);
        struct intel_display_runtime_info *display_runtime = DISPLAY_RUNTIME_INFO(display);
        enum pipe pipe;
 
@@ -1775,7 +1774,7 @@ static void __intel_display_device_info_runtime_init(struct intel_display *displ
                goto display_fused_off;
        }
 
-       if (IS_DISPLAY_VER(display, 7, 8) && HAS_PCH_SPLIT(i915)) {
+       if (IS_DISPLAY_VER(display, 7, 8) && HAS_PCH_SPLIT(display)) {
                u32 fuse_strap = intel_de_read(display, FUSE_STRAP);
                u32 sfuse_strap = intel_de_read(display, SFUSE_STRAP);
 
@@ -1790,7 +1789,7 @@ static void __intel_display_device_info_runtime_init(struct intel_display *displ
                 */
                if (fuse_strap & ILK_INTERNAL_DISPLAY_DISABLE ||
                    sfuse_strap & SFUSE_STRAP_DISPLAY_DISABLED ||
-                   (HAS_PCH_CPT(i915) &&
+                   (HAS_PCH_CPT(display) &&
                     !(sfuse_strap & SFUSE_STRAP_FUSE_LOCK))) {
                        drm_info(display->drm,
                                 "Display fused off, disabling\n");
index d28ad70a35380b3a2d19756b1b5aa079e30117fb..2e34af6e40f08e61966ee174be864eee9b8a9311 100644 (file)
@@ -879,7 +879,7 @@ static void ilk_gtt_fault_irq_handler(struct intel_display *display)
 
 void ilk_display_irq_handler(struct intel_display *display, u32 de_iir)
 {
-       struct drm_i915_private *dev_priv = to_i915(display->drm);
+       struct drm_i915_private __maybe_unused *dev_priv = to_i915(display->drm);
        enum pipe pipe;
        u32 hotplug_trigger = de_iir & DE_DP_A_HOTPLUG;
 
@@ -916,7 +916,7 @@ void ilk_display_irq_handler(struct intel_display *display, u32 de_iir)
        if (de_iir & DE_PCH_EVENT) {
                u32 pch_iir = intel_de_read(display, SDEIIR);
 
-               if (HAS_PCH_CPT(dev_priv))
+               if (HAS_PCH_CPT(display))
                        cpt_irq_handler(display, pch_iir);
                else
                        ibx_irq_handler(display, pch_iir);
@@ -931,7 +931,6 @@ void ilk_display_irq_handler(struct intel_display *display, u32 de_iir)
 
 void ivb_display_irq_handler(struct intel_display *display, u32 de_iir)
 {
-       struct drm_i915_private *dev_priv = to_i915(display->drm);
        enum pipe pipe;
        u32 hotplug_trigger = de_iir & DE_DP_A_HOTPLUG_IVB;
 
@@ -969,7 +968,7 @@ void ivb_display_irq_handler(struct intel_display *display, u32 de_iir)
        }
 
        /* check event from PCH */
-       if (!HAS_PCH_NOP(dev_priv) && (de_iir & DE_PCH_EVENT_IVB)) {
+       if (!HAS_PCH_NOP(display) && (de_iir & DE_PCH_EVENT_IVB)) {
                u32 pch_iir = intel_de_read(display, SDEIIR);
 
                cpt_irq_handler(display, pch_iir);
@@ -1311,7 +1310,6 @@ static u32 gen8_de_pipe_flip_done_mask(struct intel_display *display)
 
 static void gen8_read_and_ack_pch_irqs(struct intel_display *display, u32 *pch_iir, u32 *pica_iir)
 {
-       struct drm_i915_private *i915 = to_i915(display->drm);
        u32 pica_ier = 0;
 
        *pica_iir = 0;
@@ -1325,7 +1323,7 @@ static void gen8_read_and_ack_pch_irqs(struct intel_display *display, u32 *pch_i
         * their flags both in the PICA and SDE IIR.
         */
        if (*pch_iir & SDE_PICAINTERRUPT) {
-               drm_WARN_ON(display->drm, INTEL_PCH_TYPE(i915) < PCH_MTL);
+               drm_WARN_ON(display->drm, INTEL_PCH_TYPE(display) < PCH_MTL);
 
                pica_ier = intel_de_rmw(display, PICAINTERRUPT_IER, ~0, 0);
                *pica_iir = intel_de_read(display, PICAINTERRUPT_IIR);
@@ -1340,7 +1338,6 @@ static void gen8_read_and_ack_pch_irqs(struct intel_display *display, u32 *pch_i
 
 void gen8_de_irq_handler(struct intel_display *display, u32 master_ctl)
 {
-       struct drm_i915_private *dev_priv = to_i915(display->drm);
        u32 iir;
        enum pipe pipe;
 
@@ -1465,7 +1462,7 @@ void gen8_de_irq_handler(struct intel_display *display, u32 master_ctl)
                                                     pipe, fault_errors);
        }
 
-       if (HAS_PCH_SPLIT(dev_priv) && !HAS_PCH_NOP(dev_priv) &&
+       if (HAS_PCH_SPLIT(display) && !HAS_PCH_NOP(display) &&
            master_ctl & GEN8_DE_PCH_IRQ) {
                u32 pica_iir;
 
@@ -1479,9 +1476,9 @@ void gen8_de_irq_handler(struct intel_display *display, u32 master_ctl)
                        if (pica_iir)
                                xelpdp_pica_irq_handler(display, pica_iir);
 
-                       if (INTEL_PCH_TYPE(dev_priv) >= PCH_ICP)
+                       if (INTEL_PCH_TYPE(display) >= PCH_ICP)
                                icp_irq_handler(display, iir);
-                       else if (INTEL_PCH_TYPE(dev_priv) >= PCH_SPT)
+                       else if (INTEL_PCH_TYPE(display) >= PCH_SPT)
                                spt_irq_handler(display, iir);
                        else
                                cpt_irq_handler(display, iir);
@@ -1998,7 +1995,6 @@ void gen8_display_irq_reset(struct intel_display *display)
 
 void gen11_display_irq_reset(struct intel_display *display)
 {
-       struct drm_i915_private *dev_priv = to_i915(display->drm);
        enum pipe pipe;
        u32 trans_mask = BIT(TRANSCODER_A) | BIT(TRANSCODER_B) |
                BIT(TRANSCODER_C) | BIT(TRANSCODER_D);
@@ -2043,7 +2039,7 @@ void gen11_display_irq_reset(struct intel_display *display)
        else
                intel_display_irq_regs_reset(display, GEN11_DE_HPD_IRQ_REGS);
 
-       if (INTEL_PCH_TYPE(dev_priv) >= PCH_ICP)
+       if (INTEL_PCH_TYPE(display) >= PCH_ICP)
                intel_display_irq_regs_reset(display, SDE_IRQ_REGS);
 }
 
@@ -2105,15 +2101,14 @@ void gen8_irq_power_well_pre_disable(struct intel_display *display,
  */
 static void ibx_irq_postinstall(struct intel_display *display)
 {
-       struct drm_i915_private *dev_priv = to_i915(display->drm);
        u32 mask;
 
-       if (HAS_PCH_NOP(dev_priv))
+       if (HAS_PCH_NOP(display))
                return;
 
-       if (HAS_PCH_IBX(dev_priv))
+       if (HAS_PCH_IBX(display))
                mask = SDE_GMBUS | SDE_AUX_MASK | SDE_POISON;
-       else if (HAS_PCH_CPT(dev_priv) || HAS_PCH_LPT(dev_priv))
+       else if (HAS_PCH_CPT(display) || HAS_PCH_LPT(display))
                mask = SDE_GMBUS_CPT | SDE_AUX_MASK_CPT;
        else
                mask = SDE_GMBUS_CPT;
@@ -2201,8 +2196,6 @@ static void icp_irq_postinstall(struct intel_display *display);
 
 void gen8_de_irq_postinstall(struct intel_display *display)
 {
-       struct drm_i915_private *dev_priv = to_i915(display->drm);
-
        u32 de_pipe_masked = gen8_de_pipe_fault_mask(display) |
                GEN8_PIPE_CDCLK_CRC_DONE;
        u32 de_pipe_enables;
@@ -2218,9 +2211,9 @@ void gen8_de_irq_postinstall(struct intel_display *display)
 
        if (DISPLAY_VER(display) >= 14)
                mtp_irq_postinstall(display);
-       else if (INTEL_PCH_TYPE(dev_priv) >= PCH_ICP)
+       else if (INTEL_PCH_TYPE(display) >= PCH_ICP)
                icp_irq_postinstall(display);
-       else if (HAS_PCH_SPLIT(dev_priv))
+       else if (HAS_PCH_SPLIT(display))
                ibx_irq_postinstall(display);
 
        if (DISPLAY_VER(display) < 11)
index c78315eb44facfaca6efe68c8c057607cb2bd55b..77f26d9856fd008afe5356d05793978353a733d8 100644 (file)
@@ -1365,11 +1365,9 @@ static void hsw_restore_lcpll(struct intel_display *display)
  */
 static void hsw_enable_pc8(struct intel_display *display)
 {
-       struct drm_i915_private *dev_priv = to_i915(display->drm);
-
        drm_dbg_kms(display->drm, "Enabling package C8+\n");
 
-       if (HAS_PCH_LPT_LP(dev_priv))
+       if (HAS_PCH_LPT_LP(display))
                intel_de_rmw(display, SOUTH_DSPCLK_GATE_D,
                             PCH_LP_PARTITION_LEVEL_DISABLE, 0);
 
@@ -1415,14 +1413,13 @@ static void intel_pch_reset_handshake(struct intel_display *display,
 static void skl_display_core_init(struct intel_display *display,
                                  bool resume)
 {
-       struct drm_i915_private *dev_priv = to_i915(display->drm);
        struct i915_power_domains *power_domains = &display->power.domains;
        struct i915_power_well *well;
 
        gen9_set_dc_state(display, DC_STATE_DISABLE);
 
        /* enable PCH reset handshake */
-       intel_pch_reset_handshake(display, !HAS_PCH_NOP(dev_priv));
+       intel_pch_reset_handshake(display, !HAS_PCH_NOP(display));
 
        if (!HAS_DISPLAY(display))
                return;
@@ -1624,20 +1621,19 @@ static void tgl_bw_buddy_init(struct intel_display *display)
 static void icl_display_core_init(struct intel_display *display,
                                  bool resume)
 {
-       struct drm_i915_private *dev_priv = to_i915(display->drm);
        struct i915_power_domains *power_domains = &display->power.domains;
        struct i915_power_well *well;
 
        gen9_set_dc_state(display, DC_STATE_DISABLE);
 
        /* Wa_14011294188:ehl,jsl,tgl,rkl,adl-s */
-       if (INTEL_PCH_TYPE(dev_priv) >= PCH_TGP &&
-           INTEL_PCH_TYPE(dev_priv) < PCH_DG1)
+       if (INTEL_PCH_TYPE(display) >= PCH_TGP &&
+           INTEL_PCH_TYPE(display) < PCH_DG1)
                intel_de_rmw(display, SOUTH_DSPCLK_GATE_D, 0,
                             PCH_DPMGUNIT_CLOCK_GATE_DISABLE);
 
        /* 1. Enable PCH reset handshake. */
-       intel_pch_reset_handshake(display, !HAS_PCH_NOP(dev_priv));
+       intel_pch_reset_handshake(display, !HAS_PCH_NOP(display));
 
        if (!HAS_DISPLAY(display))
                return;
@@ -1908,7 +1904,6 @@ static void intel_power_domains_verify_state(struct intel_display *display);
  */
 void intel_power_domains_init_hw(struct intel_display *display, bool resume)
 {
-       struct drm_i915_private *i915 = to_i915(display->drm);
        struct i915_power_domains *power_domains = &display->power.domains;
 
        power_domains->initializing = true;
@@ -1932,9 +1927,9 @@ void intel_power_domains_init_hw(struct intel_display *display, bool resume)
                assert_isp_power_gated(display);
        } else if (display->platform.broadwell || display->platform.haswell) {
                hsw_assert_cdclk(display);
-               intel_pch_reset_handshake(display, !HAS_PCH_NOP(i915));
+               intel_pch_reset_handshake(display, !HAS_PCH_NOP(display));
        } else if (display->platform.ivybridge) {
-               intel_pch_reset_handshake(display, !HAS_PCH_NOP(i915));
+               intel_pch_reset_handshake(display, !HAS_PCH_NOP(display));
        }
 
        /*
@@ -2229,8 +2224,6 @@ static void intel_power_domains_verify_state(struct intel_display *display)
 
 void intel_display_power_suspend_late(struct intel_display *display, bool s2idle)
 {
-       struct drm_i915_private *i915 = to_i915(display->drm);
-
        intel_power_domains_suspend(display, s2idle);
 
        if (DISPLAY_VER(display) >= 11 || display->platform.geminilake ||
@@ -2241,14 +2234,12 @@ void intel_display_power_suspend_late(struct intel_display *display, bool s2idle
        }
 
        /* Tweaked Wa_14010685332:cnp,icp,jsp,mcc,tgp,adp */
-       if (INTEL_PCH_TYPE(i915) >= PCH_CNP && INTEL_PCH_TYPE(i915) < PCH_DG1)
-               intel_de_rmw(i915, SOUTH_CHICKEN1, SBCLK_RUN_REFCLK_DIS, SBCLK_RUN_REFCLK_DIS);
+       if (INTEL_PCH_TYPE(display) >= PCH_CNP && INTEL_PCH_TYPE(display) < PCH_DG1)
+               intel_de_rmw(display, SOUTH_CHICKEN1, SBCLK_RUN_REFCLK_DIS, SBCLK_RUN_REFCLK_DIS);
 }
 
 void intel_display_power_resume_early(struct intel_display *display)
 {
-       struct drm_i915_private *i915 = to_i915(display->drm);
-
        if (DISPLAY_VER(display) >= 11 || display->platform.geminilake ||
            display->platform.broxton) {
                gen9_sanitize_dc_state(display);
@@ -2258,8 +2249,8 @@ void intel_display_power_resume_early(struct intel_display *display)
        }
 
        /* Tweaked Wa_14010685332:cnp,icp,jsp,mcc,tgp,adp */
-       if (INTEL_PCH_TYPE(i915) >= PCH_CNP && INTEL_PCH_TYPE(i915) < PCH_DG1)
-               intel_de_rmw(i915, SOUTH_CHICKEN1, SBCLK_RUN_REFCLK_DIS, 0);
+       if (INTEL_PCH_TYPE(display) >= PCH_CNP && INTEL_PCH_TYPE(display) < PCH_DG1)
+               intel_de_rmw(display, SOUTH_CHICKEN1, SBCLK_RUN_REFCLK_DIS, 0);
 
        intel_power_domains_resume(display);
 }
index d7a30d0992b7a65250a3abfecde876321326fa84..660b1a8c39a46aaeeecaaf4f9cc7bbb61a278b0a 100644 (file)
@@ -6344,7 +6344,6 @@ static bool intel_edp_init_connector(struct intel_dp *intel_dp,
                                     struct intel_connector *connector)
 {
        struct intel_display *display = to_intel_display(intel_dp);
-       struct drm_i915_private *dev_priv = to_i915(display->drm);
        struct drm_display_mode *fixed_mode;
        struct intel_encoder *encoder = &dp_to_dig_port(intel_dp)->base;
        bool has_dpcd;
@@ -6361,7 +6360,7 @@ static bool intel_edp_init_connector(struct intel_dp *intel_dp,
         */
        if (intel_get_lvds_encoder(display)) {
                drm_WARN_ON(display->drm,
-                           !(HAS_PCH_IBX(dev_priv) || HAS_PCH_CPT(dev_priv)));
+                           !(HAS_PCH_IBX(display) || HAS_PCH_CPT(display)));
                drm_info(display->drm,
                         "LVDS was detected, not registering eDP\n");
 
index 3245a15935dba25e7a226621aaecec93606ee565..b6ef2116164cd28bbfecf671b7770535cf98612d 100644 (file)
@@ -111,10 +111,9 @@ static u32 ilk_get_aux_clock_divider(struct intel_dp *intel_dp, int index)
 static u32 hsw_get_aux_clock_divider(struct intel_dp *intel_dp, int index)
 {
        struct intel_display *display = to_intel_display(intel_dp);
-       struct drm_i915_private *i915 = to_i915(display->drm);
        struct intel_digital_port *dig_port = dp_to_dig_port(intel_dp);
 
-       if (dig_port->aux_ch != AUX_CH_A && HAS_PCH_LPT_H(i915)) {
+       if (dig_port->aux_ch != AUX_CH_A && HAS_PCH_LPT_H(display)) {
                /* Workaround for non-ULT HSW */
                switch (index) {
                case 0: return 63;
@@ -785,7 +784,6 @@ void intel_dp_aux_fini(struct intel_dp *intel_dp)
 void intel_dp_aux_init(struct intel_dp *intel_dp)
 {
        struct intel_display *display = to_intel_display(intel_dp);
-       struct drm_i915_private *i915 = to_i915(display->drm);
        struct intel_digital_port *dig_port = dp_to_dig_port(intel_dp);
        struct intel_encoder *encoder = &dig_port->base;
        enum aux_ch aux_ch = dig_port->aux_ch;
@@ -800,7 +798,7 @@ void intel_dp_aux_init(struct intel_dp *intel_dp)
        } else if (DISPLAY_VER(display) >= 9) {
                intel_dp->aux_ch_ctl_reg = skl_aux_ctl_reg;
                intel_dp->aux_ch_data_reg = skl_aux_data_reg;
-       } else if (HAS_PCH_SPLIT(i915)) {
+       } else if (HAS_PCH_SPLIT(display)) {
                intel_dp->aux_ch_ctl_reg = ilk_aux_ctl_reg;
                intel_dp->aux_ch_data_reg = ilk_aux_data_reg;
        } else if (display->platform.valleyview || display->platform.cherryview) {
@@ -815,7 +813,7 @@ void intel_dp_aux_init(struct intel_dp *intel_dp)
                intel_dp->get_aux_clock_divider = skl_get_aux_clock_divider;
        else if (display->platform.broadwell || display->platform.haswell)
                intel_dp->get_aux_clock_divider = hsw_get_aux_clock_divider;
-       else if (HAS_PCH_SPLIT(i915))
+       else if (HAS_PCH_SPLIT(display))
                intel_dp->get_aux_clock_divider = ilk_get_aux_clock_divider;
        else
                intel_dp->get_aux_clock_divider = g4x_get_aux_clock_divider;
index 0481b1365b85215d3adbd69f8b2ffe8ccc7530d6..a9e9b98d0bf9c9c6ee33b48ff9306003d64962da 100644 (file)
@@ -374,12 +374,11 @@ int chv_calc_dpll_params(int refclk, struct dpll *clock)
 static int i9xx_pll_refclk(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);
        const struct i9xx_dpll_hw_state *hw_state = &crtc_state->dpll_hw_state.i9xx;
 
        if ((hw_state->dpll & PLL_REF_INPUT_MASK) == PLLB_REF_INPUT_SPREADSPECTRUMIN)
                return display->vbt.lvds_ssc_freq;
-       else if (HAS_PCH_SPLIT(i915))
+       else if (HAS_PCH_SPLIT(display))
                return 120000;
        else if (DISPLAY_VER(display) != 2)
                return 96000;
@@ -1235,12 +1234,10 @@ static int mtl_crtc_compute_clock(struct intel_atomic_state *state,
 static int ilk_fb_cb_factor(const struct intel_crtc_state *crtc_state)
 {
        struct intel_display *display = to_intel_display(crtc_state);
-       struct intel_crtc *crtc = to_intel_crtc(crtc_state->uapi.crtc);
-       struct drm_i915_private *i915 = to_i915(crtc->base.dev);
 
        if (intel_crtc_has_type(crtc_state, INTEL_OUTPUT_LVDS) &&
            ((intel_panel_use_ssc(display) && display->vbt.lvds_ssc_freq == 100000) ||
-            (HAS_PCH_IBX(i915) && intel_is_dual_link_lvds(display))))
+            (HAS_PCH_IBX(display) && intel_is_dual_link_lvds(display))))
                return 25;
 
        if (crtc_state->sdvo_tv_clock)
@@ -1791,15 +1788,13 @@ int intel_dpll_crtc_get_shared_dpll(struct intel_atomic_state *state,
 void
 intel_dpll_init_clock_hook(struct intel_display *display)
 {
-       struct drm_i915_private *dev_priv = to_i915(display->drm);
-
        if (DISPLAY_VER(display) >= 14)
                display->funcs.dpll = &mtl_dpll_funcs;
        else if (display->platform.dg2)
                display->funcs.dpll = &dg2_dpll_funcs;
        else if (DISPLAY_VER(display) >= 9 || HAS_DDI(display))
                display->funcs.dpll = &hsw_dpll_funcs;
-       else if (HAS_PCH_SPLIT(dev_priv))
+       else if (HAS_PCH_SPLIT(display))
                display->funcs.dpll = &ilk_dpll_funcs;
        else if (display->platform.cherryview)
                display->funcs.dpll = &chv_dpll_funcs;
index 84df41086a892e1850ac3d7dd51ef5df4b3e3905..c2f08b2ee455bd8282b7f51d999c40faa74d2228 100644 (file)
@@ -609,13 +609,12 @@ static int ibx_get_dpll(struct intel_atomic_state *state,
                        struct intel_encoder *encoder)
 {
        struct intel_display *display = to_intel_display(state);
-       struct drm_i915_private *i915 = to_i915(display->drm);
        struct intel_crtc_state *crtc_state =
                intel_atomic_get_new_crtc_state(state, crtc);
        struct intel_shared_dpll *pll;
        enum intel_dpll_id id;
 
-       if (HAS_PCH_IBX(i915)) {
+       if (HAS_PCH_IBX(display)) {
                /* Ironlake PCH has a fixed PLL->PCH pipe mapping. */
                id = (enum intel_dpll_id) crtc->pipe;
                pll = intel_get_shared_dpll_by_id(display, id);
@@ -4305,7 +4304,6 @@ static const struct intel_dpll_mgr adlp_pll_mgr = {
  */
 void intel_shared_dpll_init(struct intel_display *display)
 {
-       struct drm_i915_private *i915 = to_i915(display->drm);
        const struct intel_dpll_mgr *dpll_mgr = NULL;
        const struct dpll_info *dpll_info;
        int i;
@@ -4335,7 +4333,7 @@ void intel_shared_dpll_init(struct intel_display *display)
                dpll_mgr = &skl_pll_mgr;
        else if (HAS_DDI(display))
                dpll_mgr = &hsw_pll_mgr;
-       else if (HAS_PCH_IBX(i915) || HAS_PCH_CPT(i915))
+       else if (HAS_PCH_IBX(display) || HAS_PCH_CPT(display))
                dpll_mgr = &pch_pll_mgr;
 
        if (!dpll_mgr)
index 40deee0769ae4f2059c9b59cc9422497fd7f9cf1..db68c20947abcbb310b22633029ae5947e083c70 100644 (file)
@@ -464,7 +464,6 @@ static void ivb_update_fdi_bc_bifurcation(const struct intel_crtc_state *crtc_st
 void intel_fdi_normal_train(struct intel_crtc *crtc)
 {
        struct intel_display *display = to_intel_display(crtc);
-       struct drm_i915_private *dev_priv = to_i915(display->drm);
        enum pipe pipe = crtc->pipe;
        i915_reg_t reg;
        u32 temp;
@@ -483,7 +482,7 @@ void intel_fdi_normal_train(struct intel_crtc *crtc)
 
        reg = FDI_RX_CTL(pipe);
        temp = intel_de_read(display, reg);
-       if (HAS_PCH_CPT(dev_priv)) {
+       if (HAS_PCH_CPT(display)) {
                temp &= ~FDI_LINK_TRAIN_PATTERN_MASK_CPT;
                temp |= FDI_LINK_TRAIN_NORMAL_CPT;
        } else {
@@ -607,7 +606,6 @@ static void gen6_fdi_link_train(struct intel_crtc *crtc,
                                const struct intel_crtc_state *crtc_state)
 {
        struct intel_display *display = to_intel_display(crtc);
-       struct drm_i915_private *dev_priv = to_i915(display->drm);
        enum pipe pipe = crtc->pipe;
        i915_reg_t reg;
        u32 temp, i, retry;
@@ -647,7 +645,7 @@ static void gen6_fdi_link_train(struct intel_crtc *crtc,
 
        reg = FDI_RX_CTL(pipe);
        temp = intel_de_read(display, reg);
-       if (HAS_PCH_CPT(dev_priv)) {
+       if (HAS_PCH_CPT(display)) {
                temp &= ~FDI_LINK_TRAIN_PATTERN_MASK_CPT;
                temp |= FDI_LINK_TRAIN_PATTERN_1_CPT;
        } else {
@@ -698,7 +696,7 @@ static void gen6_fdi_link_train(struct intel_crtc *crtc,
 
        reg = FDI_RX_CTL(pipe);
        temp = intel_de_read(display, reg);
-       if (HAS_PCH_CPT(dev_priv)) {
+       if (HAS_PCH_CPT(display)) {
                temp &= ~FDI_LINK_TRAIN_PATTERN_MASK_CPT;
                temp |= FDI_LINK_TRAIN_PATTERN_2_CPT;
        } else {
@@ -1077,7 +1075,6 @@ void ilk_fdi_pll_disable(struct intel_crtc *crtc)
 void ilk_fdi_disable(struct intel_crtc *crtc)
 {
        struct intel_display *display = to_intel_display(crtc);
-       struct drm_i915_private *dev_priv = to_i915(crtc->base.dev);
        enum pipe pipe = crtc->pipe;
        i915_reg_t reg;
        u32 temp;
@@ -1096,7 +1093,7 @@ void ilk_fdi_disable(struct intel_crtc *crtc)
        udelay(100);
 
        /* Ironlake workaround, disable clock pointer after downing FDI */
-       if (HAS_PCH_IBX(dev_priv))
+       if (HAS_PCH_IBX(display))
                intel_de_write(display, FDI_RX_CHICKEN(pipe),
                               FDI_RX_PHASE_SYNC_POINTER_OVR);
 
@@ -1106,7 +1103,7 @@ void ilk_fdi_disable(struct intel_crtc *crtc)
 
        reg = FDI_RX_CTL(pipe);
        temp = intel_de_read(display, reg);
-       if (HAS_PCH_CPT(dev_priv)) {
+       if (HAS_PCH_CPT(display)) {
                temp &= ~FDI_LINK_TRAIN_PATTERN_MASK_CPT;
                temp |= FDI_LINK_TRAIN_PATTERN_1_CPT;
        } else {
index 451cd26024f79ca5e40bb1ff69456b01c6f07ca3..7c7cd29b0944494cdd94972990f40d34843a4755 100644 (file)
@@ -346,7 +346,7 @@ bool intel_set_pch_fifo_underrun_reporting(struct intel_display *display,
        old = !crtc->pch_fifo_underrun_disabled;
        crtc->pch_fifo_underrun_disabled = !enable;
 
-       if (HAS_PCH_IBX(dev_priv))
+       if (HAS_PCH_IBX(display))
                ibx_set_fifo_underrun_reporting(display,
                                                pch_transcoder,
                                                enable);
@@ -459,7 +459,7 @@ void intel_check_pch_fifo_underruns(struct intel_display *display)
                if (crtc->pch_fifo_underrun_disabled)
                        continue;
 
-               if (HAS_PCH_CPT(dev_priv))
+               if (HAS_PCH_CPT(display))
                        cpt_check_pch_fifo_underruns(crtc);
        }
 
index 2a530bf35ebbf04cb6ed55ff40089530959dd77c..d55cc77650b797260a47e1f95b660001b462da21 100644 (file)
@@ -152,23 +152,22 @@ static const struct gmbus_pin gmbus_pins_mtp[] = {
 static const struct gmbus_pin *get_gmbus_pin(struct intel_display *display,
                                             unsigned int pin)
 {
-       struct drm_i915_private *i915 = to_i915(display->drm);
        const struct gmbus_pin *pins;
        size_t size;
 
-       if (INTEL_PCH_TYPE(i915) >= PCH_MTL) {
+       if (INTEL_PCH_TYPE(display) >= PCH_MTL) {
                pins = gmbus_pins_mtp;
                size = ARRAY_SIZE(gmbus_pins_mtp);
-       } else if (INTEL_PCH_TYPE(i915) >= PCH_DG2) {
+       } else if (INTEL_PCH_TYPE(display) >= PCH_DG2) {
                pins = gmbus_pins_dg2;
                size = ARRAY_SIZE(gmbus_pins_dg2);
-       } else if (INTEL_PCH_TYPE(i915) >= PCH_DG1) {
+       } else if (INTEL_PCH_TYPE(display) >= PCH_DG1) {
                pins = gmbus_pins_dg1;
                size = ARRAY_SIZE(gmbus_pins_dg1);
-       } else if (INTEL_PCH_TYPE(i915) >= PCH_ICP) {
+       } else if (INTEL_PCH_TYPE(display) >= PCH_ICP) {
                pins = gmbus_pins_icp;
                size = ARRAY_SIZE(gmbus_pins_icp);
-       } else if (HAS_PCH_CNP(i915)) {
+       } else if (HAS_PCH_CNP(display)) {
                pins = gmbus_pins_cnp;
                size = ARRAY_SIZE(gmbus_pins_cnp);
        } else if (display->platform.geminilake || display->platform.broxton) {
@@ -627,14 +626,13 @@ do_gmbus_xfer(struct i2c_adapter *adapter, struct i2c_msg *msgs, int num,
 {
        struct intel_gmbus *bus = to_intel_gmbus(adapter);
        struct intel_display *display = bus->display;
-       struct drm_i915_private *i915 = to_i915(display->drm);
        int i = 0, inc, try = 0;
        int ret = 0;
 
        /* Display WA #0868: skl,bxt,kbl,cfl,glk */
        if (display->platform.geminilake || display->platform.broxton)
                bxt_gmbus_clock_gating(display, false);
-       else if (HAS_PCH_SPT(i915) || HAS_PCH_CNP(i915))
+       else if (HAS_PCH_SPT(display) || HAS_PCH_CNP(display))
                pch_gmbus_clock_gating(display, false);
 
 retry:
@@ -747,7 +745,7 @@ out:
        /* Display WA #0868: skl,bxt,kbl,cfl,glk */
        if (display->platform.geminilake || display->platform.broxton)
                bxt_gmbus_clock_gating(display, true);
-       else if (HAS_PCH_SPT(i915) || HAS_PCH_CNP(i915))
+       else if (HAS_PCH_SPT(display) || HAS_PCH_CNP(display))
                pch_gmbus_clock_gating(display, true);
 
        return ret;
index 120c63dfdd02e65f712272bb2f0bf35b7114457c..e08c01e5b9d3c6580b426afc87afbf5f44a82f1e 100644 (file)
@@ -979,7 +979,6 @@ static bool intel_hdmi_set_gcp_infoframe(struct intel_encoder *encoder,
                                         const struct drm_connector_state *conn_state)
 {
        struct intel_display *display = to_intel_display(encoder);
-       struct drm_i915_private *dev_priv = to_i915(encoder->base.dev);
        struct intel_crtc *crtc = to_intel_crtc(crtc_state->uapi.crtc);
        i915_reg_t reg;
 
@@ -991,7 +990,7 @@ static bool intel_hdmi_set_gcp_infoframe(struct intel_encoder *encoder,
                reg = HSW_TVIDEO_DIP_GCP(display, crtc_state->cpu_transcoder);
        else if (display->platform.valleyview || display->platform.cherryview)
                reg = VLV_TVIDEO_DIP_GCP(crtc->pipe);
-       else if (HAS_PCH_SPLIT(dev_priv))
+       else if (HAS_PCH_SPLIT(display))
                reg = TVIDEO_DIP_GCP(crtc->pipe);
        else
                return false;
@@ -1005,7 +1004,6 @@ void intel_hdmi_read_gcp_infoframe(struct intel_encoder *encoder,
                                   struct intel_crtc_state *crtc_state)
 {
        struct intel_display *display = to_intel_display(encoder);
-       struct drm_i915_private *dev_priv = to_i915(encoder->base.dev);
        struct intel_crtc *crtc = to_intel_crtc(crtc_state->uapi.crtc);
        i915_reg_t reg;
 
@@ -1017,7 +1015,7 @@ void intel_hdmi_read_gcp_infoframe(struct intel_encoder *encoder,
                reg = HSW_TVIDEO_DIP_GCP(display, crtc_state->cpu_transcoder);
        else if (display->platform.valleyview || display->platform.cherryview)
                reg = VLV_TVIDEO_DIP_GCP(crtc->pipe);
-       else if (HAS_PCH_SPLIT(dev_priv))
+       else if (HAS_PCH_SPLIT(display))
                reg = TVIDEO_DIP_GCP(crtc->pipe);
        else
                return;
@@ -2807,7 +2805,7 @@ static u8 mcc_encoder_to_ddc_pin(struct intel_encoder *encoder)
 
 static u8 rkl_encoder_to_ddc_pin(struct intel_encoder *encoder)
 {
-       struct drm_i915_private *dev_priv = to_i915(encoder->base.dev);
+       struct intel_display *display = to_intel_display(encoder);
        enum phy phy = intel_encoder_to_phy(encoder);
 
        WARN_ON(encoder->port == PORT_C);
@@ -2818,7 +2816,7 @@ static u8 rkl_encoder_to_ddc_pin(struct intel_encoder *encoder)
         * combo outputs.  With CMP, the traditional DDI A-D pins are used for
         * all outputs.
         */
-       if (INTEL_PCH_TYPE(dev_priv) >= PCH_TGP && phy >= PHY_C)
+       if (INTEL_PCH_TYPE(display) >= PCH_TGP && phy >= PHY_C)
                return GMBUS_PIN_9_TC1_ICP + phy - PHY_C;
 
        return GMBUS_PIN_1_BXT + phy;
@@ -2827,7 +2825,6 @@ static u8 rkl_encoder_to_ddc_pin(struct intel_encoder *encoder)
 static u8 gen9bc_tgp_encoder_to_ddc_pin(struct intel_encoder *encoder)
 {
        struct intel_display *display = to_intel_display(encoder);
-       struct drm_i915_private *i915 = to_i915(encoder->base.dev);
        enum phy phy = intel_encoder_to_phy(encoder);
 
        drm_WARN_ON(display->drm, encoder->port == PORT_A);
@@ -2838,7 +2835,7 @@ static u8 gen9bc_tgp_encoder_to_ddc_pin(struct intel_encoder *encoder)
         * combo outputs.  With CMP, the traditional DDI A-D pins are used for
         * all outputs.
         */
-       if (INTEL_PCH_TYPE(i915) >= PCH_TGP && phy >= PHY_C)
+       if (INTEL_PCH_TYPE(display) >= PCH_TGP && phy >= PHY_C)
                return GMBUS_PIN_9_TC1_ICP + phy - PHY_C;
 
        return GMBUS_PIN_1_BXT + phy;
@@ -2891,23 +2888,22 @@ static u8 g4x_encoder_to_ddc_pin(struct intel_encoder *encoder)
 static u8 intel_hdmi_default_ddc_pin(struct intel_encoder *encoder)
 {
        struct intel_display *display = to_intel_display(encoder);
-       struct drm_i915_private *dev_priv = to_i915(encoder->base.dev);
        u8 ddc_pin;
 
        if (display->platform.alderlake_s)
                ddc_pin = adls_encoder_to_ddc_pin(encoder);
-       else if (INTEL_PCH_TYPE(dev_priv) >= PCH_DG1)
+       else if (INTEL_PCH_TYPE(display) >= PCH_DG1)
                ddc_pin = dg1_encoder_to_ddc_pin(encoder);
        else if (display->platform.rocketlake)
                ddc_pin = rkl_encoder_to_ddc_pin(encoder);
-       else if (DISPLAY_VER(display) == 9 && HAS_PCH_TGP(dev_priv))
+       else if (DISPLAY_VER(display) == 9 && HAS_PCH_TGP(display))
                ddc_pin = gen9bc_tgp_encoder_to_ddc_pin(encoder);
        else if ((display->platform.jasperlake || display->platform.elkhartlake) &&
-                HAS_PCH_TGP(dev_priv))
+                HAS_PCH_TGP(display))
                ddc_pin = mcc_encoder_to_ddc_pin(encoder);
-       else if (INTEL_PCH_TYPE(dev_priv) >= PCH_ICP)
+       else if (INTEL_PCH_TYPE(display) >= PCH_ICP)
                ddc_pin = icl_encoder_to_ddc_pin(encoder);
-       else if (HAS_PCH_CNP(dev_priv))
+       else if (HAS_PCH_CNP(display))
                ddc_pin = cnp_encoder_to_ddc_pin(encoder);
        else if (display->platform.geminilake || display->platform.broxton)
                ddc_pin = bxt_encoder_to_ddc_pin(encoder);
@@ -2985,8 +2981,6 @@ static u8 intel_hdmi_ddc_pin(struct intel_encoder *encoder)
 void intel_infoframe_init(struct intel_digital_port *dig_port)
 {
        struct intel_display *display = to_intel_display(dig_port);
-       struct drm_i915_private *dev_priv =
-               to_i915(dig_port->base.base.dev);
 
        if (display->platform.valleyview || display->platform.cherryview) {
                dig_port->write_infoframe = vlv_write_infoframe;
@@ -3010,7 +3004,7 @@ void intel_infoframe_init(struct intel_digital_port *dig_port)
                        dig_port->set_infoframes = hsw_set_infoframes;
                        dig_port->infoframes_enabled = hsw_infoframes_enabled;
                }
-       } else if (HAS_PCH_IBX(dev_priv)) {
+       } else if (HAS_PCH_IBX(display)) {
                dig_port->write_infoframe = ibx_write_infoframe;
                dig_port->read_infoframe = ibx_read_infoframe;
                dig_port->set_infoframes = ibx_set_infoframes;
index 2463e61e7802b2a7b36f1fe3ae7162ec5630187c..c841399e5c8898ff1c8f038e9532180675da84c3 100644 (file)
@@ -133,7 +133,6 @@ static const u32 hpd_mtp[HPD_NUM_PINS] = {
 
 static void intel_hpd_init_pins(struct intel_display *display)
 {
-       struct drm_i915_private *dev_priv = to_i915(display->drm);
        struct intel_hotplug *hpd = &display->hotplug;
 
        if (HAS_GMCH(display)) {
@@ -160,24 +159,24 @@ static void intel_hpd_init_pins(struct intel_display *display)
        else
                hpd->hpd = hpd_ilk;
 
-       if ((INTEL_PCH_TYPE(dev_priv) < PCH_DG1) &&
-           (!HAS_PCH_SPLIT(dev_priv) || HAS_PCH_NOP(dev_priv)))
+       if ((INTEL_PCH_TYPE(display) < PCH_DG1) &&
+           (!HAS_PCH_SPLIT(display) || HAS_PCH_NOP(display)))
                return;
 
-       if (INTEL_PCH_TYPE(dev_priv) >= PCH_MTL)
+       if (INTEL_PCH_TYPE(display) >= PCH_MTL)
                hpd->pch_hpd = hpd_mtp;
-       else if (INTEL_PCH_TYPE(dev_priv) >= PCH_DG1)
+       else if (INTEL_PCH_TYPE(display) >= PCH_DG1)
                hpd->pch_hpd = hpd_sde_dg1;
-       else if (INTEL_PCH_TYPE(dev_priv) >= PCH_ICP)
+       else if (INTEL_PCH_TYPE(display) >= PCH_ICP)
                hpd->pch_hpd = hpd_icp;
-       else if (HAS_PCH_CNP(dev_priv) || HAS_PCH_SPT(dev_priv))
+       else if (HAS_PCH_CNP(display) || HAS_PCH_SPT(display))
                hpd->pch_hpd = hpd_spt;
-       else if (HAS_PCH_LPT(dev_priv) || HAS_PCH_CPT(dev_priv))
+       else if (HAS_PCH_LPT(display) || HAS_PCH_CPT(display))
                hpd->pch_hpd = hpd_cpt;
-       else if (HAS_PCH_IBX(dev_priv))
+       else if (HAS_PCH_IBX(display))
                hpd->pch_hpd = hpd_ibx;
        else
-               MISSING_CASE(INTEL_PCH_TYPE(dev_priv));
+               MISSING_CASE(INTEL_PCH_TYPE(display));
 }
 
 /* For display hotplug interrupt */
@@ -711,7 +710,7 @@ static u32 ibx_hotplug_mask(enum hpd_pin hpd_pin)
 
 static u32 ibx_hotplug_enables(struct intel_encoder *encoder)
 {
-       struct drm_i915_private *i915 = to_i915(encoder->base.dev);
+       struct intel_display *display = to_intel_display(encoder);
 
        switch (encoder->hpd_pin) {
        case HPD_PORT_A:
@@ -719,7 +718,7 @@ static u32 ibx_hotplug_enables(struct intel_encoder *encoder)
                 * When CPU and PCH are on the same package, port A
                 * HPD must be enabled in both north and south.
                 */
-               return HAS_PCH_LPT_LP(i915) ?
+               return HAS_PCH_LPT_LP(display) ?
                        PORTA_HOTPLUG_ENABLE : 0;
        case HPD_PORT_B:
                return PORTB_HOTPLUG_ENABLE |
@@ -940,18 +939,17 @@ static void gen11_tbt_hpd_enable_detection(struct intel_encoder *encoder)
 
 static void gen11_hpd_enable_detection(struct intel_encoder *encoder)
 {
-       struct drm_i915_private *i915 = to_i915(encoder->base.dev);
+       struct intel_display *display = to_intel_display(encoder);
 
        gen11_tc_hpd_enable_detection(encoder);
        gen11_tbt_hpd_enable_detection(encoder);
 
-       if (INTEL_PCH_TYPE(i915) >= PCH_ICP)
+       if (INTEL_PCH_TYPE(display) >= PCH_ICP)
                icp_hpd_enable_detection(encoder);
 }
 
 static void gen11_hpd_irq_setup(struct intel_display *display)
 {
-       struct drm_i915_private *dev_priv = to_i915(display->drm);
        u32 hotplug_irqs, enabled_irqs;
 
        enabled_irqs = intel_hpd_enabled_irqs(display, display->hotplug.hpd);
@@ -964,7 +962,7 @@ static void gen11_hpd_irq_setup(struct intel_display *display)
        gen11_tc_hpd_detection_setup(display);
        gen11_tbt_hpd_detection_setup(display);
 
-       if (INTEL_PCH_TYPE(dev_priv) >= PCH_ICP)
+       if (INTEL_PCH_TYPE(display) >= PCH_ICP)
                icp_hpd_irq_setup(display);
 }
 
@@ -1138,7 +1136,6 @@ static void xelpdp_hpd_enable_detection(struct intel_encoder *encoder)
 
 static void xelpdp_hpd_irq_setup(struct intel_display *display)
 {
-       struct drm_i915_private *i915 = to_i915(display->drm);
        u32 hotplug_irqs, enabled_irqs;
 
        enabled_irqs = intel_hpd_enabled_irqs(display, display->hotplug.hpd);
@@ -1150,9 +1147,9 @@ static void xelpdp_hpd_irq_setup(struct intel_display *display)
 
        xelpdp_pica_hpd_detection_setup(display);
 
-       if (INTEL_PCH_TYPE(i915) >= PCH_LNL)
+       if (INTEL_PCH_TYPE(display) >= PCH_LNL)
                xe2lpd_sde_hpd_irq_setup(display);
-       else if (INTEL_PCH_TYPE(i915) >= PCH_MTL)
+       else if (INTEL_PCH_TYPE(display) >= PCH_MTL)
                mtp_hpd_irq_setup(display);
 }
 
@@ -1194,10 +1191,8 @@ static u32 spt_hotplug2_enables(struct intel_encoder *encoder)
 
 static void spt_hpd_detection_setup(struct intel_display *display)
 {
-       struct drm_i915_private *dev_priv = to_i915(display->drm);
-
        /* Display WA #1179 WaHardHangonHotPlug: cnp */
-       if (HAS_PCH_CNP(dev_priv)) {
+       if (HAS_PCH_CNP(display)) {
                intel_de_rmw(display, SOUTH_CHICKEN1, CHASSIS_CLK_REQ_DURATION_MASK,
                             CHASSIS_CLK_REQ_DURATION(0xf));
        }
@@ -1215,10 +1210,9 @@ static void spt_hpd_detection_setup(struct intel_display *display)
 static void spt_hpd_enable_detection(struct intel_encoder *encoder)
 {
        struct intel_display *display = to_intel_display(encoder);
-       struct drm_i915_private *i915 = to_i915(encoder->base.dev);
 
        /* Display WA #1179 WaHardHangonHotPlug: cnp */
-       if (HAS_PCH_CNP(i915)) {
+       if (HAS_PCH_CNP(display)) {
                intel_de_rmw(display, SOUTH_CHICKEN1,
                             CHASSIS_CLK_REQ_DURATION_MASK,
                             CHASSIS_CLK_REQ_DURATION(0xf));
@@ -1235,10 +1229,9 @@ static void spt_hpd_enable_detection(struct intel_encoder *encoder)
 
 static void spt_hpd_irq_setup(struct intel_display *display)
 {
-       struct drm_i915_private *dev_priv = to_i915(display->drm);
        u32 hotplug_irqs, enabled_irqs;
 
-       if (INTEL_PCH_TYPE(dev_priv) >= PCH_CNP)
+       if (INTEL_PCH_TYPE(display) >= PCH_CNP)
                intel_de_write(display, SHPD_FILTER_CNT, SHPD_FILTER_CNT_500_ADJ);
 
        enabled_irqs = intel_hpd_enabled_irqs(display, display->hotplug.pch_hpd);
@@ -1474,8 +1467,6 @@ void intel_hpd_irq_setup(struct intel_display *display)
 
 void intel_hotplug_irq_init(struct intel_display *display)
 {
-       struct drm_i915_private *i915 = to_i915(display->drm);
-
        intel_hpd_init_pins(display);
 
        intel_hpd_init_early(display);
@@ -1484,9 +1475,9 @@ void intel_hotplug_irq_init(struct intel_display *display)
                if (HAS_HOTPLUG(display))
                        display->funcs.hotplug = &i915_hpd_funcs;
        } else {
-               if (HAS_PCH_DG2(i915))
+               if (HAS_PCH_DG2(display))
                        display->funcs.hotplug = &icp_hpd_funcs;
-               else if (HAS_PCH_DG1(i915))
+               else if (HAS_PCH_DG1(display))
                        display->funcs.hotplug = &dg1_hpd_funcs;
                else if (DISPLAY_VER(display) >= 14)
                        display->funcs.hotplug = &xelpdp_hpd_funcs;
@@ -1494,9 +1485,9 @@ void intel_hotplug_irq_init(struct intel_display *display)
                        display->funcs.hotplug = &gen11_hpd_funcs;
                else if (display->platform.geminilake || display->platform.broxton)
                        display->funcs.hotplug = &bxt_hpd_funcs;
-               else if (INTEL_PCH_TYPE(i915) >= PCH_ICP)
+               else if (INTEL_PCH_TYPE(display) >= PCH_ICP)
                        display->funcs.hotplug = &icp_hpd_funcs;
-               else if (INTEL_PCH_TYPE(i915) >= PCH_SPT)
+               else if (INTEL_PCH_TYPE(display) >= PCH_SPT)
                        display->funcs.hotplug = &spt_hpd_funcs;
                else
                        display->funcs.hotplug = &ilk_hpd_funcs;
index 89d26913e2539652171e8accfa29651326c80f01..495f81cf5667c15b0fbcebdc6324b8ecb598bb18 100644 (file)
@@ -87,13 +87,12 @@ static struct intel_lvds_encoder *to_lvds_encoder(struct intel_encoder *encoder)
 bool intel_lvds_port_enabled(struct intel_display *display,
                             i915_reg_t lvds_reg, enum pipe *pipe)
 {
-       struct drm_i915_private *i915 = to_i915(display->drm);
        u32 val;
 
        val = intel_de_read(display, lvds_reg);
 
        /* asserts want to know the pipe even if the port is disabled */
-       if (HAS_PCH_CPT(i915))
+       if (HAS_PCH_CPT(display))
                *pipe = REG_FIELD_GET(LVDS_PIPE_SEL_MASK_CPT, val);
        else
                *pipe = REG_FIELD_GET(LVDS_PIPE_SEL_MASK, val);
@@ -243,13 +242,12 @@ static void intel_pre_enable_lvds(struct intel_atomic_state *state,
 {
        struct intel_display *display = to_intel_display(state);
        struct intel_lvds_encoder *lvds_encoder = to_lvds_encoder(encoder);
-       struct drm_i915_private *i915 = to_i915(encoder->base.dev);
        struct intel_crtc *crtc = to_intel_crtc(crtc_state->uapi.crtc);
        const struct drm_display_mode *adjusted_mode = &crtc_state->hw.adjusted_mode;
        enum pipe pipe = crtc->pipe;
        u32 temp;
 
-       if (HAS_PCH_SPLIT(i915)) {
+       if (HAS_PCH_SPLIT(display)) {
                assert_fdi_rx_pll_disabled(display, pipe);
                assert_shared_dpll_disabled(display, crtc_state->shared_dpll);
        } else {
@@ -261,7 +259,7 @@ static void intel_pre_enable_lvds(struct intel_atomic_state *state,
        temp = lvds_encoder->init_lvds_val;
        temp |= LVDS_PORT_EN | LVDS_A0A2_CLKA_POWER_UP;
 
-       if (HAS_PCH_CPT(i915)) {
+       if (HAS_PCH_CPT(display)) {
                temp &= ~LVDS_PIPE_SEL_MASK_CPT;
                temp |= LVDS_PIPE_SEL_CPT(pipe);
        } else {
@@ -421,7 +419,6 @@ static int intel_lvds_compute_config(struct intel_encoder *encoder,
                                     struct drm_connector_state *conn_state)
 {
        struct intel_display *display = to_intel_display(encoder);
-       struct drm_i915_private *i915 = to_i915(encoder->base.dev);
        struct intel_lvds_encoder *lvds_encoder = to_lvds_encoder(encoder);
        struct intel_connector *connector = lvds_encoder->attached_connector;
        struct drm_display_mode *adjusted_mode = &crtc_state->hw.adjusted_mode;
@@ -435,7 +432,7 @@ static int intel_lvds_compute_config(struct intel_encoder *encoder,
                return -EINVAL;
        }
 
-       if (HAS_PCH_SPLIT(i915)) {
+       if (HAS_PCH_SPLIT(display)) {
                crtc_state->has_pch_encoder = true;
                if (!intel_fdi_compute_pipe_bpp(crtc_state))
                        return -EINVAL;
@@ -798,7 +795,6 @@ bool intel_is_dual_link_lvds(struct intel_display *display)
 static bool compute_is_dual_link_lvds(struct intel_lvds_encoder *lvds_encoder)
 {
        struct intel_display *display = to_intel_display(&lvds_encoder->base);
-       struct drm_i915_private *i915 = to_i915(lvds_encoder->base.base.dev);
        struct intel_connector *connector = lvds_encoder->attached_connector;
        const struct drm_display_mode *fixed_mode =
                intel_panel_preferred_fixed_mode(connector);
@@ -822,7 +818,7 @@ static bool compute_is_dual_link_lvds(struct intel_lvds_encoder *lvds_encoder)
         * register is uninitialized.
         */
        val = intel_de_read(display, lvds_encoder->reg);
-       if (HAS_PCH_CPT(i915))
+       if (HAS_PCH_CPT(display))
                val &= ~(LVDS_DETECTED | LVDS_PIPE_SEL_MASK_CPT);
        else
                val &= ~(LVDS_DETECTED | LVDS_PIPE_SEL_MASK);
@@ -846,7 +842,6 @@ static void intel_lvds_add_properties(struct drm_connector *connector)
  */
 void intel_lvds_init(struct intel_display *display)
 {
-       struct drm_i915_private *i915 = to_i915(display->drm);
        struct intel_lvds_encoder *lvds_encoder;
        struct intel_connector *connector;
        const struct drm_edid *drm_edid;
@@ -868,14 +863,14 @@ void intel_lvds_init(struct intel_display *display)
                return;
        }
 
-       if (HAS_PCH_SPLIT(i915))
+       if (HAS_PCH_SPLIT(display))
                lvds_reg = PCH_LVDS;
        else
                lvds_reg = LVDS;
 
        lvds = intel_de_read(display, lvds_reg);
 
-       if (HAS_PCH_SPLIT(i915)) {
+       if (HAS_PCH_SPLIT(display)) {
                if ((lvds & LVDS_DETECTED) == 0)
                        return;
        }
@@ -915,7 +910,7 @@ void intel_lvds_init(struct intel_display *display)
        encoder->enable = intel_enable_lvds;
        encoder->pre_enable = intel_pre_enable_lvds;
        encoder->compute_config = intel_lvds_compute_config;
-       if (HAS_PCH_SPLIT(i915)) {
+       if (HAS_PCH_SPLIT(display)) {
                encoder->disable = pch_disable_lvds;
                encoder->post_disable = pch_post_disable_lvds;
        } else {
index b909ed18a5b2b694749b4b953ca5a881fa603028..de77d6487d797c010f8cabcf9c721ad8b89299ec 100644 (file)
 bool intel_has_pch_trancoder(struct intel_display *display,
                             enum pipe pch_transcoder)
 {
-       struct drm_i915_private *i915 = to_i915(display->drm);
-
-       return HAS_PCH_IBX(i915) || HAS_PCH_CPT(i915) ||
-               (HAS_PCH_LPT_H(i915) && pch_transcoder == PIPE_A);
+       return HAS_PCH_IBX(display) || HAS_PCH_CPT(display) ||
+               (HAS_PCH_LPT_H(display) && pch_transcoder == PIPE_A);
 }
 
 enum pipe intel_crtc_pch_transcoder(struct intel_crtc *crtc)
 {
-       struct drm_i915_private *i915 = to_i915(crtc->base.dev);
+       struct intel_display *display = to_intel_display(crtc);
 
-       if (HAS_PCH_LPT(i915))
+       if (HAS_PCH_LPT(display))
                return PIPE_A;
        else
                return crtc->pipe;
@@ -43,7 +41,6 @@ static void assert_pch_dp_disabled(struct intel_display *display,
                                   enum pipe pipe, enum port port,
                                   i915_reg_t dp_reg)
 {
-       struct drm_i915_private *dev_priv = to_i915(display->drm);
        enum pipe port_pipe;
        bool state;
 
@@ -54,7 +51,7 @@ static void assert_pch_dp_disabled(struct intel_display *display,
                                 port_name(port), pipe_name(pipe));
 
        INTEL_DISPLAY_STATE_WARN(display,
-                                HAS_PCH_IBX(dev_priv) && !state && port_pipe == PIPE_B,
+                                HAS_PCH_IBX(display) && !state && port_pipe == PIPE_B,
                                 "IBX PCH DP %c still using transcoder B\n",
                                 port_name(port));
 }
@@ -63,7 +60,6 @@ static void assert_pch_hdmi_disabled(struct intel_display *display,
                                     enum pipe pipe, enum port port,
                                     i915_reg_t hdmi_reg)
 {
-       struct drm_i915_private *dev_priv = to_i915(display->drm);
        enum pipe port_pipe;
        bool state;
 
@@ -74,7 +70,7 @@ static void assert_pch_hdmi_disabled(struct intel_display *display,
                                 port_name(port), pipe_name(pipe));
 
        INTEL_DISPLAY_STATE_WARN(display,
-                                HAS_PCH_IBX(dev_priv) && !state && port_pipe == PIPE_B,
+                                HAS_PCH_IBX(display) && !state && port_pipe == PIPE_B,
                                 "IBX PCH HDMI %c still using transcoder B\n",
                                 port_name(port));
 }
@@ -249,7 +245,6 @@ static void ilk_enable_pch_transcoder(const struct intel_crtc_state *crtc_state)
 {
        struct intel_display *display = to_intel_display(crtc_state);
        struct intel_crtc *crtc = to_intel_crtc(crtc_state->uapi.crtc);
-       struct drm_i915_private *dev_priv = to_i915(crtc->base.dev);
        enum pipe pipe = crtc->pipe;
        i915_reg_t reg;
        u32 val, pipeconf_val;
@@ -261,7 +256,7 @@ static void ilk_enable_pch_transcoder(const struct intel_crtc_state *crtc_state)
        assert_fdi_tx_enabled(display, pipe);
        assert_fdi_rx_enabled(display, pipe);
 
-       if (HAS_PCH_CPT(dev_priv)) {
+       if (HAS_PCH_CPT(display)) {
                reg = TRANS_CHICKEN2(pipe);
                val = intel_de_read(display, reg);
                /*
@@ -279,7 +274,7 @@ static void ilk_enable_pch_transcoder(const struct intel_crtc_state *crtc_state)
        val = intel_de_read(display, reg);
        pipeconf_val = intel_de_read(display, TRANSCONF(display, pipe));
 
-       if (HAS_PCH_IBX(dev_priv)) {
+       if (HAS_PCH_IBX(display)) {
                /* Configure frame start delay to match the CPU */
                val &= ~TRANS_FRAME_START_DELAY_MASK;
                val |= TRANS_FRAME_START_DELAY(crtc_state->framestart_delay - 1);
@@ -298,7 +293,7 @@ static void ilk_enable_pch_transcoder(const struct intel_crtc_state *crtc_state)
 
        val &= ~TRANS_INTERLACE_MASK;
        if ((pipeconf_val & TRANSCONF_INTERLACE_MASK_ILK) == TRANSCONF_INTERLACE_IF_ID_ILK) {
-               if (HAS_PCH_IBX(dev_priv) &&
+               if (HAS_PCH_IBX(display) &&
                    intel_crtc_has_type(crtc_state, INTEL_OUTPUT_SDVO))
                        val |= TRANS_INTERLACE_LEGACY_VSYNC_IBX;
                else
@@ -316,7 +311,6 @@ static void ilk_enable_pch_transcoder(const struct intel_crtc_state *crtc_state)
 static void ilk_disable_pch_transcoder(struct intel_crtc *crtc)
 {
        struct intel_display *display = to_intel_display(crtc);
-       struct drm_i915_private *dev_priv = to_i915(crtc->base.dev);
        enum pipe pipe = crtc->pipe;
        i915_reg_t reg;
 
@@ -334,7 +328,7 @@ static void ilk_disable_pch_transcoder(struct intel_crtc *crtc)
                drm_err(display->drm, "failed to disable transcoder %c\n",
                        pipe_name(pipe));
 
-       if (HAS_PCH_CPT(dev_priv))
+       if (HAS_PCH_CPT(display))
                /* Workaround: Clear the timing override chicken bit again. */
                intel_de_rmw(display, TRANS_CHICKEN2(pipe),
                             TRANS_CHICKEN2_TIMING_OVERRIDE, 0);
@@ -366,7 +360,6 @@ void ilk_pch_enable(struct intel_atomic_state *state,
                    struct intel_crtc *crtc)
 {
        struct intel_display *display = to_intel_display(crtc);
-       struct drm_i915_private *dev_priv = to_i915(crtc->base.dev);
        const struct intel_crtc_state *crtc_state =
                intel_atomic_get_new_crtc_state(state, crtc);
        enum pipe pipe = crtc->pipe;
@@ -381,7 +374,7 @@ void ilk_pch_enable(struct intel_atomic_state *state,
         * We need to program the right clock selection
         * before writing the pixel multiplier into the DPLL.
         */
-       if (HAS_PCH_CPT(dev_priv)) {
+       if (HAS_PCH_CPT(display)) {
                u32 sel;
 
                temp = intel_de_read(display, PCH_DPLL_SEL);
@@ -417,7 +410,7 @@ void ilk_pch_enable(struct intel_atomic_state *state,
        intel_fdi_normal_train(crtc);
 
        /* For PCH DP, enable TRANS_DP_CTL */
-       if (HAS_PCH_CPT(dev_priv) &&
+       if (HAS_PCH_CPT(display) &&
            intel_crtc_has_dp_encoder(crtc_state)) {
                const struct drm_display_mode *adjusted_mode =
                        &crtc_state->hw.adjusted_mode;
@@ -459,14 +452,13 @@ void ilk_pch_post_disable(struct intel_atomic_state *state,
                          struct intel_crtc *crtc)
 {
        struct intel_display *display = to_intel_display(crtc);
-       struct drm_i915_private *dev_priv = to_i915(crtc->base.dev);
        const struct intel_crtc_state *old_crtc_state =
                intel_atomic_get_old_crtc_state(state, crtc);
        enum pipe pipe = crtc->pipe;
 
        ilk_disable_pch_transcoder(crtc);
 
-       if (HAS_PCH_CPT(dev_priv)) {
+       if (HAS_PCH_CPT(display)) {
                /* disable TRANS_DP_CTL */
                intel_de_rmw(display, TRANS_DP_CTL(pipe),
                             TRANS_DP_OUTPUT_ENABLE | TRANS_DP_PORT_SEL_MASK,
@@ -503,7 +495,6 @@ void ilk_pch_get_config(struct intel_crtc_state *crtc_state)
 {
        struct intel_display *display = to_intel_display(crtc_state);
        struct intel_crtc *crtc = to_intel_crtc(crtc_state->uapi.crtc);
-       struct drm_i915_private *dev_priv = to_i915(crtc->base.dev);
        struct intel_shared_dpll *pll;
        enum pipe pipe = crtc->pipe;
        enum intel_dpll_id pll_id;
@@ -522,7 +513,7 @@ void ilk_pch_get_config(struct intel_crtc_state *crtc_state)
        intel_cpu_transcoder_get_m1_n1(crtc, crtc_state->cpu_transcoder,
                                       &crtc_state->fdi_m_n);
 
-       if (HAS_PCH_IBX(dev_priv)) {
+       if (HAS_PCH_IBX(display)) {
                /*
                 * The pipe->pch transcoder and pch transcoder->pll
                 * mapping is fixed.
@@ -646,8 +637,6 @@ void lpt_pch_get_config(struct intel_crtc_state *crtc_state)
 
 void intel_pch_sanitize(struct intel_display *display)
 {
-       struct drm_i915_private *i915 = to_i915(display->drm);
-
-       if (HAS_PCH_IBX(i915))
+       if (HAS_PCH_IBX(display))
                ibx_sanitize_pch_ports(display);
 }
index 1307a478861aec3b4a2df1c4941bb231a8e676a6..693b90e3dfc3cfed83ab0e697b330584c2f2c20b 100644 (file)
@@ -281,7 +281,7 @@ static void lpt_enable_clkout_dp(struct intel_display *display,
        if (drm_WARN(display->drm, with_fdi && !with_spread,
                     "FDI requires downspread\n"))
                with_spread = true;
-       if (drm_WARN(display->drm, HAS_PCH_LPT_LP(dev_priv) &&
+       if (drm_WARN(display->drm, HAS_PCH_LPT_LP(display) &&
                     with_fdi, "LP PCH doesn't have FDI\n"))
                with_fdi = false;
 
@@ -303,7 +303,7 @@ static void lpt_enable_clkout_dp(struct intel_display *display,
                        lpt_fdi_program_mphy(display);
        }
 
-       reg = HAS_PCH_LPT_LP(dev_priv) ? SBI_GEN0 : SBI_DBUFF0;
+       reg = HAS_PCH_LPT_LP(display) ? SBI_GEN0 : SBI_DBUFF0;
        tmp = intel_sbi_read(dev_priv, reg, SBI_ICLK);
        tmp |= SBI_GEN0_CFG_BUFFENABLE_DISABLE;
        intel_sbi_write(dev_priv, reg, tmp, SBI_ICLK);
@@ -319,7 +319,7 @@ void lpt_disable_clkout_dp(struct intel_display *display)
 
        intel_sbi_lock(dev_priv);
 
-       reg = HAS_PCH_LPT_LP(dev_priv) ? SBI_GEN0 : SBI_DBUFF0;
+       reg = HAS_PCH_LPT_LP(display) ? SBI_GEN0 : SBI_DBUFF0;
        tmp = intel_sbi_read(dev_priv, reg, SBI_ICLK);
        tmp &= ~SBI_GEN0_CFG_BUFFENABLE_DISABLE;
        intel_sbi_write(dev_priv, reg, tmp, SBI_ICLK);
@@ -498,7 +498,6 @@ static void lpt_init_pch_refclk(struct intel_display *display)
 
 static void ilk_init_pch_refclk(struct intel_display *display)
 {
-       struct drm_i915_private *dev_priv = to_i915(display->drm);
        struct intel_encoder *encoder;
        struct intel_shared_dpll *pll;
        int i;
@@ -527,7 +526,7 @@ static void ilk_init_pch_refclk(struct intel_display *display)
                }
        }
 
-       if (HAS_PCH_IBX(dev_priv)) {
+       if (HAS_PCH_IBX(display)) {
                has_ck505 = display->vbt.display_clock_mode;
                can_ssc = has_ck505;
        } else {
@@ -678,10 +677,8 @@ static void ilk_init_pch_refclk(struct intel_display *display)
  */
 void intel_init_pch_refclk(struct intel_display *display)
 {
-       struct drm_i915_private *dev_priv = to_i915(display->drm);
-
-       if (HAS_PCH_IBX(dev_priv) || HAS_PCH_CPT(dev_priv))
+       if (HAS_PCH_IBX(display) || HAS_PCH_CPT(display))
                ilk_init_pch_refclk(display);
-       else if (HAS_PCH_LPT(dev_priv))
+       else if (HAS_PCH_LPT(display))
                lpt_init_pch_refclk(display);
 }
index 4d4e2b9f5f2dc37761bab615692ed47af1050507..05e1e5c7e8b7cc88cb4f1213293939f1f7c88761 100644 (file)
@@ -350,21 +350,19 @@ vlv_initial_power_sequencer_setup(struct intel_dp *intel_dp)
 
 static int intel_num_pps(struct intel_display *display)
 {
-       struct drm_i915_private *i915 = to_i915(display->drm);
-
        if (display->platform.valleyview || display->platform.cherryview)
                return 2;
 
        if (display->platform.geminilake || display->platform.broxton)
                return 2;
 
-       if (INTEL_PCH_TYPE(i915) >= PCH_MTL)
+       if (INTEL_PCH_TYPE(display) >= PCH_MTL)
                return 2;
 
-       if (INTEL_PCH_TYPE(i915) >= PCH_DG1)
+       if (INTEL_PCH_TYPE(display) >= PCH_DG1)
                return 1;
 
-       if (INTEL_PCH_TYPE(i915) >= PCH_ICP)
+       if (INTEL_PCH_TYPE(display) >= PCH_ICP)
                return 2;
 
        return 1;
@@ -373,11 +371,10 @@ static int intel_num_pps(struct intel_display *display)
 static bool intel_pps_is_valid(struct intel_dp *intel_dp)
 {
        struct intel_display *display = to_intel_display(intel_dp);
-       struct drm_i915_private *i915 = to_i915(display->drm);
 
        if (intel_dp->pps.pps_idx == 1 &&
-           INTEL_PCH_TYPE(i915) >= PCH_ICP &&
-           INTEL_PCH_TYPE(i915) <= PCH_ADP)
+           INTEL_PCH_TYPE(display) >= PCH_ICP &&
+           INTEL_PCH_TYPE(display) <= PCH_ADP)
                return intel_de_read(display, SOUTH_CHICKEN1) & ICP_SECOND_PPS_IO_SELECT;
 
        return true;
@@ -499,7 +496,6 @@ static void intel_pps_get_registers(struct intel_dp *intel_dp,
                                    struct pps_registers *regs)
 {
        struct intel_display *display = to_intel_display(intel_dp);
-       struct drm_i915_private *dev_priv = to_i915(display->drm);
        int pps_idx;
 
        memset(regs, 0, sizeof(*regs));
@@ -518,7 +514,7 @@ static void intel_pps_get_registers(struct intel_dp *intel_dp,
 
        /* Cycle delay moved from PP_DIVISOR to PP_CONTROL */
        if (display->platform.geminilake || display->platform.broxton ||
-           INTEL_PCH_TYPE(dev_priv) >= PCH_CNP)
+           INTEL_PCH_TYPE(display) >= PCH_CNP)
                regs->pp_div = INVALID_MMIO_REG;
        else
                regs->pp_div = PP_DIVISOR(display, pps_idx);
@@ -1591,7 +1587,6 @@ static void pps_init_delays(struct intel_dp *intel_dp)
 static void pps_init_registers(struct intel_dp *intel_dp, bool force_disable_vdd)
 {
        struct intel_display *display = to_intel_display(intel_dp);
-       struct drm_i915_private *dev_priv = to_i915(display->drm);
        u32 pp_on, pp_off, port_sel = 0;
        int div = DISPLAY_RUNTIME_INFO(display)->rawclk_freq / 1000;
        struct pps_registers regs;
@@ -1638,7 +1633,7 @@ static void pps_init_registers(struct intel_dp *intel_dp, bool force_disable_vdd
         * power sequencer any more. */
        if (display->platform.valleyview || display->platform.cherryview) {
                port_sel = PANEL_PORT_SELECT_VLV(port);
-       } else if (HAS_PCH_IBX(dev_priv) || HAS_PCH_CPT(dev_priv)) {
+       } else if (HAS_PCH_IBX(display) || HAS_PCH_CPT(display)) {
                switch (port) {
                case PORT_A:
                        port_sel = PANEL_PORT_SELECT_DPA;
@@ -1791,9 +1786,7 @@ void intel_pps_unlock_regs_wa(struct intel_display *display)
 
 void intel_pps_setup(struct intel_display *display)
 {
-       struct drm_i915_private *i915 = to_i915(display->drm);
-
-       if (HAS_PCH_SPLIT(i915) || display->platform.geminilake || display->platform.broxton)
+       if (HAS_PCH_SPLIT(display) || display->platform.geminilake || display->platform.broxton)
                display->pps.mmio_base = PCH_PPS_BASE;
        else if (display->platform.valleyview || display->platform.cherryview)
                display->pps.mmio_base = VLV_PPS_BASE;
@@ -1836,7 +1829,6 @@ void intel_pps_connector_debugfs_add(struct intel_connector *connector)
 
 void assert_pps_unlocked(struct intel_display *display, enum pipe pipe)
 {
-       struct drm_i915_private *dev_priv = to_i915(display->drm);
        i915_reg_t pp_reg;
        u32 val;
        enum pipe panel_pipe = INVALID_PIPE;
@@ -1845,7 +1837,7 @@ void assert_pps_unlocked(struct intel_display *display, enum pipe pipe)
        if (drm_WARN_ON(display->drm, HAS_DDI(display)))
                return;
 
-       if (HAS_PCH_SPLIT(dev_priv)) {
+       if (HAS_PCH_SPLIT(display)) {
                u32 port_sel;
 
                pp_reg = PP_CONTROL(display, 0);
index 757b9ce7e3b1c9ac0eb4040626db8d25f1dcffb6..c8e5ad4c4e79aaf306d87ab25e879a304dd39c1b 100644 (file)
@@ -214,18 +214,17 @@ intel_sdvo_create_enhance_property(struct intel_sdvo *intel_sdvo,
 static void intel_sdvo_write_sdvox(struct intel_sdvo *intel_sdvo, u32 val)
 {
        struct intel_display *display = to_intel_display(&intel_sdvo->base);
-       struct drm_i915_private *dev_priv = to_i915(display->drm);
        u32 bval = val, cval = val;
        int i;
 
-       if (HAS_PCH_SPLIT(dev_priv)) {
+       if (HAS_PCH_SPLIT(display)) {
                intel_de_write(display, intel_sdvo->sdvo_reg, val);
                intel_de_posting_read(display, intel_sdvo->sdvo_reg);
                /*
                 * HW workaround, need to write this twice for issue
                 * that may result in first write getting masked.
                 */
-               if (HAS_PCH_IBX(dev_priv)) {
+               if (HAS_PCH_IBX(display)) {
                        intel_de_write(display, intel_sdvo->sdvo_reg, val);
                        intel_de_posting_read(display, intel_sdvo->sdvo_reg);
                }
@@ -1360,14 +1359,13 @@ static int intel_sdvo_compute_config(struct intel_encoder *encoder,
                                     struct drm_connector_state *conn_state)
 {
        struct intel_display *display = to_intel_display(encoder);
-       struct drm_i915_private *i915 = to_i915(encoder->base.dev);
        struct intel_sdvo *intel_sdvo = to_sdvo(encoder);
        struct intel_sdvo_connector *intel_sdvo_connector =
                to_intel_sdvo_connector(conn_state->connector);
        struct drm_display_mode *adjusted_mode = &pipe_config->hw.adjusted_mode;
        struct drm_display_mode *mode = &pipe_config->hw.mode;
 
-       if (HAS_PCH_SPLIT(i915)) {
+       if (HAS_PCH_SPLIT(display)) {
                pipe_config->has_pch_encoder = true;
                if (!intel_fdi_compute_pipe_bpp(pipe_config))
                        return -EINVAL;
@@ -1527,7 +1525,6 @@ static void intel_sdvo_pre_enable(struct intel_atomic_state *state,
                                  const struct drm_connector_state *conn_state)
 {
        struct intel_display *display = to_intel_display(intel_encoder);
-       struct drm_i915_private *dev_priv = to_i915(intel_encoder->base.dev);
        struct intel_crtc *crtc = to_intel_crtc(crtc_state->uapi.crtc);
        const struct drm_display_mode *adjusted_mode = &crtc_state->hw.adjusted_mode;
        const struct intel_sdvo_connector_state *sdvo_state =
@@ -1634,7 +1631,7 @@ static void intel_sdvo_pre_enable(struct intel_atomic_state *state,
                sdvox |= (9 << 19) | SDVO_BORDER_ENABLE;
        }
 
-       if (HAS_PCH_CPT(dev_priv))
+       if (HAS_PCH_CPT(display))
                sdvox |= SDVO_PIPE_SEL_CPT(crtc->pipe);
        else
                sdvox |= SDVO_PIPE_SEL(crtc->pipe);
@@ -1670,13 +1667,12 @@ static bool intel_sdvo_connector_get_hw_state(struct intel_connector *connector)
 bool intel_sdvo_port_enabled(struct intel_display *display,
                             i915_reg_t sdvo_reg, enum pipe *pipe)
 {
-       struct drm_i915_private *dev_priv = to_i915(display->drm);
        u32 val;
 
        val = intel_de_read(display, sdvo_reg);
 
        /* asserts want to know the pipe even if the port is disabled */
-       if (HAS_PCH_CPT(dev_priv))
+       if (HAS_PCH_CPT(display))
                *pipe = (val & SDVO_PIPE_SEL_MASK_CPT) >> SDVO_PIPE_SEL_SHIFT_CPT;
        else if (display->platform.cherryview)
                *pipe = (val & SDVO_PIPE_SEL_MASK_CHV) >> SDVO_PIPE_SEL_SHIFT_CHV;
@@ -1841,7 +1837,6 @@ static void intel_disable_sdvo(struct intel_atomic_state *state,
                               const struct drm_connector_state *conn_state)
 {
        struct intel_display *display = to_intel_display(encoder);
-       struct drm_i915_private *dev_priv = to_i915(encoder->base.dev);
        struct intel_sdvo *intel_sdvo = to_sdvo(encoder);
        struct intel_crtc *crtc = to_intel_crtc(old_crtc_state->uapi.crtc);
        u32 temp;
@@ -1861,7 +1856,7 @@ static void intel_disable_sdvo(struct intel_atomic_state *state,
         * to transcoder A after disabling it to allow the
         * matching DP port to be enabled on transcoder A.
         */
-       if (HAS_PCH_IBX(dev_priv) && crtc->pipe == PIPE_B) {
+       if (HAS_PCH_IBX(display) && crtc->pipe == PIPE_B) {
                /*
                 * We get CPU/PCH FIFO underruns on the other pipe when
                 * doing the workaround. Sweep them under the rug.
@@ -3367,9 +3362,7 @@ intel_sdvo_init_ddc_proxy(struct intel_sdvo_ddc *ddc,
 
 static bool is_sdvo_port_valid(struct intel_display *display, enum port port)
 {
-       struct drm_i915_private *dev_priv = to_i915(display->drm);
-
-       if (HAS_PCH_SPLIT(dev_priv))
+       if (HAS_PCH_SPLIT(display))
                return port == PORT_B;
        else
                return port == PORT_B || port == PORT_C;
@@ -3384,7 +3377,6 @@ static bool assert_sdvo_port_valid(struct intel_display *display, enum port port
 bool intel_sdvo_init(struct intel_display *display,
                     i915_reg_t sdvo_reg, enum port port)
 {
-       struct drm_i915_private *dev_priv = to_i915(display->drm);
        struct intel_encoder *intel_encoder;
        struct intel_sdvo *intel_sdvo;
        int i;
@@ -3427,7 +3419,7 @@ bool intel_sdvo_init(struct intel_display *display,
        }
 
        intel_encoder->compute_config = intel_sdvo_compute_config;
-       if (HAS_PCH_SPLIT(dev_priv)) {
+       if (HAS_PCH_SPLIT(display)) {
                intel_encoder->disable = pch_disable_sdvo;
                intel_encoder->post_disable = pch_post_disable_sdvo;
        } else {