drm/i915: Use enum plane_id in SKL wm code
authorVille Syrjälä <ville.syrjala@linux.intel.com>
Tue, 22 Nov 2016 16:01:58 +0000 (18:01 +0200)
committerVille Syrjälä <ville.syrjala@linux.intel.com>
Wed, 23 Nov 2016 20:03:36 +0000 (22:03 +0200)
Nuke skl_wm_plane_id() and just use the new intel_plane->id.

v2: Convert skl_write_plane_wm() as well
v3: Convert skl_pipe_wm_get_hw_state() correctly
v4: Rebase due to changes in the wm code
    Drop the cursor FIXME from the total data rate calc (Paulo)
    Use the "[PLANE:%d:%s]" format in debug print (Paulo)

Cc: Matt Roper <matthew.d.roper@intel.com>
Cc: Paulo Zanoni <paulo.r.zanoni@intel.com>
Cc: Maarten Lankhorst <maarten.lankhorst@linux.intel.com>
Cc: Lyude <cpaul@redhat.com>
Signed-off-by: Ville Syrjälä <ville.syrjala@linux.intel.com>
Reviewed-by: Paulo Zanoni <paulo.r.zanoni@intel.com>
Link: http://patchwork.freedesktop.org/patch/msgid/1479830524-7882-4-git-send-email-ville.syrjala@linux.intel.com
Reviewed-by: Lyude <lyude@redhat.com>
drivers/gpu/drm/i915/intel_pm.c

index cbd0f3269b2d6a6aeecc0027914c47065252b1b4..5b7d306a1947c7f57e2f099ac6f6f8a02c040d9d 100644 (file)
@@ -2866,28 +2866,6 @@ bool ilk_disable_lp_wm(struct drm_device *dev)
 
 #define SKL_SAGV_BLOCK_TIME    30 /* µs */
 
-/*
- * Return the index of a plane in the SKL DDB and wm result arrays.  Primary
- * plane is always in slot 0, cursor is always in slot I915_MAX_PLANES-1, and
- * other universal planes are in indices 1..n.  Note that this may leave unused
- * indices between the top "sprite" plane and the cursor.
- */
-static int
-skl_wm_plane_id(const struct intel_plane *plane)
-{
-       switch (plane->base.type) {
-       case DRM_PLANE_TYPE_PRIMARY:
-               return 0;
-       case DRM_PLANE_TYPE_CURSOR:
-               return PLANE_CURSOR;
-       case DRM_PLANE_TYPE_OVERLAY:
-               return plane->plane + 1;
-       default:
-               MISSING_CASE(plane->base.type);
-               return plane->plane;
-       }
-}
-
 /*
  * FIXME: We still don't have the proper code detect if we need to apply the WA,
  * so assume we'll always need it in order to avoid underruns.
@@ -3026,7 +3004,6 @@ bool intel_can_enable_sagv(struct drm_atomic_state *state)
        struct intel_crtc *crtc;
        struct intel_plane *plane;
        struct intel_crtc_state *cstate;
-       struct skl_plane_wm *wm;
        enum pipe pipe;
        int level, latency;
 
@@ -3053,7 +3030,8 @@ bool intel_can_enable_sagv(struct drm_atomic_state *state)
                return false;
 
        for_each_intel_plane_on_crtc(dev, crtc, plane) {
-               wm = &cstate->wm.skl.optimal.planes[skl_wm_plane_id(plane)];
+               struct skl_plane_wm *wm =
+                       &cstate->wm.skl.optimal.planes[plane->id];
 
                /* Skip this plane if it's not enabled */
                if (!wm->wm[0].plane_en)
@@ -3156,28 +3134,29 @@ static void skl_ddb_entry_init_from_hw(struct skl_ddb_entry *entry, u32 reg)
 void skl_ddb_get_hw_state(struct drm_i915_private *dev_priv,
                          struct skl_ddb_allocation *ddb /* out */)
 {
-       enum pipe pipe;
-       int plane;
-       u32 val;
+       struct intel_crtc *crtc;
 
        memset(ddb, 0, sizeof(*ddb));
 
-       for_each_pipe(dev_priv, pipe) {
+       for_each_intel_crtc(&dev_priv->drm, crtc) {
                enum intel_display_power_domain power_domain;
+               enum plane_id plane_id;
+               enum pipe pipe = crtc->pipe;
 
                power_domain = POWER_DOMAIN_PIPE(pipe);
                if (!intel_display_power_get_if_enabled(dev_priv, power_domain))
                        continue;
 
-               for_each_universal_plane(dev_priv, pipe, plane) {
-                       val = I915_READ(PLANE_BUF_CFG(pipe, plane));
-                       skl_ddb_entry_init_from_hw(&ddb->plane[pipe][plane],
-                                                  val);
-               }
+               for_each_plane_id_on_crtc(crtc, plane_id) {
+                       u32 val;
+
+                       if (plane_id != PLANE_CURSOR)
+                               val = I915_READ(PLANE_BUF_CFG(pipe, plane_id));
+                       else
+                               val = I915_READ(CUR_BUF_CFG(pipe));
 
-               val = I915_READ(CUR_BUF_CFG(pipe));
-               skl_ddb_entry_init_from_hw(&ddb->plane[pipe][PLANE_CURSOR],
-                                          val);
+                       skl_ddb_entry_init_from_hw(&ddb->plane[pipe][plane_id], val);
+               }
 
                intel_display_power_put(dev_priv, power_domain);
        }
@@ -3278,30 +3257,28 @@ skl_get_total_relative_data_rate(struct intel_crtc_state *intel_cstate,
        struct drm_crtc_state *cstate = &intel_cstate->base;
        struct drm_atomic_state *state = cstate->state;
        struct drm_plane *plane;
-       const struct intel_plane *intel_plane;
        const struct drm_plane_state *pstate;
-       unsigned int rate, total_data_rate = 0;
-       int id;
+       unsigned int total_data_rate = 0;
 
        if (WARN_ON(!state))
                return 0;
 
        /* Calculate and cache data rate for each plane */
        drm_atomic_crtc_state_for_each_plane_state(plane, pstate, cstate) {
-               id = skl_wm_plane_id(to_intel_plane(plane));
-               intel_plane = to_intel_plane(plane);
+               enum plane_id plane_id = to_intel_plane(plane)->id;
+               unsigned int rate;
 
                /* packed/uv */
                rate = skl_plane_relative_data_rate(intel_cstate,
                                                    pstate, 0);
-               plane_data_rate[id] = rate;
+               plane_data_rate[plane_id] = rate;
 
                total_data_rate += rate;
 
                /* y-plane */
                rate = skl_plane_relative_data_rate(intel_cstate,
                                                    pstate, 1);
-               plane_y_data_rate[id] = rate;
+               plane_y_data_rate[plane_id] = rate;
 
                total_data_rate += rate;
        }
@@ -3380,17 +3357,16 @@ skl_ddb_calc_min(const struct intel_crtc_state *cstate, int num_active,
        struct drm_plane *plane;
 
        drm_atomic_crtc_state_for_each_plane_state(plane, pstate, &cstate->base) {
-               struct intel_plane *intel_plane = to_intel_plane(plane);
-               int id = skl_wm_plane_id(intel_plane);
+               enum plane_id plane_id = to_intel_plane(plane)->id;
 
-               if (id == PLANE_CURSOR)
+               if (plane_id == PLANE_CURSOR)
                        continue;
 
                if (!pstate->visible)
                        continue;
 
-               minimum[id] = skl_ddb_min_alloc(pstate, 0);
-               y_minimum[id] = skl_ddb_min_alloc(pstate, 1);
+               minimum[plane_id] = skl_ddb_min_alloc(pstate, 0);
+               y_minimum[plane_id] = skl_ddb_min_alloc(pstate, 1);
        }
 
        minimum[PLANE_CURSOR] = skl_cursor_allocation(num_active);
@@ -3410,8 +3386,8 @@ skl_allocate_pipe_ddb(struct intel_crtc_state *cstate,
        uint16_t minimum[I915_MAX_PLANES] = {};
        uint16_t y_minimum[I915_MAX_PLANES] = {};
        unsigned int total_data_rate;
+       enum plane_id plane_id;
        int num_active;
-       int id, i;
        unsigned plane_data_rate[I915_MAX_PLANES] = {};
        unsigned plane_y_data_rate[I915_MAX_PLANES] = {};
 
@@ -3442,9 +3418,9 @@ skl_allocate_pipe_ddb(struct intel_crtc_state *cstate,
         * proportional to the data rate.
         */
 
-       for (i = 0; i < I915_MAX_PLANES; i++) {
-               alloc_size -= minimum[i];
-               alloc_size -= y_minimum[i];
+       for_each_plane_id_on_crtc(intel_crtc, plane_id) {
+               alloc_size -= minimum[plane_id];
+               alloc_size -= y_minimum[plane_id];
        }
 
        ddb->plane[pipe][PLANE_CURSOR].start = alloc->end - minimum[PLANE_CURSOR];
@@ -3463,28 +3439,28 @@ skl_allocate_pipe_ddb(struct intel_crtc_state *cstate,
                return 0;
 
        start = alloc->start;
-       for (id = 0; id < I915_MAX_PLANES; id++) {
+       for_each_plane_id_on_crtc(intel_crtc, plane_id) {
                unsigned int data_rate, y_data_rate;
                uint16_t plane_blocks, y_plane_blocks = 0;
 
-               if (id == PLANE_CURSOR)
+               if (plane_id == PLANE_CURSOR)
                        continue;
 
-               data_rate = plane_data_rate[id];
+               data_rate = plane_data_rate[plane_id];
 
                /*
                 * allocation for (packed formats) or (uv-plane part of planar format):
                 * promote the expression to 64 bits to avoid overflowing, the
                 * result is < available as data_rate / total_data_rate < 1
                 */
-               plane_blocks = minimum[id];
+               plane_blocks = minimum[plane_id];
                plane_blocks += div_u64((uint64_t)alloc_size * data_rate,
                                        total_data_rate);
 
                /* Leave disabled planes at (0,0) */
                if (data_rate) {
-                       ddb->plane[pipe][id].start = start;
-                       ddb->plane[pipe][id].end = start + plane_blocks;
+                       ddb->plane[pipe][plane_id].start = start;
+                       ddb->plane[pipe][plane_id].end = start + plane_blocks;
                }
 
                start += plane_blocks;
@@ -3492,15 +3468,15 @@ skl_allocate_pipe_ddb(struct intel_crtc_state *cstate,
                /*
                 * allocation for y_plane part of planar format:
                 */
-               y_data_rate = plane_y_data_rate[id];
+               y_data_rate = plane_y_data_rate[plane_id];
 
-               y_plane_blocks = y_minimum[id];
+               y_plane_blocks = y_minimum[plane_id];
                y_plane_blocks += div_u64((uint64_t)alloc_size * y_data_rate,
                                        total_data_rate);
 
                if (y_data_rate) {
-                       ddb->y_plane[pipe][id].start = start;
-                       ddb->y_plane[pipe][id].end = start + y_plane_blocks;
+                       ddb->y_plane[pipe][plane_id].start = start;
+                       ddb->y_plane[pipe][plane_id].end = start + y_plane_blocks;
                }
 
                start += y_plane_blocks;
@@ -3692,12 +3668,12 @@ static int skl_compute_plane_wm(const struct drm_i915_private *dev_priv,
                if (level) {
                        return 0;
                } else {
+                       struct drm_plane *plane = pstate->plane;
+
                        DRM_DEBUG_KMS("Requested display configuration exceeds system watermark limitations\n");
-                       DRM_DEBUG_KMS("Plane %d.%d: blocks required = %u/%u, lines required = %u/31\n",
-                                     to_intel_crtc(cstate->base.crtc)->pipe,
-                                     skl_wm_plane_id(to_intel_plane(pstate->plane)),
+                       DRM_DEBUG_KMS("[PLANE:%d:%s] blocks required = %u/%u, lines required = %u/31\n",
+                                     plane->base.id, plane->name,
                                      res_blocks, ddb_allocation, res_lines);
-
                        return -EINVAL;
                }
        }
@@ -3724,7 +3700,6 @@ skl_compute_wm_level(const struct drm_i915_private *dev_priv,
        uint16_t ddb_blocks;
        enum pipe pipe = intel_crtc->pipe;
        int ret;
-       int i = skl_wm_plane_id(intel_plane);
 
        if (state)
                intel_pstate =
@@ -3747,7 +3722,7 @@ skl_compute_wm_level(const struct drm_i915_private *dev_priv,
 
        WARN_ON(!intel_pstate->base.fb);
 
-       ddb_blocks = skl_ddb_entry_size(&ddb->plane[pipe][i]);
+       ddb_blocks = skl_ddb_entry_size(&ddb->plane[pipe][intel_plane->id]);
 
        ret = skl_compute_plane_wm(dev_priv,
                                   cstate,
@@ -3810,7 +3785,7 @@ static int skl_build_pipe_wm(struct intel_crtc_state *cstate,
        for_each_intel_plane_mask(&dev_priv->drm,
                                  intel_plane,
                                  cstate->base.plane_mask) {
-               wm = &pipe_wm->planes[skl_wm_plane_id(intel_plane)];
+               wm = &pipe_wm->planes[intel_plane->id];
 
                for (level = 0; level <= max_level; level++) {
                        ret = skl_compute_wm_level(dev_priv, ddb, cstate,
@@ -3854,7 +3829,7 @@ static void skl_write_wm_level(struct drm_i915_private *dev_priv,
 static void skl_write_plane_wm(struct intel_crtc *intel_crtc,
                               const struct skl_plane_wm *wm,
                               const struct skl_ddb_allocation *ddb,
-                              int plane)
+                              enum plane_id plane_id)
 {
        struct drm_crtc *crtc = &intel_crtc->base;
        struct drm_device *dev = crtc->dev;
@@ -3863,16 +3838,16 @@ static void skl_write_plane_wm(struct intel_crtc *intel_crtc,
        enum pipe pipe = intel_crtc->pipe;
 
        for (level = 0; level <= max_level; level++) {
-               skl_write_wm_level(dev_priv, PLANE_WM(pipe, plane, level),
+               skl_write_wm_level(dev_priv, PLANE_WM(pipe, plane_id, level),
                                   &wm->wm[level]);
        }
-       skl_write_wm_level(dev_priv, PLANE_WM_TRANS(pipe, plane),
+       skl_write_wm_level(dev_priv, PLANE_WM_TRANS(pipe, plane_id),
                           &wm->trans_wm);
 
-       skl_ddb_entry_write(dev_priv, PLANE_BUF_CFG(pipe, plane),
-                           &ddb->plane[pipe][plane]);
-       skl_ddb_entry_write(dev_priv, PLANE_NV12_BUF_CFG(pipe, plane),
-                           &ddb->y_plane[pipe][plane]);
+       skl_ddb_entry_write(dev_priv, PLANE_BUF_CFG(pipe, plane_id),
+                           &ddb->plane[pipe][plane_id]);
+       skl_ddb_entry_write(dev_priv, PLANE_NV12_BUF_CFG(pipe, plane_id),
+                           &ddb->y_plane[pipe][plane_id]);
 }
 
 static void skl_write_cursor_wm(struct intel_crtc *intel_crtc,
@@ -3977,17 +3952,16 @@ skl_ddb_add_affected_planes(struct intel_crtc_state *cstate)
        struct drm_plane_state *plane_state;
        struct drm_plane *plane;
        enum pipe pipe = intel_crtc->pipe;
-       int id;
 
        WARN_ON(!drm_atomic_get_existing_crtc_state(state, crtc));
 
        drm_for_each_plane_mask(plane, dev, cstate->base.plane_mask) {
-               id = skl_wm_plane_id(to_intel_plane(plane));
+               enum plane_id plane_id = to_intel_plane(plane)->id;
 
-               if (skl_ddb_entry_equal(&cur_ddb->plane[pipe][id],
-                                       &new_ddb->plane[pipe][id]) &&
-                   skl_ddb_entry_equal(&cur_ddb->y_plane[pipe][id],
-                                       &new_ddb->y_plane[pipe][id]))
+               if (skl_ddb_entry_equal(&cur_ddb->plane[pipe][plane_id],
+                                       &new_ddb->plane[pipe][plane_id]) &&
+                   skl_ddb_entry_equal(&cur_ddb->y_plane[pipe][plane_id],
+                                       &new_ddb->y_plane[pipe][plane_id]))
                        continue;
 
                plane_state = drm_atomic_get_plane_state(state, plane);
@@ -4099,7 +4073,6 @@ skl_print_wm_changes(const struct drm_atomic_state *state)
        const struct intel_plane *intel_plane;
        const struct skl_ddb_allocation *old_ddb = &dev_priv->wm.skl_hw.ddb;
        const struct skl_ddb_allocation *new_ddb = &intel_state->wm_results.ddb;
-       int id;
        int i;
 
        for_each_crtc_in_state(state, crtc, cstate, i) {
@@ -4107,11 +4080,11 @@ skl_print_wm_changes(const struct drm_atomic_state *state)
                enum pipe pipe = intel_crtc->pipe;
 
                for_each_intel_plane_on_crtc(dev, intel_crtc, intel_plane) {
+                       enum plane_id plane_id = intel_plane->id;
                        const struct skl_ddb_entry *old, *new;
 
-                       id = skl_wm_plane_id(intel_plane);
-                       old = &old_ddb->plane[pipe][id];
-                       new = &new_ddb->plane[pipe][id];
+                       old = &old_ddb->plane[pipe][plane_id];
+                       new = &new_ddb->plane[pipe][plane_id];
 
                        if (skl_ddb_entry_equal(old, new))
                                continue;
@@ -4201,17 +4174,21 @@ static void skl_atomic_update_crtc_wm(struct intel_atomic_state *state,
        struct skl_pipe_wm *pipe_wm = &cstate->wm.skl.optimal;
        const struct skl_ddb_allocation *ddb = &state->wm_results.ddb;
        enum pipe pipe = crtc->pipe;
-       int plane;
+       enum plane_id plane_id;
 
        if (!(state->wm_results.dirty_pipes & drm_crtc_mask(&crtc->base)))
                return;
 
        I915_WRITE(PIPE_WM_LINETIME(pipe), pipe_wm->linetime);
 
-       for_each_universal_plane(dev_priv, pipe, plane)
-               skl_write_plane_wm(crtc, &pipe_wm->planes[plane], ddb, plane);
-
-       skl_write_cursor_wm(crtc, &pipe_wm->planes[PLANE_CURSOR], ddb);
+       for_each_plane_id_on_crtc(crtc, plane_id) {
+               if (plane_id != PLANE_CURSOR)
+                       skl_write_plane_wm(crtc, &pipe_wm->planes[plane_id],
+                                          ddb, plane_id);
+               else
+                       skl_write_cursor_wm(crtc, &pipe_wm->planes[plane_id],
+                                           ddb);
+       }
 }
 
 static void skl_initial_wm(struct intel_atomic_state *state,
@@ -4326,32 +4303,29 @@ static inline void skl_wm_level_from_reg_val(uint32_t val,
 void skl_pipe_wm_get_hw_state(struct drm_crtc *crtc,
                              struct skl_pipe_wm *out)
 {
-       struct drm_device *dev = crtc->dev;
-       struct drm_i915_private *dev_priv = to_i915(dev);
+       struct drm_i915_private *dev_priv = to_i915(crtc->dev);
        struct intel_crtc *intel_crtc = to_intel_crtc(crtc);
-       struct intel_plane *intel_plane;
-       struct skl_plane_wm *wm;
        enum pipe pipe = intel_crtc->pipe;
-       int level, id, max_level;
+       int level, max_level;
+       enum plane_id plane_id;
        uint32_t val;
 
        max_level = ilk_wm_max_level(dev_priv);
 
-       for_each_intel_plane_on_crtc(dev, intel_crtc, intel_plane) {
-               id = skl_wm_plane_id(intel_plane);
-               wm = &out->planes[id];
+       for_each_plane_id_on_crtc(intel_crtc, plane_id) {
+               struct skl_plane_wm *wm = &out->planes[plane_id];
 
                for (level = 0; level <= max_level; level++) {
-                       if (id != PLANE_CURSOR)
-                               val = I915_READ(PLANE_WM(pipe, id, level));
+                       if (plane_id != PLANE_CURSOR)
+                               val = I915_READ(PLANE_WM(pipe, plane_id, level));
                        else
                                val = I915_READ(CUR_WM(pipe, level));
 
                        skl_wm_level_from_reg_val(val, &wm->wm[level]);
                }
 
-               if (id != PLANE_CURSOR)
-                       val = I915_READ(PLANE_WM_TRANS(pipe, id));
+               if (plane_id != PLANE_CURSOR)
+                       val = I915_READ(PLANE_WM_TRANS(pipe, plane_id));
                else
                        val = I915_READ(CUR_WM_TRANS(pipe));