Commit b6e201f5 authored by Ville Syrjälä's avatar Ville Syrjälä

drm/i915/fbc: Pass whole plane state to intel_fbc_min_limit()

No reason to burden the caller with the details on how the minimum
compression limit is calculated, so just pass in the whole plane
state instead of just the cpp value.
Signed-off-by: default avatarVille Syrjälä <ville.syrjala@linux.intel.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20211124113652.22090-3-ville.syrjala@linux.intel.comReviewed-by: default avatarMika Kahola <mika.kahola@intel.com>
parent 248e2515
...@@ -679,8 +679,10 @@ static u64 intel_fbc_stolen_end(struct drm_i915_private *i915) ...@@ -679,8 +679,10 @@ static u64 intel_fbc_stolen_end(struct drm_i915_private *i915)
return min(end, intel_fbc_cfb_base_max(i915)); return min(end, intel_fbc_cfb_base_max(i915));
} }
static int intel_fbc_min_limit(int fb_cpp) static int intel_fbc_min_limit(const struct intel_plane_state *plane_state)
{ {
int fb_cpp = plane_state->hw.fb ? plane_state->hw.fb->format->cpp[0] : 0;
return fb_cpp == 2 ? 2 : 1; return fb_cpp == 2 ? 2 : 1;
} }
...@@ -1466,8 +1468,7 @@ static void intel_fbc_enable(struct intel_atomic_state *state, ...@@ -1466,8 +1468,7 @@ static void intel_fbc_enable(struct intel_atomic_state *state,
cache = &fbc->state_cache; cache = &fbc->state_cache;
min_limit = intel_fbc_min_limit(plane_state->hw.fb ? min_limit = intel_fbc_min_limit(plane_state);
plane_state->hw.fb->format->cpp[0] : 0);
mutex_lock(&fbc->lock); mutex_lock(&fbc->lock);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment