drm/i915/wm: convert skl_watermark.h external interfaces to struct intel_display

Going forward, struct intel_display is the main display device data
pointer. Convert the skl_watermark.h interface to struct intel_display.

Reviewed-by: Suraj Kandpal <suraj.kandpal@intel.com>
Link: https://lore.kernel.org/r/cd2b1863dee25b69b4766090dd183a7467c4edea.1744119460.git.jani.nikula@intel.com
Signed-off-by: Jani Nikula <jani.nikula@intel.com>

diff --git a/drivers/gpu/drm/i915/display/intel_bw.c b/drivers/gpu/drm/i915/display/intel_bw.c
index e819db78dfbdece8849f7ffa26408206979fcb46..a5dd2932b852d656a70794e91f1eb0dec5ec98e6 100644 (file)
--- a/drivers/gpu/drm/i915/display/intel_bw.c
+++ b/drivers/gpu/drm/i915/display/intel_bw.c
@@ -976,7 +976,6 @@ static int mtl_find_qgv_points(struct intel_display *display,
                               unsigned int num_active_planes,
                               struct intel_bw_state *new_bw_state)
 {
-       struct drm_i915_private *i915 = to_i915(display->drm);
        unsigned int best_rate = UINT_MAX;
        unsigned int num_qgv_points = display->bw.max[0].num_qgv_points;
        unsigned int qgv_peak_bw  = 0;
@@ -992,7 +991,7 @@ static int mtl_find_qgv_points(struct intel_display *display,
         * for qgv peak bw in PM Demand request. So assign UINT_MAX if SAGV is
         * not enabled. PM Demand code will clamp the value for the register
         */
-       if (!intel_can_enable_sagv(i915, new_bw_state)) {
+       if (!intel_can_enable_sagv(display, new_bw_state)) {
                new_bw_state->qgv_point_peakbw = U16_MAX;
                drm_dbg_kms(display->drm, "No SAGV, use UINT_MAX as peak bw.");
                return 0;
@@ -1049,7 +1048,6 @@ static int icl_find_qgv_points(struct intel_display *display,
                               const struct intel_bw_state *old_bw_state,
                               struct intel_bw_state *new_bw_state)
 {
-       struct drm_i915_private *i915 = to_i915(display->drm);
        unsigned int num_psf_gv_points = display->bw.max[0].num_psf_gv_points;
        unsigned int num_qgv_points = display->bw.max[0].num_qgv_points;
        u16 psf_points = 0;
@@ -1106,7 +1104,7 @@ static int icl_find_qgv_points(struct intel_display *display,
         * we can't enable SAGV due to the increased memory latency it may
         * cause.
         */
-       if (!intel_can_enable_sagv(i915, new_bw_state)) {
+       if (!intel_can_enable_sagv(display, new_bw_state)) {
                qgv_points = icl_max_bw_qgv_point_mask(display, num_active_planes);
                drm_dbg_kms(display->drm, "No SAGV, using single QGV point mask 0x%x\n",
                            qgv_points);
@@ -1195,8 +1193,7 @@ static void skl_plane_calc_dbuf_bw(struct intel_dbuf_bw *dbuf_bw,
                                   unsigned int data_rate)
 {
        struct intel_display *display = to_intel_display(crtc);
-       struct drm_i915_private *i915 = to_i915(display->drm);
-       unsigned int dbuf_mask = skl_ddb_dbuf_slice_mask(i915, ddb);
+       unsigned int dbuf_mask = skl_ddb_dbuf_slice_mask(display, ddb);
        enum dbuf_slice slice;
 
        /*
@@ -1446,7 +1443,6 @@ static int intel_bw_modeset_checks(struct intel_atomic_state *state)
 static int intel_bw_check_sagv_mask(struct intel_atomic_state *state)
 {
        struct intel_display *display = to_intel_display(state);
-       struct drm_i915_private *i915 = to_i915(display->drm);
        const struct intel_crtc_state *old_crtc_state;
        const struct intel_crtc_state *new_crtc_state;
        const struct intel_bw_state *old_bw_state = NULL;
@@ -1475,8 +1471,8 @@ static int intel_bw_check_sagv_mask(struct intel_atomic_state *state)
        if (!new_bw_state)
                return 0;
 
-       if (intel_can_enable_sagv(i915, new_bw_state) !=
-           intel_can_enable_sagv(i915, old_bw_state)) {
+       if (intel_can_enable_sagv(display, new_bw_state) !=
+           intel_can_enable_sagv(display, old_bw_state)) {
                ret = intel_atomic_serialize_global_state(&new_bw_state->base);
                if (ret)
                        return ret;
@@ -1492,13 +1488,12 @@ static int intel_bw_check_sagv_mask(struct intel_atomic_state *state)
 int intel_bw_atomic_check(struct intel_atomic_state *state, bool any_ms)
 {
        struct intel_display *display = to_intel_display(state);
-       struct drm_i915_private *i915 = to_i915(display->drm);
        bool changed = false;
        struct intel_bw_state *new_bw_state;
        const struct intel_bw_state *old_bw_state;
        int ret;
 
-       if (DISPLAY_VER(i915) < 9)
+       if (DISPLAY_VER(display) < 9)
                return 0;
 
        if (any_ms) {
@@ -1523,8 +1518,8 @@ int intel_bw_atomic_check(struct intel_atomic_state *state, bool any_ms)
        new_bw_state = intel_atomic_get_new_bw_state(state);
 
        if (new_bw_state &&
-           intel_can_enable_sagv(i915, old_bw_state) !=
-           intel_can_enable_sagv(i915, new_bw_state))
+           intel_can_enable_sagv(display, old_bw_state) !=
+           intel_can_enable_sagv(display, new_bw_state))
                changed = true;
 
        /*
@@ -1628,7 +1623,6 @@ static const struct intel_global_state_funcs intel_bw_funcs = {
 
 int intel_bw_init(struct intel_display *display)
 {
-       struct drm_i915_private *i915 = to_i915(display->drm);
        struct intel_bw_state *state;
 
        state = kzalloc(sizeof(*state), GFP_KERNEL);
@@ -1642,7 +1636,7 @@ int intel_bw_init(struct intel_display *display)
         * Limit this only if we have SAGV. And for Display version 14 onwards
         * sagv is handled though pmdemand requests
         */
-       if (intel_has_sagv(i915) && IS_DISPLAY_VER(display, 11, 13))
+       if (intel_has_sagv(display) && IS_DISPLAY_VER(display, 11, 13))
                icl_force_disable_sagv(display, state);
 
        return 0;

diff --git a/drivers/gpu/drm/i915/display/intel_cdclk.c b/drivers/gpu/drm/i915/display/intel_cdclk.c
index ea2fbee2d62f21a79be92ae8276f44d5ea718fa0..6830950aae3f2379350691353195eb473dcf25c8 100644 (file)
--- a/drivers/gpu/drm/i915/display/intel_cdclk.c
+++ b/drivers/gpu/drm/i915/display/intel_cdclk.c
@@ -1972,9 +1972,7 @@ int intel_mdclk_cdclk_ratio(struct intel_display *display,
 static void xe2lpd_mdclk_cdclk_ratio_program(struct intel_display *display,
                                             const struct intel_cdclk_config *cdclk_config)
 {
-       struct drm_i915_private *i915 = to_i915(display->drm);
-
-       intel_dbuf_mdclk_cdclk_ratio_update(i915,
+       intel_dbuf_mdclk_cdclk_ratio_update(display,
                                            intel_mdclk_cdclk_ratio(display, cdclk_config),
                                            cdclk_config->joined_mbus);
 }

diff --git a/drivers/gpu/drm/i915/display/intel_display.c b/drivers/gpu/drm/i915/display/intel_display.c
index 16c4e25f958ac96a7c7aba279ab3132b9372ed6b..745eef470badd7bf9a9d37dfdbb403e1f94e4de1 100644 (file)
--- a/drivers/gpu/drm/i915/display/intel_display.c
+++ b/drivers/gpu/drm/i915/display/intel_display.c
@@ -4162,8 +4162,6 @@ static u16 hsw_ips_linetime_wm(const struct intel_crtc_state *crtc_state,
 static u16 skl_linetime_wm(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);
        const struct drm_display_mode *pipe_mode =
                &crtc_state->hw.pipe_mode;
        int linetime_wm;
@@ -4176,7 +4174,7 @@ static u16 skl_linetime_wm(const struct intel_crtc_state *crtc_state)
 
        /* Display WA #1135: BXT:ALL GLK:ALL */
        if ((display->platform.geminilake || display->platform.broxton) &&
-           skl_watermark_ipc_enabled(dev_priv))
+           skl_watermark_ipc_enabled(display))
                linetime_wm /= 2;
 
        return min(linetime_wm, 0x1ff);

diff --git a/drivers/gpu/drm/i915/display/intel_display_driver.c b/drivers/gpu/drm/i915/display/intel_display_driver.c
index 44cf34517a626c4d60f0d2a9a607c5ff6876b216..f6d5d51dda0fb38824c83c0be8e36bba92f1c7a6 100644 (file)
--- a/drivers/gpu/drm/i915/display/intel_display_driver.c
+++ b/drivers/gpu/drm/i915/display/intel_display_driver.c
@@ -252,7 +252,7 @@ int intel_display_driver_probe_noirq(struct intel_display *display)
        if (ret)
                goto cleanup_vga_client_pw_domain_dmc;
 
-       ret = intel_dbuf_init(i915);
+       ret = intel_dbuf_init(display);
        if (ret)
                goto cleanup_vga_client_pw_domain_dmc;
 
@@ -491,7 +491,6 @@ err_mode_config:
 /* part #3: call after gem init */
 int intel_display_driver_probe(struct intel_display *display)
 {
-       struct drm_i915_private *i915 = to_i915(display->drm);
        int ret;
 
        if (!HAS_DISPLAY(display))
@@ -519,7 +518,7 @@ int intel_display_driver_probe(struct intel_display *display)
        /* Only enable hotplug handling once the fbdev is fully set up. */
        intel_hpd_init(display);
 
-       skl_watermark_ipc_init(i915);
+       skl_watermark_ipc_init(display);
 
        return 0;
 }
@@ -726,7 +725,6 @@ __intel_display_driver_resume(struct intel_display *display,
 
 void intel_display_driver_resume(struct intel_display *display)
 {
-       struct drm_i915_private *i915 = to_i915(display->drm);
        struct drm_atomic_state *state = display->restore.modeset_state;
        struct drm_modeset_acquire_ctx ctx;
        int ret;
@@ -754,7 +752,7 @@ void intel_display_driver_resume(struct intel_display *display)
        if (!ret)
                ret = __intel_display_driver_resume(display, state, &ctx);
 
-       skl_watermark_ipc_update(i915);
+       skl_watermark_ipc_update(display);
        drm_modeset_drop_locks(&ctx);
        drm_modeset_acquire_fini(&ctx);
 

diff --git a/drivers/gpu/drm/i915/display/intel_dsb.c b/drivers/gpu/drm/i915/display/intel_dsb.c
index 0ddcdedf5453c567441e9ff1f07dbf2303164c1c..72fe390c5af22a6ba650c9c43f6ca36aaf0edf97 100644 (file)
--- a/drivers/gpu/drm/i915/display/intel_dsb.c
+++ b/drivers/gpu/drm/i915/display/intel_dsb.c
@@ -143,10 +143,10 @@ static int dsb_vtotal(struct intel_atomic_state *state,
 static int dsb_dewake_scanline_start(struct intel_atomic_state *state,
                                     struct intel_crtc *crtc)
 {
+       struct intel_display *display = to_intel_display(state);
        const struct intel_crtc_state *crtc_state =
                intel_pre_commit_crtc_state(state, crtc);
-       struct drm_i915_private *i915 = to_i915(state->base.dev);
-       unsigned int latency = skl_watermark_max_latency(i915, 0);
+       unsigned int latency = skl_watermark_max_latency(display, 0);
 
        return intel_mode_vdisplay(&crtc_state->hw.adjusted_mode) -
                intel_usecs_to_scanlines(&crtc_state->hw.adjusted_mode, latency);

diff --git a/drivers/gpu/drm/i915/display/intel_wm.c b/drivers/gpu/drm/i915/display/intel_wm.c
index 9899e4c3ae9665ba05c73a89b1aec36a42fd5e4e..43c69a672bb34a50b819b3c48417eeebcb8442b4 100644 (file)
--- a/drivers/gpu/drm/i915/display/intel_wm.c
+++ b/drivers/gpu/drm/i915/display/intel_wm.c
@@ -172,7 +172,7 @@ void intel_wm_init(struct intel_display *display)
        struct drm_i915_private *i915 = to_i915(display->drm);
 
        if (DISPLAY_VER(display) >= 9)
-               skl_wm_init(i915);
+               skl_wm_init(display);
        else
                i9xx_wm_init(i915);
 }
@@ -389,7 +389,6 @@ static const struct file_operations i915_cur_wm_latency_fops = {
 
 void intel_wm_debugfs_register(struct intel_display *display)
 {
-       struct drm_i915_private *i915 = to_i915(display->drm);
        struct drm_minor *minor = display->drm->primary;
 
        debugfs_create_file("i915_pri_wm_latency", 0644, minor->debugfs_root,
@@ -401,5 +400,5 @@ void intel_wm_debugfs_register(struct intel_display *display)
        debugfs_create_file("i915_cur_wm_latency", 0644, minor->debugfs_root,
                            display, &i915_cur_wm_latency_fops);
 
-       skl_watermark_debugfs_register(i915);
+       skl_watermark_debugfs_register(display);
 }

diff --git a/drivers/gpu/drm/i915/display/skl_watermark.c b/drivers/gpu/drm/i915/display/skl_watermark.c
index 9ca86f44f365056c18288170515ed59eb4c44849..74df2cae4146d57e8a7f30e501e1d2eea10a7e07 100644 (file)
--- a/drivers/gpu/drm/i915/display/skl_watermark.c
+++ b/drivers/gpu/drm/i915/display/skl_watermark.c
@@ -76,10 +76,8 @@ static bool skl_needs_memory_bw_wa(struct drm_i915_private *i915)
 }
 
 bool
-intel_has_sagv(struct drm_i915_private *i915)
+intel_has_sagv(struct intel_display *display)
 {
-       struct intel_display *display = &i915->display;
-
        return HAS_SAGV(display) && display->sagv.status != I915_SAGV_NOT_CONTROLLED;
 }
 
@@ -135,7 +133,7 @@ static void intel_sagv_init(struct drm_i915_private *i915)
        display->sagv.block_time_us = intel_sagv_block_time(i915);
 
        drm_dbg_kms(display->drm, "SAGV supported: %s, original SAGV block time: %u us\n",
-                   str_yes_no(intel_has_sagv(i915)), display->sagv.block_time_us);
+                   str_yes_no(intel_has_sagv(display)), display->sagv.block_time_us);
 
        /* avoid overflow when adding with wm0 latency/etc. */
        if (drm_WARN(display->drm, display->sagv.block_time_us > U16_MAX,
@@ -143,7 +141,7 @@ static void intel_sagv_init(struct drm_i915_private *i915)
                     display->sagv.block_time_us))
                display->sagv.block_time_us = 0;
 
-       if (!intel_has_sagv(i915))
+       if (!intel_has_sagv(display))
                display->sagv.block_time_us = 0;
 }
 
@@ -160,9 +158,10 @@ static void intel_sagv_init(struct drm_i915_private *i915)
  */
 static void skl_sagv_enable(struct drm_i915_private *i915)
 {
+       struct intel_display *display = &i915->display;
        int ret;
 
-       if (!intel_has_sagv(i915))
+       if (!intel_has_sagv(display))
                return;
 
        if (i915->display.sagv.status == I915_SAGV_ENABLED)
@@ -192,9 +191,10 @@ static void skl_sagv_enable(struct drm_i915_private *i915)
 
 static void skl_sagv_disable(struct drm_i915_private *i915)
 {
+       struct intel_display *display = &i915->display;
        int ret;
 
-       if (!intel_has_sagv(i915))
+       if (!intel_has_sagv(display))
                return;
 
        if (i915->display.sagv.status == I915_SAGV_DISABLED)
@@ -224,6 +224,7 @@ static void skl_sagv_disable(struct drm_i915_private *i915)
 
 static void skl_sagv_pre_plane_update(struct intel_atomic_state *state)
 {
+       struct intel_display *display = to_intel_display(state);
        struct drm_i915_private *i915 = to_i915(state->base.dev);
        const struct intel_bw_state *new_bw_state =
                intel_atomic_get_new_bw_state(state);
@@ -231,12 +232,13 @@ static void skl_sagv_pre_plane_update(struct intel_atomic_state *state)
        if (!new_bw_state)
                return;
 
-       if (!intel_can_enable_sagv(i915, new_bw_state))
+       if (!intel_can_enable_sagv(display, new_bw_state))
                skl_sagv_disable(i915);
 }
 
 static void skl_sagv_post_plane_update(struct intel_atomic_state *state)
 {
+       struct intel_display *display = to_intel_display(state);
        struct drm_i915_private *i915 = to_i915(state->base.dev);
        const struct intel_bw_state *new_bw_state =
                intel_atomic_get_new_bw_state(state);
@@ -244,7 +246,7 @@ static void skl_sagv_post_plane_update(struct intel_atomic_state *state)
        if (!new_bw_state)
                return;
 
-       if (intel_can_enable_sagv(i915, new_bw_state))
+       if (intel_can_enable_sagv(display, new_bw_state))
                skl_sagv_enable(i915);
 }
 
@@ -316,6 +318,7 @@ static void icl_sagv_post_plane_update(struct intel_atomic_state *state)
 
 void intel_sagv_pre_plane_update(struct intel_atomic_state *state)
 {
+       struct intel_display *display = to_intel_display(state);
        struct drm_i915_private *i915 = to_i915(state->base.dev);
 
        /*
@@ -325,7 +328,7 @@ void intel_sagv_pre_plane_update(struct intel_atomic_state *state)
         * disabled in a BIOS, we are not even allowed to send a PCode request,
         * as it will throw an error. So have to check it here.
         */
-       if (!intel_has_sagv(i915))
+       if (!intel_has_sagv(display))
                return;
 
        if (DISPLAY_VER(i915) >= 11)
@@ -336,6 +339,7 @@ void intel_sagv_pre_plane_update(struct intel_atomic_state *state)
 
 void intel_sagv_post_plane_update(struct intel_atomic_state *state)
 {
+       struct intel_display *display = to_intel_display(state);
        struct drm_i915_private *i915 = to_i915(state->base.dev);
 
        /*
@@ -345,7 +349,7 @@ void intel_sagv_post_plane_update(struct intel_atomic_state *state)
         * disabled in a BIOS, we are not even allowed to send a PCode request,
         * as it will throw an error. So have to check it here.
         */
-       if (!intel_has_sagv(i915))
+       if (!intel_has_sagv(display))
                return;
 
        if (DISPLAY_VER(i915) >= 11)
@@ -356,12 +360,13 @@ void intel_sagv_post_plane_update(struct intel_atomic_state *state)
 
 static bool skl_crtc_can_enable_sagv(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);
        enum plane_id plane_id;
        int max_level = INT_MAX;
 
-       if (!intel_has_sagv(i915))
+       if (!intel_has_sagv(display))
                return false;
 
        if (!crtc_state->hw.active)
@@ -448,10 +453,10 @@ bool intel_crtc_can_enable_sagv(const struct intel_crtc_state *crtc_state)
                return skl_crtc_can_enable_sagv(crtc_state);
 }
 
-bool intel_can_enable_sagv(struct drm_i915_private *i915,
+bool intel_can_enable_sagv(struct intel_display *display,
                           const struct intel_bw_state *bw_state)
 {
-       if (DISPLAY_VER(i915) < 11 &&
+       if (DISPLAY_VER(display) < 11 &&
            bw_state->active_pipes && !is_power_of_2(bw_state->active_pipes))
                return false;
 
@@ -506,9 +511,10 @@ static unsigned int mbus_ddb_offset(struct drm_i915_private *i915, u8 slice_mask
        return ddb.start;
 }
 
-u32 skl_ddb_dbuf_slice_mask(struct drm_i915_private *i915,
+u32 skl_ddb_dbuf_slice_mask(struct intel_display *display,
                            const struct skl_ddb_entry *entry)
 {
+       struct drm_i915_private *i915 = to_i915(display->drm);
        int slice_size = intel_dbuf_slice_size(i915);
        enum dbuf_slice start_slice, end_slice;
        u8 slice_mask = 0;
@@ -674,6 +680,7 @@ static void skl_compute_plane_wm(const struct intel_crtc_state *crtc_state,
 static unsigned int skl_wm_latency(struct drm_i915_private *i915, int level,
                                   const struct skl_wm_params *wp)
 {
+       struct intel_display *display = &i915->display;
        unsigned int latency = i915->display.wm.skl_latency[level];
 
        if (latency == 0)
@@ -684,7 +691,7 @@ static unsigned int skl_wm_latency(struct drm_i915_private *i915, int level,
         * Display WA #1141: kbl,cfl
         */
        if ((IS_KABYLAKE(i915) || IS_COFFEELAKE(i915) || IS_COMETLAKE(i915)) &&
-           skl_watermark_ipc_enabled(i915))
+           skl_watermark_ipc_enabled(display))
                latency += 4;
 
        if (skl_needs_memory_bw_wa(i915) && wp && wp->x_tiled)
@@ -2050,11 +2057,12 @@ static void skl_compute_transition_wm(struct drm_i915_private *i915,
                                      const struct skl_wm_level *wm0,
                                      const struct skl_wm_params *wp)
 {
+       struct intel_display *display = &i915->display;
        u16 trans_min, trans_amount, trans_y_tile_min;
        u16 wm0_blocks, trans_offset, blocks;
 
        /* Transition WM don't make any sense if ipc is disabled */
-       if (!skl_watermark_ipc_enabled(i915))
+       if (!skl_watermark_ipc_enabled(display))
                return;
 
        /*
@@ -2939,7 +2947,6 @@ void
 intel_program_dpkgc_latency(struct intel_atomic_state *state)
 {
        struct intel_display *display = to_intel_display(state);
-       struct drm_i915_private *i915 = to_i915(display->drm);
        struct intel_crtc *crtc;
        struct intel_crtc_state *new_crtc_state;
        u32 latency = LNL_PKG_C_LATENCY_MASK;
@@ -2965,7 +2972,7 @@ intel_program_dpkgc_latency(struct intel_atomic_state *state)
                added_wake_time = DSB_EXE_TIME +
                        display->sagv.block_time_us;
 
-               latency = skl_watermark_max_latency(i915, 1);
+               latency = skl_watermark_max_latency(display, 1);
 
                /* Wa_22020432604 */
                if ((DISPLAY_VER(display) == 20 || DISPLAY_VER(display) == 30) && !latency) {
@@ -3172,7 +3179,7 @@ static void skl_wm_get_hw_state(struct intel_display *display)
 
                /* The slices actually used by the planes on the pipe */
                dbuf_state->slices[pipe] =
-                       skl_ddb_dbuf_slice_mask(i915, &crtc_state->wm.skl.ddb);
+                       skl_ddb_dbuf_slice_mask(display, &crtc_state->wm.skl.ddb);
 
                drm_dbg_kms(display->drm,
                            "[CRTC:%d:%s] dbuf slices 0x%x, ddb (%d - %d), active pipes 0x%x, mbus joined: %s\n",
@@ -3185,18 +3192,18 @@ static void skl_wm_get_hw_state(struct intel_display *display)
        dbuf_state->enabled_slices = display->dbuf.enabled_slices;
 }
 
-bool skl_watermark_ipc_enabled(struct drm_i915_private *i915)
+bool skl_watermark_ipc_enabled(struct intel_display *display)
 {
-       return i915->display.wm.ipc_enabled;
+       return display->wm.ipc_enabled;
 }
 
-void skl_watermark_ipc_update(struct drm_i915_private *i915)
+void skl_watermark_ipc_update(struct intel_display *display)
 {
-       if (!HAS_IPC(i915))
+       if (!HAS_IPC(display))
                return;
 
-       intel_de_rmw(i915, DISP_ARB_CTL2, DISP_IPC_ENABLE,
-                    skl_watermark_ipc_enabled(i915) ? DISP_IPC_ENABLE : 0);
+       intel_de_rmw(display, DISP_ARB_CTL2, DISP_IPC_ENABLE,
+                    skl_watermark_ipc_enabled(display) ? DISP_IPC_ENABLE : 0);
 }
 
 static bool skl_watermark_ipc_can_enable(struct drm_i915_private *i915)
@@ -3214,14 +3221,16 @@ static bool skl_watermark_ipc_can_enable(struct drm_i915_private *i915)
        return true;
 }
 
-void skl_watermark_ipc_init(struct drm_i915_private *i915)
+void skl_watermark_ipc_init(struct intel_display *display)
 {
-       if (!HAS_IPC(i915))
+       struct drm_i915_private *i915 = to_i915(display->drm);
+
+       if (!HAS_IPC(display))
                return;
 
-       i915->display.wm.ipc_enabled = skl_watermark_ipc_can_enable(i915);
+       display->wm.ipc_enabled = skl_watermark_ipc_can_enable(i915);
 
-       skl_watermark_ipc_update(i915);
+       skl_watermark_ipc_update(display);
 }
 
 static void
@@ -3377,9 +3386,8 @@ intel_atomic_get_dbuf_state(struct intel_atomic_state *state)
        return to_intel_dbuf_state(dbuf_state);
 }
 
-int intel_dbuf_init(struct drm_i915_private *i915)
+int intel_dbuf_init(struct intel_display *display)
 {
-       struct intel_display *display = &i915->display;
        struct intel_dbuf_state *dbuf_state;
 
        dbuf_state = kzalloc(sizeof(*dbuf_state), GFP_KERNEL);
@@ -3501,10 +3509,9 @@ int intel_dbuf_state_set_mdclk_cdclk_ratio(struct intel_atomic_state *state,
        return intel_atomic_lock_global_state(&dbuf_state->base);
 }
 
-void intel_dbuf_mdclk_cdclk_ratio_update(struct drm_i915_private *i915,
+void intel_dbuf_mdclk_cdclk_ratio_update(struct intel_display *display,
                                         int ratio, bool joined_mbus)
 {
-       struct intel_display *display = &i915->display;
        enum dbuf_slice slice;
 
        if (!HAS_MBUS_JOINING(display))
@@ -3528,7 +3535,7 @@ void intel_dbuf_mdclk_cdclk_ratio_update(struct drm_i915_private *i915,
 
 static void intel_dbuf_mdclk_min_tracker_update(struct intel_atomic_state *state)
 {
-       struct drm_i915_private *i915 = to_i915(state->base.dev);
+       struct intel_display *display = to_intel_display(state);
        const struct intel_dbuf_state *old_dbuf_state =
                intel_atomic_get_old_dbuf_state(state);
        const struct intel_dbuf_state *new_dbuf_state =
@@ -3543,7 +3550,7 @@ static void intel_dbuf_mdclk_min_tracker_update(struct intel_atomic_state *state
                mdclk_cdclk_ratio = new_dbuf_state->mdclk_cdclk_ratio;
        }
 
-       intel_dbuf_mdclk_cdclk_ratio_update(i915, mdclk_cdclk_ratio,
+       intel_dbuf_mdclk_cdclk_ratio_update(display, mdclk_cdclk_ratio,
                                            new_dbuf_state->joined_mbus);
 }
 
@@ -3725,7 +3732,7 @@ static void skl_mbus_sanitize(struct drm_i915_private *i915)
                    dbuf_state->active_pipes);
 
        dbuf_state->joined_mbus = false;
-       intel_dbuf_mdclk_cdclk_ratio_update(i915,
+       intel_dbuf_mdclk_cdclk_ratio_update(display,
                                            dbuf_state->mdclk_cdclk_ratio,
                                            dbuf_state->joined_mbus);
        pipe_mbus_dbox_ctl_update(i915, dbuf_state);
@@ -3983,21 +3990,24 @@ static const struct intel_wm_funcs skl_wm_funcs = {
        .sanitize = skl_wm_sanitize,
 };
 
-void skl_wm_init(struct drm_i915_private *i915)
+void skl_wm_init(struct intel_display *display)
 {
+       struct drm_i915_private *i915 = to_i915(display->drm);
+
        intel_sagv_init(i915);
 
        skl_setup_wm_latency(i915);
 
-       i915->display.funcs.wm = &skl_wm_funcs;
+       display->funcs.wm = &skl_wm_funcs;
 }
 
 static int skl_watermark_ipc_status_show(struct seq_file *m, void *data)
 {
        struct drm_i915_private *i915 = m->private;
+       struct intel_display *display = &i915->display;
 
        seq_printf(m, "Isochronous Priority Control: %s\n",
-                  str_yes_no(skl_watermark_ipc_enabled(i915)));
+                  str_yes_no(skl_watermark_ipc_enabled(display)));
        return 0;
 }
 
@@ -4023,11 +4033,11 @@ static ssize_t skl_watermark_ipc_status_write(struct file *file,
                return ret;
 
        with_intel_display_rpm(display) {
-               if (!skl_watermark_ipc_enabled(i915) && enable)
+               if (!skl_watermark_ipc_enabled(display) && enable)
                        drm_info(display->drm,
                                 "Enabling IPC: WM will be proper only after next commit\n");
                display->wm.ipc_enabled = enable;
-               skl_watermark_ipc_update(i915);
+               skl_watermark_ipc_update(display);
        }
 
        return len;
@@ -4045,6 +4055,7 @@ static const struct file_operations skl_watermark_ipc_status_fops = {
 static int intel_sagv_status_show(struct seq_file *m, void *unused)
 {
        struct drm_i915_private *i915 = m->private;
+       struct intel_display *display = &i915->display;
        static const char * const sagv_status[] = {
                [I915_SAGV_UNKNOWN] = "unknown",
                [I915_SAGV_DISABLED] = "disabled",
@@ -4052,7 +4063,7 @@ static int intel_sagv_status_show(struct seq_file *m, void *unused)
                [I915_SAGV_NOT_CONTROLLED] = "not controlled",
        };
 
-       seq_printf(m, "SAGV available: %s\n", str_yes_no(intel_has_sagv(i915)));
+       seq_printf(m, "SAGV available: %s\n", str_yes_no(intel_has_sagv(display)));
        seq_printf(m, "SAGV modparam: %s\n",
                   str_enabled_disabled(i915->display.params.enable_sagv));
        seq_printf(m, "SAGV status: %s\n", sagv_status[i915->display.sagv.status]);
@@ -4063,9 +4074,9 @@ static int intel_sagv_status_show(struct seq_file *m, void *unused)
 
 DEFINE_SHOW_ATTRIBUTE(intel_sagv_status);
 
-void skl_watermark_debugfs_register(struct drm_i915_private *i915)
+void skl_watermark_debugfs_register(struct intel_display *display)
 {
-       struct intel_display *display = &i915->display;
+       struct drm_i915_private *i915 = to_i915(display->drm);
        struct drm_minor *minor = display->drm->primary;
 
        if (HAS_IPC(display))
@@ -4077,11 +4088,12 @@ void skl_watermark_debugfs_register(struct drm_i915_private *i915)
                                    &intel_sagv_status_fops);
 }
 
-unsigned int skl_watermark_max_latency(struct drm_i915_private *i915, int initial_wm_level)
+unsigned int skl_watermark_max_latency(struct intel_display *display, int initial_wm_level)
 {
+       struct drm_i915_private *i915 = to_i915(display->drm);
        int level;
 
-       for (level = i915->display.wm.num_levels - 1; level >= initial_wm_level; level--) {
+       for (level = display->wm.num_levels - 1; level >= initial_wm_level; level--) {
                unsigned int latency = skl_wm_latency(i915, level, NULL);
 
                if (latency)

diff --git a/drivers/gpu/drm/i915/display/skl_watermark.h b/drivers/gpu/drm/i915/display/skl_watermark.h
index 7e8107f808b60846b7450440ff8c797c9b2ff410..95b0b599d5c369a8362d378524fd314d4dca14d7 100644 (file)
--- a/drivers/gpu/drm/i915/display/skl_watermark.h
+++ b/drivers/gpu/drm/i915/display/skl_watermark.h
@@ -12,7 +12,6 @@
 #include "intel_global_state.h"
 #include "intel_wm_types.h"
 
-struct drm_i915_private;
 struct intel_atomic_state;
 struct intel_bw_state;
 struct intel_crtc;
@@ -28,11 +27,11 @@ u8 intel_enabled_dbuf_slices_mask(struct intel_display *display);
 void intel_sagv_pre_plane_update(struct intel_atomic_state *state);
 void intel_sagv_post_plane_update(struct intel_atomic_state *state);
 bool intel_crtc_can_enable_sagv(const struct intel_crtc_state *crtc_state);
-bool intel_can_enable_sagv(struct drm_i915_private *i915,
+bool intel_can_enable_sagv(struct intel_display *display,
                           const struct intel_bw_state *bw_state);
-bool intel_has_sagv(struct drm_i915_private *i915);
+bool intel_has_sagv(struct intel_display *display);
 
-u32 skl_ddb_dbuf_slice_mask(struct drm_i915_private *i915,
+u32 skl_ddb_dbuf_slice_mask(struct intel_display *display,
                            const struct skl_ddb_entry *entry);
 
 bool skl_ddb_allocation_overlaps(const struct skl_ddb_entry *ddb,
@@ -46,14 +45,14 @@ void skl_wm_crtc_disable_noatomic(struct intel_crtc *crtc);
 void skl_wm_plane_disable_noatomic(struct intel_crtc *crtc,
                                   struct intel_plane *plane);
 
-void skl_watermark_ipc_init(struct drm_i915_private *i915);
-void skl_watermark_ipc_update(struct drm_i915_private *i915);
-bool skl_watermark_ipc_enabled(struct drm_i915_private *i915);
-void skl_watermark_debugfs_register(struct drm_i915_private *i915);
+void skl_watermark_ipc_init(struct intel_display *display);
+void skl_watermark_ipc_update(struct intel_display *display);
+bool skl_watermark_ipc_enabled(struct intel_display *display);
+void skl_watermark_debugfs_register(struct intel_display *display);
 
-unsigned int skl_watermark_max_latency(struct drm_i915_private *i915,
+unsigned int skl_watermark_max_latency(struct intel_display *display,
                                       int initial_wm_level);
-void skl_wm_init(struct drm_i915_private *i915);
+void skl_wm_init(struct intel_display *display);
 
 const struct skl_wm_level *skl_plane_wm_level(const struct skl_pipe_wm *pipe_wm,
                                              enum plane_id plane_id,
@@ -87,13 +86,13 @@ intel_atomic_get_dbuf_state(struct intel_atomic_state *state);
 #define intel_atomic_get_new_dbuf_state(state) \
        to_intel_dbuf_state(intel_atomic_get_new_global_obj_state(state, &to_intel_display(state)->dbuf.obj))
 
-int intel_dbuf_init(struct drm_i915_private *i915);
+int intel_dbuf_init(struct intel_display *display);
 int intel_dbuf_state_set_mdclk_cdclk_ratio(struct intel_atomic_state *state,
                                           int ratio);
 
 void intel_dbuf_pre_plane_update(struct intel_atomic_state *state);
 void intel_dbuf_post_plane_update(struct intel_atomic_state *state);
-void intel_dbuf_mdclk_cdclk_ratio_update(struct drm_i915_private *i915,
+void intel_dbuf_mdclk_cdclk_ratio_update(struct intel_display *display,
                                         int ratio, bool joined_mbus);
 void intel_dbuf_mbus_pre_ddb_update(struct intel_atomic_state *state);
 void intel_dbuf_mbus_post_ddb_update(struct intel_atomic_state *state);

diff --git a/drivers/gpu/drm/i915/i915_driver.c b/drivers/gpu/drm/i915/i915_driver.c
index d78de7f620dddb514844da177724764fc16d7390..f5262b8ad237d2b88a0955474e9c8e865639bd1c 100644 (file)
--- a/drivers/gpu/drm/i915/i915_driver.c
+++ b/drivers/gpu/drm/i915/i915_driver.c
@@ -1647,7 +1647,7 @@ static int intel_runtime_resume(struct device *kdev)
                intel_hpd_poll_disable(display);
        }
 
-       skl_watermark_ipc_update(dev_priv);
+       skl_watermark_ipc_update(display);
 
        enable_rpm_wakeref_asserts(rpm);
 

diff --git a/drivers/gpu/drm/xe/display/xe_display.c b/drivers/gpu/drm/xe/display/xe_display.c
index 6bd08810f1dd73c60e6c7839cfe12ce7eb4b1446..7ddda8f5b1c489fcc79f0eed08417b8699467c66 100644 (file)
--- a/drivers/gpu/drm/xe/display/xe_display.c
+++ b/drivers/gpu/drm/xe/display/xe_display.c
@@ -531,7 +531,7 @@ void xe_display_pm_runtime_resume(struct xe_device *xe)
 
        intel_hpd_init(display);
        intel_hpd_poll_disable(display);
-       skl_watermark_ipc_update(xe);
+       skl_watermark_ipc_update(display);
 }