#include "intel_pps_regs.h"
 #include "intel_quirks.h"
 
-static void vlv_steal_power_sequencer(struct drm_i915_private *dev_priv,
+static void vlv_steal_power_sequencer(struct intel_display *display,
                                      enum pipe pipe);
 
 static void pps_init_delays(struct intel_dp *intel_dp);
 
 intel_wakeref_t intel_pps_lock(struct intel_dp *intel_dp)
 {
-       struct drm_i915_private *dev_priv = dp_to_i915(intel_dp);
+       struct intel_display *display = to_intel_display(intel_dp);
+       struct drm_i915_private *dev_priv = to_i915(display->drm);
        intel_wakeref_t wakeref;
 
        /*
         * See intel_pps_reset_all() why we need a power domain reference here.
         */
        wakeref = intel_display_power_get(dev_priv, POWER_DOMAIN_DISPLAY_CORE);
-       mutex_lock(&dev_priv->display.pps.mutex);
+       mutex_lock(&display->pps.mutex);
 
        return wakeref;
 }
 intel_wakeref_t intel_pps_unlock(struct intel_dp *intel_dp,
                                 intel_wakeref_t wakeref)
 {
-       struct drm_i915_private *dev_priv = dp_to_i915(intel_dp);
+       struct intel_display *display = to_intel_display(intel_dp);
+       struct drm_i915_private *dev_priv = to_i915(display->drm);
 
-       mutex_unlock(&dev_priv->display.pps.mutex);
+       mutex_unlock(&display->pps.mutex);
        intel_display_power_put(dev_priv, POWER_DOMAIN_DISPLAY_CORE, wakeref);
 
        return 0;
 static void
 vlv_power_sequencer_kick(struct intel_dp *intel_dp)
 {
-       struct drm_i915_private *dev_priv = dp_to_i915(intel_dp);
+       struct intel_display *display = to_intel_display(intel_dp);
+       struct drm_i915_private *dev_priv = to_i915(display->drm);
        struct intel_digital_port *dig_port = dp_to_dig_port(intel_dp);
        enum pipe pipe = intel_dp->pps.pps_pipe;
        bool pll_enabled, release_cl_override = false;
        enum dpio_channel ch = vlv_pipe_to_channel(pipe);
        u32 DP;
 
-       if (drm_WARN(&dev_priv->drm,
-                    intel_de_read(dev_priv, intel_dp->output_reg) & DP_PORT_EN,
+       if (drm_WARN(display->drm,
+                    intel_de_read(display, intel_dp->output_reg) & DP_PORT_EN,
                     "skipping %s kick due to [ENCODER:%d:%s] being active\n",
                     pps_name(intel_dp),
                     dig_port->base.base.base.id, dig_port->base.base.name))
                return;
 
-       drm_dbg_kms(&dev_priv->drm,
+       drm_dbg_kms(display->drm,
                    "kicking %s for [ENCODER:%d:%s]\n",
                    pps_name(intel_dp),
                    dig_port->base.base.base.id, dig_port->base.base.name);
        /* Preserve the BIOS-computed detected bit. This is
         * supposed to be read-only.
         */
-       DP = intel_de_read(dev_priv, intel_dp->output_reg) & DP_DETECTED;
+       DP = intel_de_read(display, intel_dp->output_reg) & DP_DETECTED;
        DP |= DP_VOLTAGE_0_4 | DP_PRE_EMPHASIS_0;
        DP |= DP_PORT_WIDTH(1);
        DP |= DP_LINK_TRAIN_PAT_1;
        else
                DP |= DP_PIPE_SEL(pipe);
 
-       pll_enabled = intel_de_read(dev_priv, DPLL(dev_priv, pipe)) & DPLL_VCO_ENABLE;
+       pll_enabled = intel_de_read(display, DPLL(display, pipe)) & DPLL_VCO_ENABLE;
 
        /*
         * The DPLL for the pipe must be enabled for this to work.
                        !chv_phy_powergate_ch(dev_priv, phy, ch, true);
 
                if (vlv_force_pll_on(dev_priv, pipe, vlv_get_dpll(dev_priv))) {
-                       drm_err(&dev_priv->drm,
+                       drm_err(display->drm,
                                "Failed to force on PLL for pipe %c!\n",
                                pipe_name(pipe));
                        return;
         * to make this power sequencer lock onto the port.
         * Otherwise even VDD force bit won't work.
         */
-       intel_de_write(dev_priv, intel_dp->output_reg, DP);
-       intel_de_posting_read(dev_priv, intel_dp->output_reg);
+       intel_de_write(display, intel_dp->output_reg, DP);
+       intel_de_posting_read(display, intel_dp->output_reg);
 
-       intel_de_write(dev_priv, intel_dp->output_reg, DP | DP_PORT_EN);
-       intel_de_posting_read(dev_priv, intel_dp->output_reg);
+       intel_de_write(display, intel_dp->output_reg, DP | DP_PORT_EN);
+       intel_de_posting_read(display, intel_dp->output_reg);
 
-       intel_de_write(dev_priv, intel_dp->output_reg, DP & ~DP_PORT_EN);
-       intel_de_posting_read(dev_priv, intel_dp->output_reg);
+       intel_de_write(display, intel_dp->output_reg, DP & ~DP_PORT_EN);
+       intel_de_posting_read(display, intel_dp->output_reg);
 
        if (!pll_enabled) {
                vlv_force_pll_off(dev_priv, pipe);
        }
 }
 
-static enum pipe vlv_find_free_pps(struct drm_i915_private *dev_priv)
+static enum pipe vlv_find_free_pps(struct intel_display *display)
 {
        struct intel_encoder *encoder;
        unsigned int pipes = (1 << PIPE_A) | (1 << PIPE_B);
         * We don't have power sequencer currently.
         * Pick one that's not used by other ports.
         */
-       for_each_intel_dp(&dev_priv->drm, encoder) {
+       for_each_intel_dp(display->drm, encoder) {
                struct intel_dp *intel_dp = enc_to_intel_dp(encoder);
 
                if (encoder->type == INTEL_OUTPUT_EDP) {
-                       drm_WARN_ON(&dev_priv->drm,
+                       drm_WARN_ON(display->drm,
                                    intel_dp->pps.active_pipe != INVALID_PIPE &&
                                    intel_dp->pps.active_pipe !=
                                    intel_dp->pps.pps_pipe);
                        if (intel_dp->pps.pps_pipe != INVALID_PIPE)
                                pipes &= ~(1 << intel_dp->pps.pps_pipe);
                } else {
-                       drm_WARN_ON(&dev_priv->drm,
+                       drm_WARN_ON(display->drm,
                                    intel_dp->pps.pps_pipe != INVALID_PIPE);
 
                        if (intel_dp->pps.active_pipe != INVALID_PIPE)
 static enum pipe
 vlv_power_sequencer_pipe(struct intel_dp *intel_dp)
 {
-       struct drm_i915_private *dev_priv = dp_to_i915(intel_dp);
+       struct intel_display *display = to_intel_display(intel_dp);
        struct intel_digital_port *dig_port = dp_to_dig_port(intel_dp);
        enum pipe pipe;
 
-       lockdep_assert_held(&dev_priv->display.pps.mutex);
+       lockdep_assert_held(&display->pps.mutex);
 
        /* We should never land here with regular DP ports */
-       drm_WARN_ON(&dev_priv->drm, !intel_dp_is_edp(intel_dp));
+       drm_WARN_ON(display->drm, !intel_dp_is_edp(intel_dp));
 
-       drm_WARN_ON(&dev_priv->drm, intel_dp->pps.active_pipe != INVALID_PIPE &&
+       drm_WARN_ON(display->drm, intel_dp->pps.active_pipe != INVALID_PIPE &&
                    intel_dp->pps.active_pipe != intel_dp->pps.pps_pipe);
 
        if (intel_dp->pps.pps_pipe != INVALID_PIPE)
                return intel_dp->pps.pps_pipe;
 
-       pipe = vlv_find_free_pps(dev_priv);
+       pipe = vlv_find_free_pps(display);
 
        /*
         * Didn't find one. This should not happen since there
         * are two power sequencers and up to two eDP ports.
         */
-       if (drm_WARN_ON(&dev_priv->drm, pipe == INVALID_PIPE))
+       if (drm_WARN_ON(display->drm, pipe == INVALID_PIPE))
                pipe = PIPE_A;
 
-       vlv_steal_power_sequencer(dev_priv, pipe);
+       vlv_steal_power_sequencer(display, pipe);
        intel_dp->pps.pps_pipe = pipe;
 
-       drm_dbg_kms(&dev_priv->drm,
+       drm_dbg_kms(display->drm,
                    "picked %s for [ENCODER:%d:%s]\n",
                    pps_name(intel_dp),
                    dig_port->base.base.base.id, dig_port->base.base.name);
 static int
 bxt_power_sequencer_idx(struct intel_dp *intel_dp)
 {
-       struct drm_i915_private *dev_priv = dp_to_i915(intel_dp);
+       struct intel_display *display = to_intel_display(intel_dp);
        int pps_idx = intel_dp->pps.pps_idx;
 
-       lockdep_assert_held(&dev_priv->display.pps.mutex);
+       lockdep_assert_held(&display->pps.mutex);
 
        /* We should never land here with regular DP ports */
-       drm_WARN_ON(&dev_priv->drm, !intel_dp_is_edp(intel_dp));
+       drm_WARN_ON(display->drm, !intel_dp_is_edp(intel_dp));
 
        if (!intel_dp->pps.pps_reset)
                return pps_idx;
        return pps_idx;
 }
 
-typedef bool (*pps_check)(struct drm_i915_private *dev_priv, int pps_idx);
+typedef bool (*pps_check)(struct intel_display *display, int pps_idx);
 
-static bool pps_has_pp_on(struct drm_i915_private *dev_priv, int pps_idx)
+static bool pps_has_pp_on(struct intel_display *display, int pps_idx)
 {
-       return intel_de_read(dev_priv, PP_STATUS(dev_priv, pps_idx)) & PP_ON;
+       return intel_de_read(display, PP_STATUS(display, pps_idx)) & PP_ON;
 }
 
-static bool pps_has_vdd_on(struct drm_i915_private *dev_priv, int pps_idx)
+static bool pps_has_vdd_on(struct intel_display *display, int pps_idx)
 {
-       return intel_de_read(dev_priv, PP_CONTROL(dev_priv, pps_idx)) & EDP_FORCE_VDD;
+       return intel_de_read(display, PP_CONTROL(display, pps_idx)) & EDP_FORCE_VDD;
 }
 
-static bool pps_any(struct drm_i915_private *dev_priv, int pps_idx)
+static bool pps_any(struct intel_display *display, int pps_idx)
 {
        return true;
 }
 
 static enum pipe
-vlv_initial_pps_pipe(struct drm_i915_private *dev_priv,
+vlv_initial_pps_pipe(struct intel_display *display,
                     enum port port, pps_check check)
 {
        enum pipe pipe;
 
        for (pipe = PIPE_A; pipe <= PIPE_B; pipe++) {
-               u32 port_sel = intel_de_read(dev_priv, PP_ON_DELAYS(dev_priv, pipe)) &
+               u32 port_sel = intel_de_read(display,
+                                            PP_ON_DELAYS(display, pipe)) &
                        PANEL_PORT_SELECT_MASK;
 
                if (port_sel != PANEL_PORT_SELECT_VLV(port))
                        continue;
 
-               if (!check(dev_priv, pipe))
+               if (!check(display, pipe))
                        continue;
 
                return pipe;
 static void
 vlv_initial_power_sequencer_setup(struct intel_dp *intel_dp)
 {
-       struct drm_i915_private *dev_priv = dp_to_i915(intel_dp);
+       struct intel_display *display = to_intel_display(intel_dp);
        struct intel_digital_port *dig_port = dp_to_dig_port(intel_dp);
        enum port port = dig_port->base.port;
 
-       lockdep_assert_held(&dev_priv->display.pps.mutex);
+       lockdep_assert_held(&display->pps.mutex);
 
        /* try to find a pipe with this port selected */
        /* first pick one where the panel is on */
-       intel_dp->pps.pps_pipe = vlv_initial_pps_pipe(dev_priv, port,
+       intel_dp->pps.pps_pipe = vlv_initial_pps_pipe(display, port,
                                                      pps_has_pp_on);
        /* didn't find one? pick one where vdd is on */
        if (intel_dp->pps.pps_pipe == INVALID_PIPE)
-               intel_dp->pps.pps_pipe = vlv_initial_pps_pipe(dev_priv, port,
+               intel_dp->pps.pps_pipe = vlv_initial_pps_pipe(display, port,
                                                              pps_has_vdd_on);
        /* didn't find one? pick one with just the correct port */
        if (intel_dp->pps.pps_pipe == INVALID_PIPE)
-               intel_dp->pps.pps_pipe = vlv_initial_pps_pipe(dev_priv, port,
+               intel_dp->pps.pps_pipe = vlv_initial_pps_pipe(display, port,
                                                              pps_any);
 
        /* didn't find one? just let vlv_power_sequencer_pipe() pick one when needed */
        if (intel_dp->pps.pps_pipe == INVALID_PIPE) {
-               drm_dbg_kms(&dev_priv->drm,
+               drm_dbg_kms(display->drm,
                            "[ENCODER:%d:%s] no initial power sequencer\n",
                            dig_port->base.base.base.id, dig_port->base.base.name);
                return;
        }
 
-       drm_dbg_kms(&dev_priv->drm,
+       drm_dbg_kms(display->drm,
                    "[ENCODER:%d:%s] initial power sequencer: %s\n",
                    dig_port->base.base.base.id, dig_port->base.base.name,
                    pps_name(intel_dp));
 }
 
-static int intel_num_pps(struct drm_i915_private *i915)
+static int intel_num_pps(struct intel_display *display)
 {
+       struct drm_i915_private *i915 = to_i915(display->drm);
+
        if (IS_VALLEYVIEW(i915) || IS_CHERRYVIEW(i915))
                return 2;
 
 
 static bool intel_pps_is_valid(struct intel_dp *intel_dp)
 {
-       struct drm_i915_private *i915 = dp_to_i915(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)
-               return intel_de_read(i915, SOUTH_CHICKEN1) & ICP_SECOND_PPS_IO_SELECT;
+               return intel_de_read(display, SOUTH_CHICKEN1) & ICP_SECOND_PPS_IO_SELECT;
 
        return true;
 }
 
 static int
-bxt_initial_pps_idx(struct drm_i915_private *i915, pps_check check)
+bxt_initial_pps_idx(struct intel_display *display, pps_check check)
 {
-       int pps_idx, pps_num = intel_num_pps(i915);
+       int pps_idx, pps_num = intel_num_pps(display);
 
        for (pps_idx = 0; pps_idx < pps_num; pps_idx++) {
-               if (check(i915, pps_idx))
+               if (check(display, pps_idx))
                        return pps_idx;
        }
 
 static bool
 pps_initial_setup(struct intel_dp *intel_dp)
 {
+       struct intel_display *display = to_intel_display(intel_dp);
        struct intel_encoder *encoder = &dp_to_dig_port(intel_dp)->base;
        struct intel_connector *connector = intel_dp->attached_connector;
        struct drm_i915_private *i915 = to_i915(encoder->base.dev);
 
-       lockdep_assert_held(&i915->display.pps.mutex);
+       lockdep_assert_held(&display->pps.mutex);
 
        if (IS_VALLEYVIEW(i915) || IS_CHERRYVIEW(i915)) {
                vlv_initial_power_sequencer_setup(intel_dp);
        }
 
        /* first ask the VBT */
-       if (intel_num_pps(i915) > 1)
+       if (intel_num_pps(display) > 1)
                intel_dp->pps.pps_idx = connector->panel.vbt.backlight.controller;
        else
                intel_dp->pps.pps_idx = 0;
 
-       if (drm_WARN_ON(&i915->drm, intel_dp->pps.pps_idx >= intel_num_pps(i915)))
+       if (drm_WARN_ON(display->drm, intel_dp->pps.pps_idx >= intel_num_pps(display)))
                intel_dp->pps.pps_idx = -1;
 
        /* VBT wasn't parsed yet? pick one where the panel is on */
        if (intel_dp->pps.pps_idx < 0)
-               intel_dp->pps.pps_idx = bxt_initial_pps_idx(i915, pps_has_pp_on);
+               intel_dp->pps.pps_idx = bxt_initial_pps_idx(display, pps_has_pp_on);
        /* didn't find one? pick one where vdd is on */
        if (intel_dp->pps.pps_idx < 0)
-               intel_dp->pps.pps_idx = bxt_initial_pps_idx(i915, pps_has_vdd_on);
+               intel_dp->pps.pps_idx = bxt_initial_pps_idx(display, pps_has_vdd_on);
        /* didn't find one? pick any */
        if (intel_dp->pps.pps_idx < 0) {
-               intel_dp->pps.pps_idx = bxt_initial_pps_idx(i915, pps_any);
+               intel_dp->pps.pps_idx = bxt_initial_pps_idx(display, pps_any);
 
-               drm_dbg_kms(&i915->drm,
+               drm_dbg_kms(display->drm,
                            "[ENCODER:%d:%s] no initial power sequencer, assuming %s\n",
                            encoder->base.base.id, encoder->base.name,
                            pps_name(intel_dp));
        } else {
-               drm_dbg_kms(&i915->drm,
+               drm_dbg_kms(display->drm,
                            "[ENCODER:%d:%s] initial power sequencer: %s\n",
                            encoder->base.base.id, encoder->base.name,
                            pps_name(intel_dp));
        return intel_pps_is_valid(intel_dp);
 }
 
-void intel_pps_reset_all(struct drm_i915_private *dev_priv)
+void intel_pps_reset_all(struct intel_display *display)
 {
+       struct drm_i915_private *dev_priv = to_i915(display->drm);
        struct intel_encoder *encoder;
 
-       if (drm_WARN_ON(&dev_priv->drm, !IS_LP(dev_priv)))
+       if (drm_WARN_ON(display->drm, !IS_LP(dev_priv)))
                return;
 
-       if (!HAS_DISPLAY(dev_priv))
+       if (!HAS_DISPLAY(display))
                return;
 
        /*
         * should use them always.
         */
 
-       for_each_intel_dp(&dev_priv->drm, encoder) {
+       for_each_intel_dp(display->drm, encoder) {
                struct intel_dp *intel_dp = enc_to_intel_dp(encoder);
 
-               drm_WARN_ON(&dev_priv->drm,
+               drm_WARN_ON(display->drm,
                            intel_dp->pps.active_pipe != INVALID_PIPE);
 
                if (encoder->type != INTEL_OUTPUT_EDP)
                        continue;
 
-               if (DISPLAY_VER(dev_priv) >= 9)
+               if (DISPLAY_VER(display) >= 9)
                        intel_dp->pps.pps_reset = true;
                else
                        intel_dp->pps.pps_pipe = INVALID_PIPE;
 static void intel_pps_get_registers(struct intel_dp *intel_dp,
                                    struct pps_registers *regs)
 {
-       struct drm_i915_private *dev_priv = dp_to_i915(intel_dp);
+       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));
        else
                pps_idx = intel_dp->pps.pps_idx;
 
-       regs->pp_ctrl = PP_CONTROL(dev_priv, pps_idx);
-       regs->pp_stat = PP_STATUS(dev_priv, pps_idx);
-       regs->pp_on = PP_ON_DELAYS(dev_priv, pps_idx);
-       regs->pp_off = PP_OFF_DELAYS(dev_priv, pps_idx);
+       regs->pp_ctrl = PP_CONTROL(display, pps_idx);
+       regs->pp_stat = PP_STATUS(display, pps_idx);
+       regs->pp_on = PP_ON_DELAYS(display, pps_idx);
+       regs->pp_off = PP_OFF_DELAYS(display, pps_idx);
 
        /* Cycle delay moved from PP_DIVISOR to PP_CONTROL */
        if (IS_GEMINILAKE(dev_priv) || IS_BROXTON(dev_priv) ||
            INTEL_PCH_TYPE(dev_priv) >= PCH_CNP)
                regs->pp_div = INVALID_MMIO_REG;
        else
-               regs->pp_div = PP_DIVISOR(dev_priv, pps_idx);
+               regs->pp_div = PP_DIVISOR(display, pps_idx);
 }
 
 static i915_reg_t
 
 static bool edp_have_panel_power(struct intel_dp *intel_dp)
 {
-       struct drm_i915_private *dev_priv = dp_to_i915(intel_dp);
+       struct intel_display *display = to_intel_display(intel_dp);
+       struct drm_i915_private *dev_priv = to_i915(display->drm);
 
-       lockdep_assert_held(&dev_priv->display.pps.mutex);
+       lockdep_assert_held(&display->pps.mutex);
 
        if ((IS_VALLEYVIEW(dev_priv) || IS_CHERRYVIEW(dev_priv)) &&
            intel_dp->pps.pps_pipe == INVALID_PIPE)
                return false;
 
-       return (intel_de_read(dev_priv, _pp_stat_reg(intel_dp)) & PP_ON) != 0;
+       return (intel_de_read(display, _pp_stat_reg(intel_dp)) & PP_ON) != 0;
 }
 
 static bool edp_have_panel_vdd(struct intel_dp *intel_dp)
 {
-       struct drm_i915_private *dev_priv = dp_to_i915(intel_dp);
+       struct intel_display *display = to_intel_display(intel_dp);
+       struct drm_i915_private *dev_priv = to_i915(display->drm);
 
-       lockdep_assert_held(&dev_priv->display.pps.mutex);
+       lockdep_assert_held(&display->pps.mutex);
 
        if ((IS_VALLEYVIEW(dev_priv) || IS_CHERRYVIEW(dev_priv)) &&
            intel_dp->pps.pps_pipe == INVALID_PIPE)
                return false;
 
-       return intel_de_read(dev_priv, _pp_ctrl_reg(intel_dp)) & EDP_FORCE_VDD;
+       return intel_de_read(display, _pp_ctrl_reg(intel_dp)) & EDP_FORCE_VDD;
 }
 
 void intel_pps_check_power_unlocked(struct intel_dp *intel_dp)
 {
-       struct drm_i915_private *dev_priv = dp_to_i915(intel_dp);
+       struct intel_display *display = to_intel_display(intel_dp);
        struct intel_digital_port *dig_port = dp_to_dig_port(intel_dp);
 
        if (!intel_dp_is_edp(intel_dp))
                return;
 
        if (!edp_have_panel_power(intel_dp) && !edp_have_panel_vdd(intel_dp)) {
-               drm_WARN(&dev_priv->drm, 1,
+               drm_WARN(display->drm, 1,
                         "[ENCODER:%d:%s] %s powered off while attempting AUX CH communication.\n",
                         dig_port->base.base.base.id, dig_port->base.base.name,
                         pps_name(intel_dp));
-               drm_dbg_kms(&dev_priv->drm,
+               drm_dbg_kms(display->drm,
                            "[ENCODER:%d:%s] %s PP_STATUS: 0x%08x PP_CONTROL: 0x%08x\n",
                            dig_port->base.base.base.id, dig_port->base.base.name,
                            pps_name(intel_dp),
-                           intel_de_read(dev_priv, _pp_stat_reg(intel_dp)),
-                           intel_de_read(dev_priv, _pp_ctrl_reg(intel_dp)));
+                           intel_de_read(display, _pp_stat_reg(intel_dp)),
+                           intel_de_read(display, _pp_ctrl_reg(intel_dp)));
        }
 }
 
 static void wait_panel_status(struct intel_dp *intel_dp,
                              u32 mask, u32 value)
 {
-       struct drm_i915_private *dev_priv = dp_to_i915(intel_dp);
+       struct intel_display *display = to_intel_display(intel_dp);
        struct intel_digital_port *dig_port = dp_to_dig_port(intel_dp);
        i915_reg_t pp_stat_reg, pp_ctrl_reg;
 
-       lockdep_assert_held(&dev_priv->display.pps.mutex);
+       lockdep_assert_held(&display->pps.mutex);
 
        intel_pps_verify_state(intel_dp);
 
        pp_stat_reg = _pp_stat_reg(intel_dp);
        pp_ctrl_reg = _pp_ctrl_reg(intel_dp);
 
-       drm_dbg_kms(&dev_priv->drm,
+       drm_dbg_kms(display->drm,
                    "[ENCODER:%d:%s] %s mask: 0x%08x value: 0x%08x PP_STATUS: 0x%08x PP_CONTROL: 0x%08x\n",
                    dig_port->base.base.base.id, dig_port->base.base.name,
                    pps_name(intel_dp),
                    mask, value,
-                   intel_de_read(dev_priv, pp_stat_reg),
-                   intel_de_read(dev_priv, pp_ctrl_reg));
+                   intel_de_read(display, pp_stat_reg),
+                   intel_de_read(display, pp_ctrl_reg));
 
-       if (intel_de_wait(dev_priv, pp_stat_reg, mask, value, 5000))
-               drm_err(&dev_priv->drm,
+       if (intel_de_wait(display, pp_stat_reg, mask, value, 5000))
+               drm_err(display->drm,
                        "[ENCODER:%d:%s] %s panel status timeout: PP_STATUS: 0x%08x PP_CONTROL: 0x%08x\n",
                        dig_port->base.base.base.id, dig_port->base.base.name,
                        pps_name(intel_dp),
-                       intel_de_read(dev_priv, pp_stat_reg),
-                       intel_de_read(dev_priv, pp_ctrl_reg));
+                       intel_de_read(display, pp_stat_reg),
+                       intel_de_read(display, pp_ctrl_reg));
 
-       drm_dbg_kms(&dev_priv->drm, "Wait complete\n");
+       drm_dbg_kms(display->drm, "Wait complete\n");
 }
 
 static void wait_panel_on(struct intel_dp *intel_dp)
 {
-       struct drm_i915_private *i915 = dp_to_i915(intel_dp);
+       struct intel_display *display = to_intel_display(intel_dp);
        struct intel_digital_port *dig_port = dp_to_dig_port(intel_dp);
 
-       drm_dbg_kms(&i915->drm, "[ENCODER:%d:%s] %s wait for panel power on\n",
+       drm_dbg_kms(display->drm,
+                   "[ENCODER:%d:%s] %s wait for panel power on\n",
                    dig_port->base.base.base.id, dig_port->base.base.name,
                    pps_name(intel_dp));
        wait_panel_status(intel_dp, IDLE_ON_MASK, IDLE_ON_VALUE);
 
 static void wait_panel_off(struct intel_dp *intel_dp)
 {
-       struct drm_i915_private *i915 = dp_to_i915(intel_dp);
+       struct intel_display *display = to_intel_display(intel_dp);
        struct intel_digital_port *dig_port = dp_to_dig_port(intel_dp);
 
-       drm_dbg_kms(&i915->drm, "[ENCODER:%d:%s] %s wait for panel power off time\n",
+       drm_dbg_kms(display->drm,
+                   "[ENCODER:%d:%s] %s wait for panel power off time\n",
                    dig_port->base.base.base.id, dig_port->base.base.name,
                    pps_name(intel_dp));
        wait_panel_status(intel_dp, IDLE_OFF_MASK, IDLE_OFF_VALUE);
 
 static void wait_panel_power_cycle(struct intel_dp *intel_dp)
 {
-       struct drm_i915_private *i915 = dp_to_i915(intel_dp);
+       struct intel_display *display = to_intel_display(intel_dp);
        struct intel_digital_port *dig_port = dp_to_dig_port(intel_dp);
        ktime_t panel_power_on_time;
        s64 panel_power_off_duration;
 
-       drm_dbg_kms(&i915->drm, "[ENCODER:%d:%s] %s wait for panel power cycle\n",
+       drm_dbg_kms(display->drm,
+                   "[ENCODER:%d:%s] %s wait for panel power cycle\n",
                    dig_port->base.base.base.id, dig_port->base.base.name,
                    pps_name(intel_dp));
 
 
 static  u32 ilk_get_pp_control(struct intel_dp *intel_dp)
 {
-       struct drm_i915_private *dev_priv = dp_to_i915(intel_dp);
+       struct intel_display *display = to_intel_display(intel_dp);
        u32 control;
 
-       lockdep_assert_held(&dev_priv->display.pps.mutex);
+       lockdep_assert_held(&display->pps.mutex);
 
-       control = intel_de_read(dev_priv, _pp_ctrl_reg(intel_dp));
-       if (drm_WARN_ON(&dev_priv->drm, !HAS_DDI(dev_priv) &&
+       control = intel_de_read(display, _pp_ctrl_reg(intel_dp));
+       if (drm_WARN_ON(display->drm, !HAS_DDI(display) &&
                        (control & PANEL_UNLOCK_MASK) != PANEL_UNLOCK_REGS)) {
                control &= ~PANEL_UNLOCK_MASK;
                control |= PANEL_UNLOCK_REGS;
  */
 bool intel_pps_vdd_on_unlocked(struct intel_dp *intel_dp)
 {
-       struct drm_i915_private *dev_priv = dp_to_i915(intel_dp);
+       struct intel_display *display = to_intel_display(intel_dp);
+       struct drm_i915_private *dev_priv = to_i915(display->drm);
        struct intel_digital_port *dig_port = dp_to_dig_port(intel_dp);
        u32 pp;
        i915_reg_t pp_stat_reg, pp_ctrl_reg;
        bool need_to_disable = !intel_dp->pps.want_panel_vdd;
 
-       lockdep_assert_held(&dev_priv->display.pps.mutex);
+       lockdep_assert_held(&display->pps.mutex);
 
        if (!intel_dp_is_edp(intel_dp))
                return false;
        if (edp_have_panel_vdd(intel_dp))
                return need_to_disable;
 
-       drm_WARN_ON(&dev_priv->drm, intel_dp->pps.vdd_wakeref);
+       drm_WARN_ON(display->drm, intel_dp->pps.vdd_wakeref);
        intel_dp->pps.vdd_wakeref = intel_display_power_get(dev_priv,
                                                            intel_aux_power_domain(dig_port));
 
        pp_stat_reg = _pp_stat_reg(intel_dp);
        pp_ctrl_reg = _pp_ctrl_reg(intel_dp);
 
-       drm_dbg_kms(&dev_priv->drm, "[ENCODER:%d:%s] %s turning VDD on\n",
+       drm_dbg_kms(display->drm, "[ENCODER:%d:%s] %s turning VDD on\n",
                    dig_port->base.base.base.id, dig_port->base.base.name,
                    pps_name(intel_dp));
 
        pp = ilk_get_pp_control(intel_dp);
        pp |= EDP_FORCE_VDD;
 
-       intel_de_write(dev_priv, pp_ctrl_reg, pp);
-       intel_de_posting_read(dev_priv, pp_ctrl_reg);
-       drm_dbg_kms(&dev_priv->drm, "[ENCODER:%d:%s] %s PP_STATUS: 0x%08x PP_CONTROL: 0x%08x\n",
+       intel_de_write(display, pp_ctrl_reg, pp);
+       intel_de_posting_read(display, pp_ctrl_reg);
+       drm_dbg_kms(display->drm,
+                   "[ENCODER:%d:%s] %s PP_STATUS: 0x%08x PP_CONTROL: 0x%08x\n",
                    dig_port->base.base.base.id, dig_port->base.base.name,
                    pps_name(intel_dp),
-                   intel_de_read(dev_priv, pp_stat_reg),
-                   intel_de_read(dev_priv, pp_ctrl_reg));
+                   intel_de_read(display, pp_stat_reg),
+                   intel_de_read(display, pp_ctrl_reg));
        /*
         * If the panel wasn't on, delay before accessing aux channel
         */
        if (!edp_have_panel_power(intel_dp)) {
-               drm_dbg_kms(&dev_priv->drm,
+               drm_dbg_kms(display->drm,
                            "[ENCODER:%d:%s] %s panel power wasn't enabled\n",
                            dig_port->base.base.base.id, dig_port->base.base.name,
                            pps_name(intel_dp));
  */
 void intel_pps_vdd_on(struct intel_dp *intel_dp)
 {
-       struct drm_i915_private *i915 = dp_to_i915(intel_dp);
+       struct intel_display *display = to_intel_display(intel_dp);
+       struct drm_i915_private *i915 = to_i915(display->drm);
        intel_wakeref_t wakeref;
        bool vdd;
 
 
 static void intel_pps_vdd_off_sync_unlocked(struct intel_dp *intel_dp)
 {
-       struct drm_i915_private *dev_priv = dp_to_i915(intel_dp);
-       struct intel_digital_port *dig_port =
-               dp_to_dig_port(intel_dp);
+       struct intel_display *display = to_intel_display(intel_dp);
+       struct drm_i915_private *dev_priv = to_i915(display->drm);
+       struct intel_digital_port *dig_port = dp_to_dig_port(intel_dp);
        u32 pp;
        i915_reg_t pp_stat_reg, pp_ctrl_reg;
 
-       lockdep_assert_held(&dev_priv->display.pps.mutex);
+       lockdep_assert_held(&display->pps.mutex);
 
-       drm_WARN_ON(&dev_priv->drm, intel_dp->pps.want_panel_vdd);
+       drm_WARN_ON(display->drm, intel_dp->pps.want_panel_vdd);
 
        if (!edp_have_panel_vdd(intel_dp))
                return;
 
-       drm_dbg_kms(&dev_priv->drm, "[ENCODER:%d:%s] %s turning VDD off\n",
+       drm_dbg_kms(display->drm, "[ENCODER:%d:%s] %s turning VDD off\n",
                    dig_port->base.base.base.id, dig_port->base.base.name,
                    pps_name(intel_dp));
 
        pp_ctrl_reg = _pp_ctrl_reg(intel_dp);
        pp_stat_reg = _pp_stat_reg(intel_dp);
 
-       intel_de_write(dev_priv, pp_ctrl_reg, pp);
-       intel_de_posting_read(dev_priv, pp_ctrl_reg);
+       intel_de_write(display, pp_ctrl_reg, pp);
+       intel_de_posting_read(display, pp_ctrl_reg);
 
        /* Make sure sequencer is idle before allowing subsequent activity */
-       drm_dbg_kms(&dev_priv->drm, "[ENCODER:%d:%s] %s PP_STATUS: 0x%08x PP_CONTROL: 0x%08x\n",
+       drm_dbg_kms(display->drm,
+                   "[ENCODER:%d:%s] %s PP_STATUS: 0x%08x PP_CONTROL: 0x%08x\n",
                    dig_port->base.base.base.id, dig_port->base.base.name,
                    pps_name(intel_dp),
-                   intel_de_read(dev_priv, pp_stat_reg),
-                   intel_de_read(dev_priv, pp_ctrl_reg));
+                   intel_de_read(display, pp_stat_reg),
+                   intel_de_read(display, pp_ctrl_reg));
 
        if ((pp & PANEL_POWER_ON) == 0)
                intel_dp->pps.panel_power_off_time = ktime_get_boottime();
 
 static void edp_panel_vdd_schedule_off(struct intel_dp *intel_dp)
 {
-       struct drm_i915_private *i915 = dp_to_i915(intel_dp);
+       struct intel_display *display = to_intel_display(intel_dp);
+       struct drm_i915_private *i915 = to_i915(display->drm);
        unsigned long delay;
 
        /*
  */
 void intel_pps_vdd_off_unlocked(struct intel_dp *intel_dp, bool sync)
 {
-       struct drm_i915_private *dev_priv = dp_to_i915(intel_dp);
+       struct intel_display *display = to_intel_display(intel_dp);
+       struct drm_i915_private *dev_priv = to_i915(display->drm);
 
-       lockdep_assert_held(&dev_priv->display.pps.mutex);
+       lockdep_assert_held(&display->pps.mutex);
 
        if (!intel_dp_is_edp(intel_dp))
                return;
 
 void intel_pps_on_unlocked(struct intel_dp *intel_dp)
 {
-       struct drm_i915_private *dev_priv = dp_to_i915(intel_dp);
+       struct intel_display *display = to_intel_display(intel_dp);
+       struct drm_i915_private *dev_priv = to_i915(display->drm);
        u32 pp;
        i915_reg_t pp_ctrl_reg;
 
-       lockdep_assert_held(&dev_priv->display.pps.mutex);
+       lockdep_assert_held(&display->pps.mutex);
 
        if (!intel_dp_is_edp(intel_dp))
                return;
 
-       drm_dbg_kms(&dev_priv->drm, "[ENCODER:%d:%s] %s turn panel power on\n",
+       drm_dbg_kms(display->drm, "[ENCODER:%d:%s] %s turn panel power on\n",
                    dp_to_dig_port(intel_dp)->base.base.base.id,
                    dp_to_dig_port(intel_dp)->base.base.name,
                    pps_name(intel_dp));
 
-       if (drm_WARN(&dev_priv->drm, edp_have_panel_power(intel_dp),
+       if (drm_WARN(display->drm, edp_have_panel_power(intel_dp),
                     "[ENCODER:%d:%s] %s panel power already on\n",
                     dp_to_dig_port(intel_dp)->base.base.base.id,
                     dp_to_dig_port(intel_dp)->base.base.name,
        if (IS_IRONLAKE(dev_priv)) {
                /* ILK workaround: disable reset around power sequence */
                pp &= ~PANEL_POWER_RESET;
-               intel_de_write(dev_priv, pp_ctrl_reg, pp);
-               intel_de_posting_read(dev_priv, pp_ctrl_reg);
+               intel_de_write(display, pp_ctrl_reg, pp);
+               intel_de_posting_read(display, pp_ctrl_reg);
        }
 
        /*
         * WA: 22019252566
         * Disable DPLS gating around power sequence.
         */
-       if (IS_DISPLAY_VER(dev_priv, 13, 14))
-               intel_de_rmw(dev_priv, SOUTH_DSPCLK_GATE_D,
+       if (IS_DISPLAY_VER(display, 13, 14))
+               intel_de_rmw(display, SOUTH_DSPCLK_GATE_D,
                             0, PCH_DPLSUNIT_CLOCK_GATE_DISABLE);
 
        pp |= PANEL_POWER_ON;
        if (!IS_IRONLAKE(dev_priv))
                pp |= PANEL_POWER_RESET;
 
-       intel_de_write(dev_priv, pp_ctrl_reg, pp);
-       intel_de_posting_read(dev_priv, pp_ctrl_reg);
+       intel_de_write(display, pp_ctrl_reg, pp);
+       intel_de_posting_read(display, pp_ctrl_reg);
 
        wait_panel_on(intel_dp);
        intel_dp->pps.last_power_on = jiffies;
 
-       if (IS_DISPLAY_VER(dev_priv, 13, 14))
-               intel_de_rmw(dev_priv, SOUTH_DSPCLK_GATE_D,
+       if (IS_DISPLAY_VER(display, 13, 14))
+               intel_de_rmw(display, SOUTH_DSPCLK_GATE_D,
                             PCH_DPLSUNIT_CLOCK_GATE_DISABLE, 0);
 
        if (IS_IRONLAKE(dev_priv)) {
                pp |= PANEL_POWER_RESET; /* restore panel reset bit */
-               intel_de_write(dev_priv, pp_ctrl_reg, pp);
-               intel_de_posting_read(dev_priv, pp_ctrl_reg);
+               intel_de_write(display, pp_ctrl_reg, pp);
+               intel_de_posting_read(display, pp_ctrl_reg);
        }
 }
 
 
 void intel_pps_off_unlocked(struct intel_dp *intel_dp)
 {
-       struct drm_i915_private *dev_priv = dp_to_i915(intel_dp);
+       struct intel_display *display = to_intel_display(intel_dp);
+       struct drm_i915_private *dev_priv = to_i915(display->drm);
        struct intel_digital_port *dig_port = dp_to_dig_port(intel_dp);
        u32 pp;
        i915_reg_t pp_ctrl_reg;
 
-       lockdep_assert_held(&dev_priv->display.pps.mutex);
+       lockdep_assert_held(&display->pps.mutex);
 
        if (!intel_dp_is_edp(intel_dp))
                return;
 
-       drm_dbg_kms(&dev_priv->drm, "[ENCODER:%d:%s] %s turn panel power off\n",
+       drm_dbg_kms(display->drm, "[ENCODER:%d:%s] %s turn panel power off\n",
                    dig_port->base.base.base.id, dig_port->base.base.name,
                    pps_name(intel_dp));
 
-       drm_WARN(&dev_priv->drm, !intel_dp->pps.want_panel_vdd,
+       drm_WARN(display->drm, !intel_dp->pps.want_panel_vdd,
                 "[ENCODER:%d:%s] %s need VDD to turn off panel\n",
                 dig_port->base.base.base.id, dig_port->base.base.name,
                 pps_name(intel_dp));
 
        intel_dp->pps.want_panel_vdd = false;
 
-       intel_de_write(dev_priv, pp_ctrl_reg, pp);
-       intel_de_posting_read(dev_priv, pp_ctrl_reg);
+       intel_de_write(display, pp_ctrl_reg, pp);
+       intel_de_posting_read(display, pp_ctrl_reg);
 
        wait_panel_off(intel_dp);
        intel_dp->pps.panel_power_off_time = ktime_get_boottime();
 /* Enable backlight in the panel power control. */
 void intel_pps_backlight_on(struct intel_dp *intel_dp)
 {
-       struct drm_i915_private *dev_priv = dp_to_i915(intel_dp);
+       struct intel_display *display = to_intel_display(intel_dp);
        intel_wakeref_t wakeref;
 
        /*
                pp = ilk_get_pp_control(intel_dp);
                pp |= EDP_BLC_ENABLE;
 
-               intel_de_write(dev_priv, pp_ctrl_reg, pp);
-               intel_de_posting_read(dev_priv, pp_ctrl_reg);
+               intel_de_write(display, pp_ctrl_reg, pp);
+               intel_de_posting_read(display, pp_ctrl_reg);
        }
 }
 
 /* Disable backlight in the panel power control. */
 void intel_pps_backlight_off(struct intel_dp *intel_dp)
 {
-       struct drm_i915_private *dev_priv = dp_to_i915(intel_dp);
+       struct intel_display *display = to_intel_display(intel_dp);
        intel_wakeref_t wakeref;
 
        if (!intel_dp_is_edp(intel_dp))
                pp = ilk_get_pp_control(intel_dp);
                pp &= ~EDP_BLC_ENABLE;
 
-               intel_de_write(dev_priv, pp_ctrl_reg, pp);
-               intel_de_posting_read(dev_priv, pp_ctrl_reg);
+               intel_de_write(display, pp_ctrl_reg, pp);
+               intel_de_posting_read(display, pp_ctrl_reg);
        }
 
        intel_dp->pps.last_backlight_off = jiffies;
  */
 void intel_pps_backlight_power(struct intel_connector *connector, bool enable)
 {
-       struct drm_i915_private *i915 = to_i915(connector->base.dev);
+       struct intel_display *display = to_intel_display(connector);
        struct intel_dp *intel_dp = intel_attached_dp(connector);
        intel_wakeref_t wakeref;
        bool is_enabled;
        if (is_enabled == enable)
                return;
 
-       drm_dbg_kms(&i915->drm, "panel power control backlight %s\n",
+       drm_dbg_kms(display->drm, "panel power control backlight %s\n",
                    enable ? "enable" : "disable");
 
        if (enable)
 
 static void vlv_detach_power_sequencer(struct intel_dp *intel_dp)
 {
+       struct intel_display *display = to_intel_display(intel_dp);
        struct intel_digital_port *dig_port = dp_to_dig_port(intel_dp);
-       struct drm_i915_private *dev_priv = to_i915(dig_port->base.base.dev);
        enum pipe pipe = intel_dp->pps.pps_pipe;
-       i915_reg_t pp_on_reg = PP_ON_DELAYS(dev_priv, pipe);
+       i915_reg_t pp_on_reg = PP_ON_DELAYS(display, pipe);
 
-       drm_WARN_ON(&dev_priv->drm, intel_dp->pps.active_pipe != INVALID_PIPE);
+       drm_WARN_ON(display->drm, intel_dp->pps.active_pipe != INVALID_PIPE);
 
-       if (drm_WARN_ON(&dev_priv->drm, pipe != PIPE_A && pipe != PIPE_B))
+       if (drm_WARN_ON(display->drm, pipe != PIPE_A && pipe != PIPE_B))
                return;
 
        intel_pps_vdd_off_sync_unlocked(intel_dp);
         * port select always when logically disconnecting a power sequencer
         * from a port.
         */
-       drm_dbg_kms(&dev_priv->drm,
+       drm_dbg_kms(display->drm,
                    "detaching %s from [ENCODER:%d:%s]\n",
                    pps_name(intel_dp),
                    dig_port->base.base.base.id, dig_port->base.base.name);
-       intel_de_write(dev_priv, pp_on_reg, 0);
-       intel_de_posting_read(dev_priv, pp_on_reg);
+       intel_de_write(display, pp_on_reg, 0);
+       intel_de_posting_read(display, pp_on_reg);
 
        intel_dp->pps.pps_pipe = INVALID_PIPE;
 }
 
-static void vlv_steal_power_sequencer(struct drm_i915_private *dev_priv,
+static void vlv_steal_power_sequencer(struct intel_display *display,
                                      enum pipe pipe)
 {
        struct intel_encoder *encoder;
 
-       lockdep_assert_held(&dev_priv->display.pps.mutex);
+       lockdep_assert_held(&display->pps.mutex);
 
-       for_each_intel_dp(&dev_priv->drm, encoder) {
+       for_each_intel_dp(display->drm, encoder) {
                struct intel_dp *intel_dp = enc_to_intel_dp(encoder);
 
-               drm_WARN(&dev_priv->drm, intel_dp->pps.active_pipe == pipe,
+               drm_WARN(display->drm, intel_dp->pps.active_pipe == pipe,
                         "stealing PPS %c from active [ENCODER:%d:%s]\n",
                         pipe_name(pipe), encoder->base.base.id,
                         encoder->base.name);
                if (intel_dp->pps.pps_pipe != pipe)
                        continue;
 
-               drm_dbg_kms(&dev_priv->drm,
+               drm_dbg_kms(display->drm,
                            "stealing PPS %c from [ENCODER:%d:%s]\n",
                            pipe_name(pipe), encoder->base.base.id,
                            encoder->base.name);
 void vlv_pps_init(struct intel_encoder *encoder,
                  const struct intel_crtc_state *crtc_state)
 {
-       struct drm_i915_private *dev_priv = to_i915(encoder->base.dev);
+       struct intel_display *display = to_intel_display(encoder);
        struct intel_dp *intel_dp = enc_to_intel_dp(encoder);
        struct intel_crtc *crtc = to_intel_crtc(crtc_state->uapi.crtc);
 
-       lockdep_assert_held(&dev_priv->display.pps.mutex);
+       lockdep_assert_held(&display->pps.mutex);
 
-       drm_WARN_ON(&dev_priv->drm, intel_dp->pps.active_pipe != INVALID_PIPE);
+       drm_WARN_ON(display->drm, intel_dp->pps.active_pipe != INVALID_PIPE);
 
        if (intel_dp->pps.pps_pipe != INVALID_PIPE &&
            intel_dp->pps.pps_pipe != crtc->pipe) {
         * We may be stealing the power
         * sequencer from another port.
         */
-       vlv_steal_power_sequencer(dev_priv, crtc->pipe);
+       vlv_steal_power_sequencer(display, crtc->pipe);
 
        intel_dp->pps.active_pipe = crtc->pipe;
 
        /* now it's all ours */
        intel_dp->pps.pps_pipe = crtc->pipe;
 
-       drm_dbg_kms(&dev_priv->drm,
+       drm_dbg_kms(display->drm,
                    "initializing %s for [ENCODER:%d:%s]\n",
                    pps_name(intel_dp),
                    encoder->base.base.id, encoder->base.name);
 
 static void pps_vdd_init(struct intel_dp *intel_dp)
 {
-       struct drm_i915_private *dev_priv = dp_to_i915(intel_dp);
+       struct intel_display *display = to_intel_display(intel_dp);
+       struct drm_i915_private *dev_priv = to_i915(display->drm);
        struct intel_digital_port *dig_port = dp_to_dig_port(intel_dp);
 
-       lockdep_assert_held(&dev_priv->display.pps.mutex);
+       lockdep_assert_held(&display->pps.mutex);
 
        if (!edp_have_panel_vdd(intel_dp))
                return;
         * schedule a vdd off, so we don't hold on to the reference
         * indefinitely.
         */
-       drm_dbg_kms(&dev_priv->drm,
+       drm_dbg_kms(display->drm,
                    "[ENCODER:%d:%s] %s VDD left on by BIOS, adjusting state tracking\n",
                    dig_port->base.base.base.id, dig_port->base.base.name,
                    pps_name(intel_dp));
-       drm_WARN_ON(&dev_priv->drm, intel_dp->pps.vdd_wakeref);
+       drm_WARN_ON(display->drm, intel_dp->pps.vdd_wakeref);
        intel_dp->pps.vdd_wakeref = intel_display_power_get(dev_priv,
                                                            intel_aux_power_domain(dig_port));
 }
 static void
 intel_pps_readout_hw_state(struct intel_dp *intel_dp, struct edp_power_seq *seq)
 {
-       struct drm_i915_private *dev_priv = dp_to_i915(intel_dp);
+       struct intel_display *display = to_intel_display(intel_dp);
        u32 pp_on, pp_off, pp_ctl;
        struct pps_registers regs;
 
        pp_ctl = ilk_get_pp_control(intel_dp);
 
        /* Ensure PPS is unlocked */
-       if (!HAS_DDI(dev_priv))
-               intel_de_write(dev_priv, regs.pp_ctrl, pp_ctl);
+       if (!HAS_DDI(display))
+               intel_de_write(display, regs.pp_ctrl, pp_ctl);
 
-       pp_on = intel_de_read(dev_priv, regs.pp_on);
-       pp_off = intel_de_read(dev_priv, regs.pp_off);
+       pp_on = intel_de_read(display, regs.pp_on);
+       pp_off = intel_de_read(display, regs.pp_off);
 
        /* Pull timing values out of registers */
        seq->t1_t3 = REG_FIELD_GET(PANEL_POWER_UP_DELAY_MASK, pp_on);
        if (i915_mmio_reg_valid(regs.pp_div)) {
                u32 pp_div;
 
-               pp_div = intel_de_read(dev_priv, regs.pp_div);
+               pp_div = intel_de_read(display, regs.pp_div);
 
                seq->t11_t12 = REG_FIELD_GET(PANEL_POWER_CYCLE_DELAY_MASK, pp_div) * 1000;
        } else {
 intel_pps_dump_state(struct intel_dp *intel_dp, const char *state_name,
                     const struct edp_power_seq *seq)
 {
-       struct drm_i915_private *i915 = dp_to_i915(intel_dp);
+       struct intel_display *display = to_intel_display(intel_dp);
 
-       drm_dbg_kms(&i915->drm, "%s t1_t3 %d t8 %d t9 %d t10 %d t11_t12 %d\n",
+       drm_dbg_kms(display->drm,
+                   "%s t1_t3 %d t8 %d t9 %d t10 %d t11_t12 %d\n",
                    state_name,
                    seq->t1_t3, seq->t8, seq->t9, seq->t10, seq->t11_t12);
 }
 static void
 intel_pps_verify_state(struct intel_dp *intel_dp)
 {
-       struct drm_i915_private *i915 = dp_to_i915(intel_dp);
+       struct intel_display *display = to_intel_display(intel_dp);
        struct edp_power_seq hw;
        struct edp_power_seq *sw = &intel_dp->pps.pps_delays;
 
 
        if (hw.t1_t3 != sw->t1_t3 || hw.t8 != sw->t8 || hw.t9 != sw->t9 ||
            hw.t10 != sw->t10 || hw.t11_t12 != sw->t11_t12) {
-               drm_err(&i915->drm, "PPS state mismatch\n");
+               drm_err(display->drm, "PPS state mismatch\n");
                intel_pps_dump_state(intel_dp, "sw", sw);
                intel_pps_dump_state(intel_dp, "hw", &hw);
        }
 static void pps_init_delays_bios(struct intel_dp *intel_dp,
                                 struct edp_power_seq *bios)
 {
-       struct drm_i915_private *dev_priv = dp_to_i915(intel_dp);
+       struct intel_display *display = to_intel_display(intel_dp);
 
-       lockdep_assert_held(&dev_priv->display.pps.mutex);
+       lockdep_assert_held(&display->pps.mutex);
 
        if (!pps_delays_valid(&intel_dp->pps.bios_pps_delays))
                intel_pps_readout_hw_state(intel_dp, &intel_dp->pps.bios_pps_delays);
 static void pps_init_delays_spec(struct intel_dp *intel_dp,
                                 struct edp_power_seq *spec)
 {
-       struct drm_i915_private *dev_priv = dp_to_i915(intel_dp);
+       struct intel_display *display = to_intel_display(intel_dp);
 
-       lockdep_assert_held(&dev_priv->display.pps.mutex);
+       lockdep_assert_held(&display->pps.mutex);
 
        /* Upper limits from eDP 1.3 spec. Note that we use the clunky units of
         * our hw here, which are all in 100usec. */
 
 static void pps_init_delays(struct intel_dp *intel_dp)
 {
-       struct drm_i915_private *dev_priv = dp_to_i915(intel_dp);
+       struct intel_display *display = to_intel_display(intel_dp);
        struct edp_power_seq cur, vbt, spec,
                *final = &intel_dp->pps.pps_delays;
 
-       lockdep_assert_held(&dev_priv->display.pps.mutex);
+       lockdep_assert_held(&display->pps.mutex);
 
        /* already initialized? */
        if (pps_delays_valid(final))
        intel_dp->pps.panel_power_cycle_delay = get_delay(t11_t12);
 #undef get_delay
 
-       drm_dbg_kms(&dev_priv->drm,
+       drm_dbg_kms(display->drm,
                    "panel power up delay %d, power down delay %d, power cycle delay %d\n",
                    intel_dp->pps.panel_power_up_delay,
                    intel_dp->pps.panel_power_down_delay,
                    intel_dp->pps.panel_power_cycle_delay);
 
-       drm_dbg_kms(&dev_priv->drm, "backlight on delay %d, off delay %d\n",
+       drm_dbg_kms(display->drm, "backlight on delay %d, off delay %d\n",
                    intel_dp->pps.backlight_on_delay,
                    intel_dp->pps.backlight_off_delay);
 
 
 static void pps_init_registers(struct intel_dp *intel_dp, bool force_disable_vdd)
 {
-       struct drm_i915_private *dev_priv = dp_to_i915(intel_dp);
+       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(dev_priv)->rawclk_freq / 1000;
+       int div = DISPLAY_RUNTIME_INFO(display)->rawclk_freq / 1000;
        struct pps_registers regs;
        enum port port = dp_to_dig_port(intel_dp)->base.port;
        const struct edp_power_seq *seq = &intel_dp->pps.pps_delays;
 
-       lockdep_assert_held(&dev_priv->display.pps.mutex);
+       lockdep_assert_held(&display->pps.mutex);
 
        intel_pps_get_registers(intel_dp, ®s);
 
        if (force_disable_vdd) {
                u32 pp = ilk_get_pp_control(intel_dp);
 
-               drm_WARN(&dev_priv->drm, pp & PANEL_POWER_ON,
+               drm_WARN(display->drm, pp & PANEL_POWER_ON,
                         "Panel power already on\n");
 
                if (pp & EDP_FORCE_VDD)
-                       drm_dbg_kms(&dev_priv->drm,
+                       drm_dbg_kms(display->drm,
                                    "VDD already on, disabling first\n");
 
                pp &= ~EDP_FORCE_VDD;
 
-               intel_de_write(dev_priv, regs.pp_ctrl, pp);
+               intel_de_write(display, regs.pp_ctrl, pp);
        }
 
        pp_on = REG_FIELD_PREP(PANEL_POWER_UP_DELAY_MASK, seq->t1_t3) |
 
        pp_on |= port_sel;
 
-       intel_de_write(dev_priv, regs.pp_on, pp_on);
-       intel_de_write(dev_priv, regs.pp_off, pp_off);
+       intel_de_write(display, regs.pp_on, pp_on);
+       intel_de_write(display, regs.pp_off, pp_off);
 
        /*
         * Compute the divisor for the pp clock, simply match the Bspec formula.
         */
        if (i915_mmio_reg_valid(regs.pp_div))
-               intel_de_write(dev_priv, regs.pp_div,
+               intel_de_write(display, regs.pp_div,
                               REG_FIELD_PREP(PP_REFERENCE_DIVIDER_MASK, (100 * div) / 2 - 1) | REG_FIELD_PREP(PANEL_POWER_CYCLE_DELAY_MASK, DIV_ROUND_UP(seq->t11_t12, 1000)));
        else
-               intel_de_rmw(dev_priv, regs.pp_ctrl, BXT_POWER_CYCLE_DELAY_MASK,
+               intel_de_rmw(display, regs.pp_ctrl, BXT_POWER_CYCLE_DELAY_MASK,
                             REG_FIELD_PREP(BXT_POWER_CYCLE_DELAY_MASK,
                                            DIV_ROUND_UP(seq->t11_t12, 1000)));
 
-       drm_dbg_kms(&dev_priv->drm,
+       drm_dbg_kms(display->drm,
                    "panel power sequencer register settings: PP_ON %#x, PP_OFF %#x, PP_DIV %#x\n",
-                   intel_de_read(dev_priv, regs.pp_on),
-                   intel_de_read(dev_priv, regs.pp_off),
+                   intel_de_read(display, regs.pp_on),
+                   intel_de_read(display, regs.pp_off),
                    i915_mmio_reg_valid(regs.pp_div) ?
-                   intel_de_read(dev_priv, regs.pp_div) :
-                   (intel_de_read(dev_priv, regs.pp_ctrl) & BXT_POWER_CYCLE_DELAY_MASK));
+                   intel_de_read(display, regs.pp_div) :
+                   (intel_de_read(display, regs.pp_ctrl) & BXT_POWER_CYCLE_DELAY_MASK));
 }
 
 void intel_pps_encoder_reset(struct intel_dp *intel_dp)
 {
-       struct drm_i915_private *i915 = dp_to_i915(intel_dp);
+       struct intel_display *display = to_intel_display(intel_dp);
+       struct drm_i915_private *i915 = to_i915(display->drm);
        intel_wakeref_t wakeref;
 
        if (!intel_dp_is_edp(intel_dp))
 
 static void pps_init_late(struct intel_dp *intel_dp)
 {
-       struct drm_i915_private *i915 = dp_to_i915(intel_dp);
+       struct intel_display *display = to_intel_display(intel_dp);
+       struct drm_i915_private *i915 = to_i915(display->drm);
        struct intel_encoder *encoder = &dp_to_dig_port(intel_dp)->base;
        struct intel_connector *connector = intel_dp->attached_connector;
 
        if (IS_VALLEYVIEW(i915) || IS_CHERRYVIEW(i915))
                return;
 
-       if (intel_num_pps(i915) < 2)
+       if (intel_num_pps(display) < 2)
                return;
 
-       drm_WARN(&i915->drm, connector->panel.vbt.backlight.controller >= 0 &&
+       drm_WARN(display->drm,
+                connector->panel.vbt.backlight.controller >= 0 &&
                 intel_dp->pps.pps_idx != connector->panel.vbt.backlight.controller,
                 "[ENCODER:%d:%s] power sequencer mismatch: %d (initial) vs. %d (VBT)\n",
                 encoder->base.base.id, encoder->base.name,
        }
 }
 
-void intel_pps_unlock_regs_wa(struct drm_i915_private *dev_priv)
+void intel_pps_unlock_regs_wa(struct intel_display *display)
 {
        int pps_num;
        int pps_idx;
 
-       if (!HAS_DISPLAY(dev_priv) || HAS_DDI(dev_priv))
+       if (!HAS_DISPLAY(display) || HAS_DDI(display))
                return;
        /*
         * This w/a is needed at least on CPT/PPT, but to be sure apply it
         * everywhere where registers can be write protected.
         */
-       pps_num = intel_num_pps(dev_priv);
+       pps_num = intel_num_pps(display);
 
        for (pps_idx = 0; pps_idx < pps_num; pps_idx++)
-               intel_de_rmw(dev_priv, PP_CONTROL(dev_priv, pps_idx),
+               intel_de_rmw(display, PP_CONTROL(display, pps_idx),
                             PANEL_UNLOCK_MASK, PANEL_UNLOCK_REGS);
 }
 
-void intel_pps_setup(struct drm_i915_private *i915)
+void intel_pps_setup(struct intel_display *display)
 {
+       struct drm_i915_private *i915 = to_i915(display->drm);
+
        if (HAS_PCH_SPLIT(i915) || IS_GEMINILAKE(i915) || IS_BROXTON(i915))
-               i915->display.pps.mmio_base = PCH_PPS_BASE;
+               display->pps.mmio_base = PCH_PPS_BASE;
        else if (IS_VALLEYVIEW(i915) || IS_CHERRYVIEW(i915))
-               i915->display.pps.mmio_base = VLV_PPS_BASE;
+               display->pps.mmio_base = VLV_PPS_BASE;
        else
-               i915->display.pps.mmio_base = PPS_BASE;
+               display->pps.mmio_base = PPS_BASE;
 }
 
 static int intel_pps_show(struct seq_file *m, void *data)
                                    connector, &intel_pps_fops);
 }
 
-void assert_pps_unlocked(struct drm_i915_private *dev_priv, enum pipe pipe)
+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;
        bool locked = true;
 
-       if (drm_WARN_ON(&dev_priv->drm, HAS_DDI(dev_priv)))
+       if (drm_WARN_ON(display->drm, HAS_DDI(display)))
                return;
 
        if (HAS_PCH_SPLIT(dev_priv)) {
                u32 port_sel;
 
-               pp_reg = PP_CONTROL(dev_priv, 0);
-               port_sel = intel_de_read(dev_priv, PP_ON_DELAYS(dev_priv, 0)) & PANEL_PORT_SELECT_MASK;
+               pp_reg = PP_CONTROL(display, 0);
+               port_sel = intel_de_read(display, PP_ON_DELAYS(display, 0)) &
+                       PANEL_PORT_SELECT_MASK;
 
                switch (port_sel) {
                case PANEL_PORT_SELECT_LVDS:
                }
        } else if (IS_VALLEYVIEW(dev_priv) || IS_CHERRYVIEW(dev_priv)) {
                /* presumably write lock depends on pipe, not port select */
-               pp_reg = PP_CONTROL(dev_priv, pipe);
+               pp_reg = PP_CONTROL(display, pipe);
                panel_pipe = pipe;
        } else {
                u32 port_sel;
 
-               pp_reg = PP_CONTROL(dev_priv, 0);
-               port_sel = intel_de_read(dev_priv, PP_ON_DELAYS(dev_priv, 0)) & PANEL_PORT_SELECT_MASK;
+               pp_reg = PP_CONTROL(display, 0);
+               port_sel = intel_de_read(display, PP_ON_DELAYS(display, 0)) &
+                       PANEL_PORT_SELECT_MASK;
 
-               drm_WARN_ON(&dev_priv->drm,
+               drm_WARN_ON(display->drm,
                            port_sel != PANEL_PORT_SELECT_LVDS);
                intel_lvds_port_enabled(dev_priv, LVDS, &panel_pipe);
        }
 
-       val = intel_de_read(dev_priv, pp_reg);
+       val = intel_de_read(display, pp_reg);
        if (!(val & PANEL_POWER_ON) ||
            ((val & PANEL_UNLOCK_MASK) == PANEL_UNLOCK_REGS))
                locked = false;