mirror of
https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git
synced 2024-08-23 09:19:51 +00:00
drm/i915: Nuke force_min_cdclk_changed
Since we now have proper old and new cdclk state we no longer need to keep this flag to indicate that the force min cdclk has changed. Instead just check if the old vs. new value are different. Signed-off-by: Ville Syrjälä <ville.syrjala@linux.intel.com> Link: https://patchwork.freedesktop.org/patch/msgid/20200714152626.380-4-ville.syrjala@linux.intel.com Reviewed-by: Stanislav Lisovskiy <stanislav.lisovskiy@intel.com>
This commit is contained in:
parent
e161da3316
commit
dc98f50fa5
4 changed files with 6 additions and 9 deletions
|
@ -958,13 +958,8 @@ static int glk_force_audio_cdclk_commit(struct intel_atomic_state *state,
|
|||
if (IS_ERR(cdclk_state))
|
||||
return PTR_ERR(cdclk_state);
|
||||
|
||||
cdclk_state->force_min_cdclk_changed = true;
|
||||
cdclk_state->force_min_cdclk = enable ? 2 * 96000 : 0;
|
||||
|
||||
ret = intel_atomic_lock_global_state(&cdclk_state->base);
|
||||
if (ret)
|
||||
return ret;
|
||||
|
||||
return drm_atomic_commit(&state->base);
|
||||
}
|
||||
|
||||
|
|
|
@ -2426,7 +2426,6 @@ static struct intel_global_state *intel_cdclk_duplicate_state(struct intel_globa
|
|||
if (!cdclk_state)
|
||||
return NULL;
|
||||
|
||||
cdclk_state->force_min_cdclk_changed = false;
|
||||
cdclk_state->pipe = INVALID_PIPE;
|
||||
|
||||
return &cdclk_state->base;
|
||||
|
@ -2501,6 +2500,7 @@ int intel_modeset_calc_cdclk(struct intel_atomic_state *state)
|
|||
if (ret)
|
||||
return ret;
|
||||
} else if (old_cdclk_state->active_pipes != new_cdclk_state->active_pipes ||
|
||||
old_cdclk_state->force_min_cdclk != new_cdclk_state->force_min_cdclk ||
|
||||
intel_cdclk_changed(&old_cdclk_state->logical,
|
||||
&new_cdclk_state->logical)) {
|
||||
ret = intel_atomic_lock_global_state(&new_cdclk_state->base);
|
||||
|
|
|
@ -49,7 +49,6 @@ struct intel_cdclk_state {
|
|||
|
||||
/* forced minimum cdclk for glk+ audio w/a */
|
||||
int force_min_cdclk;
|
||||
bool force_min_cdclk_changed;
|
||||
|
||||
/* bitmask of active pipes */
|
||||
u8 active_pipes;
|
||||
|
|
|
@ -14794,7 +14794,8 @@ static int intel_atomic_check_cdclk(struct intel_atomic_state *state,
|
|||
bool *need_cdclk_calc)
|
||||
{
|
||||
struct drm_i915_private *dev_priv = to_i915(state->base.dev);
|
||||
struct intel_cdclk_state *new_cdclk_state;
|
||||
const struct intel_cdclk_state *old_cdclk_state;
|
||||
const struct intel_cdclk_state *new_cdclk_state;
|
||||
struct intel_plane_state *plane_state;
|
||||
struct intel_bw_state *new_bw_state;
|
||||
struct intel_plane *plane;
|
||||
|
@ -14813,9 +14814,11 @@ static int intel_atomic_check_cdclk(struct intel_atomic_state *state,
|
|||
return ret;
|
||||
}
|
||||
|
||||
old_cdclk_state = intel_atomic_get_old_cdclk_state(state);
|
||||
new_cdclk_state = intel_atomic_get_new_cdclk_state(state);
|
||||
|
||||
if (new_cdclk_state && new_cdclk_state->force_min_cdclk_changed)
|
||||
if (new_cdclk_state &&
|
||||
old_cdclk_state->force_min_cdclk != new_cdclk_state->force_min_cdclk)
|
||||
*need_cdclk_calc = true;
|
||||
|
||||
ret = dev_priv->display.bw_calc_min_cdclk(state);
|
||||
|
|
Loading…
Reference in a new issue