Re: [PATCH 1/2] drm/i915: Use intel_ types more consistently for watermark code

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Thu, Dec 06, 2018 at 04:54:00PM -0800, Matt Roper wrote:
> Try to be more consistent about intel_* types rather than drm_* types
> for lower-level driver functions.
> 
> Signed-off-by: Matt Roper <matthew.d.roper@xxxxxxxxx>
> ---
>  drivers/gpu/drm/i915/i915_drv.h      |   5 +-
>  drivers/gpu/drm/i915/intel_display.c |  32 ++---
>  drivers/gpu/drm/i915/intel_drv.h     |  10 +-
>  drivers/gpu/drm/i915/intel_pm.c      | 250 ++++++++++++++++-------------------
>  4 files changed, 134 insertions(+), 163 deletions(-)
> 
> diff --git a/drivers/gpu/drm/i915/i915_drv.h b/drivers/gpu/drm/i915/i915_drv.h
> index 0689e67c966e..7469a7785253 100644
> --- a/drivers/gpu/drm/i915/i915_drv.h
> +++ b/drivers/gpu/drm/i915/i915_drv.h
> @@ -281,8 +281,7 @@ struct drm_i915_display_funcs {
>  	int (*get_fifo_size)(struct drm_i915_private *dev_priv,
>  			     enum i9xx_plane_id i9xx_plane);
>  	int (*compute_pipe_wm)(struct intel_crtc_state *cstate);
> -	int (*compute_intermediate_wm)(struct drm_device *dev,
> -				       struct intel_crtc *intel_crtc,
> +	int (*compute_intermediate_wm)(struct intel_crtc *intel_crtc,

We could omit this argument too since it can be derived from the crtc
state.

Anyways, patch is
Reviewed-by: Ville Syrjälä <ville.syrjala@xxxxxxxxxxxxxxx>

>  				       struct intel_crtc_state *newstate);
>  	void (*initial_watermarks)(struct intel_atomic_state *state,
>  				   struct intel_crtc_state *cstate);
> @@ -290,7 +289,7 @@ struct drm_i915_display_funcs {
>  					 struct intel_crtc_state *cstate);
>  	void (*optimize_watermarks)(struct intel_atomic_state *state,
>  				    struct intel_crtc_state *cstate);
> -	int (*compute_global_watermarks)(struct drm_atomic_state *state);
> +	int (*compute_global_watermarks)(struct intel_atomic_state *state);
>  	void (*update_wm)(struct intel_crtc *crtc);
>  	int (*modeset_calc_cdclk)(struct drm_atomic_state *state);
>  	/* Returns the active state of the crtc, and if the crtc is active,
> diff --git a/drivers/gpu/drm/i915/intel_display.c b/drivers/gpu/drm/i915/intel_display.c
> index 07c861884c70..db6004a883c7 100644
> --- a/drivers/gpu/drm/i915/intel_display.c
> +++ b/drivers/gpu/drm/i915/intel_display.c
> @@ -10661,12 +10661,9 @@ static void intel_crtc_destroy(struct drm_crtc *crtc)
>   *
>   * Returns true or false.
>   */
> -static bool intel_wm_need_update(struct drm_plane *plane,
> -				 struct drm_plane_state *state)
> +static bool intel_wm_need_update(struct intel_plane_state *cur,
> +				 struct intel_plane_state *new)
>  {
> -	struct intel_plane_state *new = to_intel_plane_state(state);
> -	struct intel_plane_state *cur = to_intel_plane_state(plane->state);
> -
>  	/* Update watermarks on tiling or size changes. */
>  	if (new->base.visible != cur->base.visible)
>  		return true;
> @@ -10775,7 +10772,8 @@ int intel_plane_atomic_calc_changes(const struct intel_crtc_state *old_crtc_stat
>  		/* must disable cxsr around plane enable/disable */
>  		if (plane->id != PLANE_CURSOR)
>  			pipe_config->disable_cxsr = true;
> -	} else if (intel_wm_need_update(&plane->base, plane_state)) {
> +	} else if (intel_wm_need_update(to_intel_plane_state(plane->base.state),
> +					to_intel_plane_state(plane_state))) {
>  		if (INTEL_GEN(dev_priv) < 5 && !IS_G4X(dev_priv)) {
>  			/* FIXME bollocks */
>  			pipe_config->update_wm_pre = true;
> @@ -10954,8 +10952,7 @@ static int icl_check_nv12_planes(struct intel_crtc_state *crtc_state)
>  static int intel_crtc_atomic_check(struct drm_crtc *crtc,
>  				   struct drm_crtc_state *crtc_state)
>  {
> -	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_crtc_state *pipe_config =
>  		to_intel_crtc_state(crtc_state);
> @@ -11004,8 +11001,7 @@ static int intel_crtc_atomic_check(struct drm_crtc *crtc,
>  		 * old state and the new state.  We can program these
>  		 * immediately.
>  		 */
> -		ret = dev_priv->display.compute_intermediate_wm(dev,
> -								intel_crtc,
> +		ret = dev_priv->display.compute_intermediate_wm(intel_crtc,
>  								pipe_config);
>  		if (ret) {
>  			DRM_DEBUG_KMS("No valid intermediate pipe watermarks are possible\n");
> @@ -11964,7 +11960,7 @@ static void verify_wm_state(struct drm_crtc *crtc,
>  	if (INTEL_GEN(dev_priv) < 9 || !new_state->active)
>  		return;
>  
> -	skl_pipe_wm_get_hw_state(crtc, &hw_wm);
> +	skl_pipe_wm_get_hw_state(intel_crtc, &hw_wm);
>  	sw_wm = &to_intel_crtc_state(new_state)->wm.skl.optimal;
>  
>  	skl_pipe_ddb_get_hw_state(intel_crtc, hw_ddb_y, hw_ddb_uv);
> @@ -12619,9 +12615,9 @@ static int intel_modeset_checks(struct drm_atomic_state *state)
>   * phase.  The code here should be run after the per-crtc and per-plane 'check'
>   * handlers to ensure that all derived state has been updated.
>   */
> -static int calc_watermark_data(struct drm_atomic_state *state)
> +static int calc_watermark_data(struct intel_atomic_state *state)
>  {
> -	struct drm_device *dev = state->dev;
> +	struct drm_device *dev = state->base.dev;
>  	struct drm_i915_private *dev_priv = to_i915(dev);
>  
>  	/* Is there platform-specific watermark information to calculate? */
> @@ -12713,7 +12709,7 @@ static int intel_atomic_check(struct drm_device *dev,
>  		return ret;
>  
>  	intel_fbc_choose_crtc(dev_priv, intel_state);
> -	return calc_watermark_data(state);
> +	return calc_watermark_data(intel_state);
>  }
>  
>  static int intel_atomic_prepare_commit(struct drm_device *dev,
> @@ -15843,15 +15839,15 @@ intel_modeset_setup_hw_state(struct drm_device *dev,
>  	}
>  
>  	if (IS_G4X(dev_priv)) {
> -		g4x_wm_get_hw_state(dev);
> +		g4x_wm_get_hw_state(dev_priv);
>  		g4x_wm_sanitize(dev_priv);
>  	} else if (IS_VALLEYVIEW(dev_priv) || IS_CHERRYVIEW(dev_priv)) {
> -		vlv_wm_get_hw_state(dev);
> +		vlv_wm_get_hw_state(dev_priv);
>  		vlv_wm_sanitize(dev_priv);
>  	} else if (INTEL_GEN(dev_priv) >= 9) {
> -		skl_wm_get_hw_state(dev);
> +		skl_wm_get_hw_state(dev_priv);
>  	} else if (HAS_PCH_SPLIT(dev_priv)) {
> -		ilk_wm_get_hw_state(dev);
> +		ilk_wm_get_hw_state(dev_priv);
>  	}
>  
>  	for_each_intel_crtc(dev, crtc) {
> diff --git a/drivers/gpu/drm/i915/intel_drv.h b/drivers/gpu/drm/i915/intel_drv.h
> index f94a04b4ad87..a8177ccf1d4f 100644
> --- a/drivers/gpu/drm/i915/intel_drv.h
> +++ b/drivers/gpu/drm/i915/intel_drv.h
> @@ -2200,16 +2200,16 @@ void gen6_rps_busy(struct drm_i915_private *dev_priv);
>  void gen6_rps_reset_ei(struct drm_i915_private *dev_priv);
>  void gen6_rps_idle(struct drm_i915_private *dev_priv);
>  void gen6_rps_boost(struct i915_request *rq, struct intel_rps_client *rps);
> -void g4x_wm_get_hw_state(struct drm_device *dev);
> -void vlv_wm_get_hw_state(struct drm_device *dev);
> -void ilk_wm_get_hw_state(struct drm_device *dev);
> -void skl_wm_get_hw_state(struct drm_device *dev);
> +void g4x_wm_get_hw_state(struct drm_i915_private *dev_priv);
> +void vlv_wm_get_hw_state(struct drm_i915_private *dev_priv);
> +void ilk_wm_get_hw_state(struct drm_i915_private *dev_priv);
> +void skl_wm_get_hw_state(struct drm_i915_private *dev_priv);
>  void skl_pipe_ddb_get_hw_state(struct intel_crtc *crtc,
>  			       struct skl_ddb_entry *ddb_y,
>  			       struct skl_ddb_entry *ddb_uv);
>  void skl_ddb_get_hw_state(struct drm_i915_private *dev_priv,
>  			  struct skl_ddb_allocation *ddb /* out */);
> -void skl_pipe_wm_get_hw_state(struct drm_crtc *crtc,
> +void skl_pipe_wm_get_hw_state(struct intel_crtc *crtc,
>  			      struct skl_pipe_wm *out);
>  void g4x_wm_sanitize(struct drm_i915_private *dev_priv);
>  void vlv_wm_sanitize(struct drm_i915_private *dev_priv);
> diff --git a/drivers/gpu/drm/i915/intel_pm.c b/drivers/gpu/drm/i915/intel_pm.c
> index a26b4eddda25..efb8bcf77fb8 100644
> --- a/drivers/gpu/drm/i915/intel_pm.c
> +++ b/drivers/gpu/drm/i915/intel_pm.c
> @@ -1399,8 +1399,7 @@ static int g4x_compute_pipe_wm(struct intel_crtc_state *crtc_state)
>  	return 0;
>  }
>  
> -static int g4x_compute_intermediate_wm(struct drm_device *dev,
> -				       struct intel_crtc *crtc,
> +static int g4x_compute_intermediate_wm(struct intel_crtc *crtc,
>  				       struct intel_crtc_state *new_crtc_state)
>  {
>  	struct g4x_wm_state *intermediate = &new_crtc_state->wm.g4x.intermediate;
> @@ -2032,8 +2031,7 @@ static void vlv_atomic_update_fifo(struct intel_atomic_state *state,
>  
>  #undef VLV_FIFO
>  
> -static int vlv_compute_intermediate_wm(struct drm_device *dev,
> -				       struct intel_crtc *crtc,
> +static int vlv_compute_intermediate_wm(struct intel_crtc *crtc,
>  				       struct intel_crtc_state *new_crtc_state)
>  {
>  	struct vlv_wm_state *intermediate = &new_crtc_state->wm.vlv.intermediate;
> @@ -2626,13 +2624,12 @@ static unsigned int ilk_fbc_wm_reg_max(const struct drm_i915_private *dev_priv)
>  }
>  
>  /* Calculate the maximum primary/sprite plane watermark */
> -static unsigned int ilk_plane_wm_max(const struct drm_device *dev,
> +static unsigned int ilk_plane_wm_max(const struct drm_i915_private *dev_priv,
>  				     int level,
>  				     const struct intel_wm_config *config,
>  				     enum intel_ddb_partitioning ddb_partitioning,
>  				     bool is_sprite)
>  {
> -	struct drm_i915_private *dev_priv = to_i915(dev);
>  	unsigned int fifo_size = ilk_display_fifo_size(dev_priv);
>  
>  	/* if sprites aren't enabled, sprites get nothing */
> @@ -2668,7 +2665,7 @@ static unsigned int ilk_plane_wm_max(const struct drm_device *dev,
>  }
>  
>  /* Calculate the maximum cursor plane watermark */
> -static unsigned int ilk_cursor_wm_max(const struct drm_device *dev,
> +static unsigned int ilk_cursor_wm_max(const struct drm_i915_private *dev_priv,
>  				      int level,
>  				      const struct intel_wm_config *config)
>  {
> @@ -2677,19 +2674,19 @@ static unsigned int ilk_cursor_wm_max(const struct drm_device *dev,
>  		return 64;
>  
>  	/* otherwise just report max that registers can hold */
> -	return ilk_cursor_wm_reg_max(to_i915(dev), level);
> +	return ilk_cursor_wm_reg_max(dev_priv, level);
>  }
>  
> -static void ilk_compute_wm_maximums(const struct drm_device *dev,
> +static void ilk_compute_wm_maximums(const struct drm_i915_private *dev_priv,
>  				    int level,
>  				    const struct intel_wm_config *config,
>  				    enum intel_ddb_partitioning ddb_partitioning,
>  				    struct ilk_wm_maximums *max)
>  {
> -	max->pri = ilk_plane_wm_max(dev, level, config, ddb_partitioning, false);
> -	max->spr = ilk_plane_wm_max(dev, level, config, ddb_partitioning, true);
> -	max->cur = ilk_cursor_wm_max(dev, level, config);
> -	max->fbc = ilk_fbc_wm_reg_max(to_i915(dev));
> +	max->pri = ilk_plane_wm_max(dev_priv, level, config, ddb_partitioning, false);
> +	max->spr = ilk_plane_wm_max(dev_priv, level, config, ddb_partitioning, true);
> +	max->cur = ilk_cursor_wm_max(dev_priv, level, config);
> +	max->fbc = ilk_fbc_wm_reg_max(dev_priv);
>  }
>  
>  static void ilk_compute_wm_reg_maximums(const struct drm_i915_private *dev_priv,
> @@ -3073,7 +3070,7 @@ static void skl_setup_wm_latency(struct drm_i915_private *dev_priv)
>  	intel_print_wm_latency(dev_priv, "Gen9 Plane", dev_priv->wm.skl_latency);
>  }
>  
> -static bool ilk_validate_pipe_wm(struct drm_device *dev,
> +static bool ilk_validate_pipe_wm(const struct drm_i915_private *dev_priv,
>  				 struct intel_pipe_wm *pipe_wm)
>  {
>  	/* LP0 watermark maximums depend on this pipe alone */
> @@ -3085,7 +3082,7 @@ static bool ilk_validate_pipe_wm(struct drm_device *dev,
>  	struct ilk_wm_maximums max;
>  
>  	/* LP0 watermarks always use 1/2 DDB partitioning */
> -	ilk_compute_wm_maximums(dev, 0, &config, INTEL_DDB_PART_1_2, &max);
> +	ilk_compute_wm_maximums(dev_priv, 0, &config, INTEL_DDB_PART_1_2, &max);
>  
>  	/* At least LP0 must be valid */
>  	if (!ilk_validate_wm_level(0, &max, &pipe_wm->wm[0])) {
> @@ -3150,7 +3147,7 @@ static int ilk_compute_pipe_wm(struct intel_crtc_state *cstate)
>  	if (IS_HASWELL(dev_priv) || IS_BROADWELL(dev_priv))
>  		pipe_wm->linetime = hsw_compute_linetime_wm(cstate);
>  
> -	if (!ilk_validate_pipe_wm(dev, pipe_wm))
> +	if (!ilk_validate_pipe_wm(dev_priv, pipe_wm))
>  		return -EINVAL;
>  
>  	ilk_compute_wm_reg_maximums(dev_priv, 1, &max);
> @@ -3180,17 +3177,17 @@ static int ilk_compute_pipe_wm(struct intel_crtc_state *cstate)
>   * state and the new state.  These can be programmed to the hardware
>   * immediately.
>   */
> -static int ilk_compute_intermediate_wm(struct drm_device *dev,
> -				       struct intel_crtc *intel_crtc,
> +static int ilk_compute_intermediate_wm(struct intel_crtc *intel_crtc,
>  				       struct intel_crtc_state *newstate)
>  {
> +	struct drm_i915_private *dev_priv = to_i915(intel_crtc->base.dev);
>  	struct intel_pipe_wm *a = &newstate->wm.ilk.intermediate;
>  	struct intel_atomic_state *intel_state =
>  		to_intel_atomic_state(newstate->base.state);
>  	const struct intel_crtc_state *oldstate =
>  		intel_atomic_get_old_crtc_state(intel_state, intel_crtc);
>  	const struct intel_pipe_wm *b = &oldstate->wm.ilk.optimal;
> -	int level, max_level = ilk_wm_max_level(to_i915(dev));
> +	int level, max_level = ilk_wm_max_level(dev_priv);
>  
>  	/*
>  	 * Start with the final, target watermarks, then combine with the
> @@ -3223,7 +3220,7 @@ static int ilk_compute_intermediate_wm(struct drm_device *dev,
>  	 * there's no safe way to transition from the old state to
>  	 * the new state, so we need to fail the atomic transaction.
>  	 */
> -	if (!ilk_validate_pipe_wm(dev, a))
> +	if (!ilk_validate_pipe_wm(dev_priv, a))
>  		return -EINVAL;
>  
>  	/*
> @@ -3239,7 +3236,7 @@ static int ilk_compute_intermediate_wm(struct drm_device *dev,
>  /*
>   * Merge the watermarks from all active pipes for a specific level.
>   */
> -static void ilk_merge_wm_level(struct drm_device *dev,
> +static void ilk_merge_wm_level(struct drm_i915_private *dev_priv,
>  			       int level,
>  			       struct intel_wm_level *ret_wm)
>  {
> @@ -3247,7 +3244,7 @@ static void ilk_merge_wm_level(struct drm_device *dev,
>  
>  	ret_wm->enable = true;
>  
> -	for_each_intel_crtc(dev, intel_crtc) {
> +	for_each_intel_crtc(&dev_priv->drm, intel_crtc) {
>  		const struct intel_pipe_wm *active = &intel_crtc->wm.active.ilk;
>  		const struct intel_wm_level *wm = &active->wm[level];
>  
> @@ -3272,12 +3269,11 @@ static void ilk_merge_wm_level(struct drm_device *dev,
>  /*
>   * Merge all low power watermarks for all active pipes.
>   */
> -static void ilk_wm_merge(struct drm_device *dev,
> +static void ilk_wm_merge(struct drm_i915_private *dev_priv,
>  			 const struct intel_wm_config *config,
>  			 const struct ilk_wm_maximums *max,
>  			 struct intel_pipe_wm *merged)
>  {
> -	struct drm_i915_private *dev_priv = to_i915(dev);
>  	int level, max_level = ilk_wm_max_level(dev_priv);
>  	int last_enabled_level = max_level;
>  
> @@ -3293,7 +3289,7 @@ static void ilk_wm_merge(struct drm_device *dev,
>  	for (level = 1; level <= max_level; level++) {
>  		struct intel_wm_level *wm = &merged->wm[level];
>  
> -		ilk_merge_wm_level(dev, level, wm);
> +		ilk_merge_wm_level(dev_priv, level, wm);
>  
>  		if (level > last_enabled_level)
>  			wm->enable = false;
> @@ -3335,22 +3331,20 @@ static int ilk_wm_lp_to_level(int wm_lp, const struct intel_pipe_wm *pipe_wm)
>  }
>  
>  /* The value we need to program into the WM_LPx latency field */
> -static unsigned int ilk_wm_lp_latency(struct drm_device *dev, int level)
> +static unsigned int ilk_wm_lp_latency(struct drm_i915_private *dev_priv,
> +				      int level)
>  {
> -	struct drm_i915_private *dev_priv = to_i915(dev);
> -
>  	if (IS_HASWELL(dev_priv) || IS_BROADWELL(dev_priv))
>  		return 2 * level;
>  	else
>  		return dev_priv->wm.pri_latency[level];
>  }
>  
> -static void ilk_compute_wm_results(struct drm_device *dev,
> +static void ilk_compute_wm_results(struct drm_i915_private *dev_priv,
>  				   const struct intel_pipe_wm *merged,
>  				   enum intel_ddb_partitioning partitioning,
>  				   struct ilk_wm_values *results)
>  {
> -	struct drm_i915_private *dev_priv = to_i915(dev);
>  	struct intel_crtc *intel_crtc;
>  	int level, wm_lp;
>  
> @@ -3370,7 +3364,7 @@ static void ilk_compute_wm_results(struct drm_device *dev,
>  		 * disabled. Doing otherwise could cause underruns.
>  		 */
>  		results->wm_lp[wm_lp - 1] =
> -			(ilk_wm_lp_latency(dev, level) << WM1_LP_LATENCY_SHIFT) |
> +			(ilk_wm_lp_latency(dev_priv, level) << WM1_LP_LATENCY_SHIFT) |
>  			(r->pri_val << WM1_LP_SR_SHIFT) |
>  			r->cur_val;
>  
> @@ -3396,7 +3390,7 @@ static void ilk_compute_wm_results(struct drm_device *dev,
>  	}
>  
>  	/* LP0 register values */
> -	for_each_intel_crtc(dev, intel_crtc) {
> +	for_each_intel_crtc(&dev_priv->drm, intel_crtc) {
>  		enum pipe pipe = intel_crtc->pipe;
>  		const struct intel_wm_level *r =
>  			&intel_crtc->wm.active.ilk.wm[0];
> @@ -3415,11 +3409,12 @@ static void ilk_compute_wm_results(struct drm_device *dev,
>  
>  /* Find the result with the highest level enabled. Check for enable_fbc_wm in
>   * case both are at the same level. Prefer r1 in case they're the same. */
> -static struct intel_pipe_wm *ilk_find_best_result(struct drm_device *dev,
> -						  struct intel_pipe_wm *r1,
> -						  struct intel_pipe_wm *r2)
> +static struct intel_pipe_wm *
> +ilk_find_best_result(struct drm_i915_private *dev_priv,
> +		     struct intel_pipe_wm *r1,
> +		     struct intel_pipe_wm *r2)
>  {
> -	int level, max_level = ilk_wm_max_level(to_i915(dev));
> +	int level, max_level = ilk_wm_max_level(dev_priv);
>  	int level1 = 0, level2 = 0;
>  
>  	for (level = 1; level <= max_level; level++) {
> @@ -4307,10 +4302,9 @@ icl_get_total_relative_data_rate(struct intel_crtc_state *intel_cstate,
>  }
>  
>  static uint16_t
> -skl_ddb_min_alloc(const struct drm_plane_state *pstate, const int plane)
> +skl_ddb_min_alloc(const struct intel_plane_state *plane_state, const int plane)
>  {
> -	struct drm_framebuffer *fb = pstate->fb;
> -	struct intel_plane_state *intel_pstate = to_intel_plane_state(pstate);
> +	struct drm_framebuffer *fb = plane_state->base.fb;
>  	uint32_t src_w, src_h;
>  	uint32_t min_scanlines = 8;
>  	uint8_t plane_bpp;
> @@ -4334,8 +4328,8 @@ skl_ddb_min_alloc(const struct drm_plane_state *pstate, const int plane)
>  	 * the 90/270 degree plane rotation cases (to match the
>  	 * GTT mapping), hence no need to account for rotation here.
>  	 */
> -	src_w = drm_rect_width(&intel_pstate->base.src) >> 16;
> -	src_h = drm_rect_height(&intel_pstate->base.src) >> 16;
> +	src_w = drm_rect_width(&plane_state->base.src) >> 16;
> +	src_h = drm_rect_height(&plane_state->base.src) >> 16;
>  
>  	/* Halve UV plane width and height for NV12 */
>  	if (plane == 1) {
> @@ -4345,7 +4339,7 @@ skl_ddb_min_alloc(const struct drm_plane_state *pstate, const int plane)
>  
>  	plane_bpp = fb->format->cpp[plane];
>  
> -	if (drm_rotation_90_or_270(pstate->rotation)) {
> +	if (drm_rotation_90_or_270(plane_state->base.rotation)) {
>  		switch (plane_bpp) {
>  		case 1:
>  			min_scanlines = 32;
> @@ -4388,14 +4382,15 @@ skl_ddb_calc_min(const struct intel_crtc_state *cstate, int num_active,
>  			continue;
>  
>  		if (!plane_state->linked_plane) {
> -			minimum[plane_id] = skl_ddb_min_alloc(pstate, 0);
> -			uv_minimum[plane_id] = skl_ddb_min_alloc(pstate, 1);
> +			minimum[plane_id] = skl_ddb_min_alloc(plane_state, 0);
> +			uv_minimum[plane_id] =
> +				skl_ddb_min_alloc(plane_state, 1);
>  		} else {
>  			enum plane_id y_plane_id =
>  				plane_state->linked_plane->id;
>  
> -			minimum[y_plane_id] = skl_ddb_min_alloc(pstate, 0);
> -			minimum[plane_id] = skl_ddb_min_alloc(pstate, 1);
> +			minimum[y_plane_id] = skl_ddb_min_alloc(plane_state, 0);
> +			minimum[plane_id] = skl_ddb_min_alloc(plane_state, 1);
>  		}
>  	}
>  
> @@ -5251,15 +5246,14 @@ bool skl_ddb_allocation_overlaps(const struct skl_ddb_entry *ddb,
>  	return false;
>  }
>  
> -static int skl_update_pipe_wm(struct drm_crtc_state *cstate,
> +static int skl_update_pipe_wm(struct intel_crtc_state *cstate,
>  			      const struct skl_pipe_wm *old_pipe_wm,
>  			      struct skl_pipe_wm *pipe_wm, /* out */
>  			      bool *changed /* out */)
>  {
> -	struct intel_crtc_state *intel_cstate = to_intel_crtc_state(cstate);
>  	int ret;
>  
> -	ret = skl_build_pipe_wm(intel_cstate, pipe_wm);
> +	ret = skl_build_pipe_wm(cstate, pipe_wm);
>  	if (ret)
>  		return ret;
>  
> @@ -5272,14 +5266,14 @@ static int skl_update_pipe_wm(struct drm_crtc_state *cstate,
>  }
>  
>  static uint32_t
> -pipes_modified(struct drm_atomic_state *state)
> +pipes_modified(struct intel_atomic_state *state)
>  {
> -	struct drm_crtc *crtc;
> -	struct drm_crtc_state *cstate;
> +	struct intel_crtc *crtc;
> +	struct intel_crtc_state *cstate;
>  	uint32_t i, ret = 0;
>  
> -	for_each_new_crtc_in_state(state, crtc, cstate, i)
> -		ret |= drm_crtc_mask(crtc);
> +	for_each_new_intel_crtc_in_state(state, crtc, cstate, i)
> +		ret |= drm_crtc_mask(&crtc->base);
>  
>  	return ret;
>  }
> @@ -5314,11 +5308,10 @@ skl_ddb_add_affected_planes(const struct intel_crtc_state *old_crtc_state,
>  }
>  
>  static int
> -skl_compute_ddb(struct drm_atomic_state *state)
> +skl_compute_ddb(struct intel_atomic_state *state)
>  {
> -	const struct drm_i915_private *dev_priv = to_i915(state->dev);
> -	struct intel_atomic_state *intel_state = to_intel_atomic_state(state);
> -	struct skl_ddb_allocation *ddb = &intel_state->wm_results.ddb;
> +	const struct drm_i915_private *dev_priv = to_i915(state->base.dev);
> +	struct skl_ddb_allocation *ddb = &state->wm_results.ddb;
>  	struct intel_crtc_state *old_crtc_state;
>  	struct intel_crtc_state *new_crtc_state;
>  	struct intel_crtc *crtc;
> @@ -5326,7 +5319,7 @@ skl_compute_ddb(struct drm_atomic_state *state)
>  
>  	memcpy(ddb, &dev_priv->wm.skl_hw.ddb, sizeof(*ddb));
>  
> -	for_each_oldnew_intel_crtc_in_state(intel_state, crtc, old_crtc_state,
> +	for_each_oldnew_intel_crtc_in_state(state, crtc, old_crtc_state,
>  					    new_crtc_state, i) {
>  		ret = skl_allocate_pipe_ddb(new_crtc_state, ddb);
>  		if (ret)
> @@ -5372,14 +5365,12 @@ skl_print_wm_changes(struct intel_atomic_state *state)
>  }
>  
>  static int
> -skl_ddb_add_affected_pipes(struct drm_atomic_state *state, bool *changed)
> +skl_ddb_add_affected_pipes(struct intel_atomic_state *state, bool *changed)
>  {
> -	struct drm_device *dev = state->dev;
> +	struct drm_device *dev = state->base.dev;
>  	const struct drm_i915_private *dev_priv = to_i915(dev);
> -	const struct drm_crtc *crtc;
> -	const struct drm_crtc_state *cstate;
> -	struct intel_crtc *intel_crtc;
> -	struct intel_atomic_state *intel_state = to_intel_atomic_state(state);
> +	struct intel_crtc *crtc;
> +	struct intel_crtc_state *crtc_state;
>  	uint32_t realloc_pipes = pipes_modified(state);
>  	int ret, i;
>  
> @@ -5398,7 +5389,7 @@ skl_ddb_add_affected_pipes(struct drm_atomic_state *state, bool *changed)
>  	 * since any racing commits that want to update them would need to
>  	 * hold _all_ CRTC state mutexes.
>  	 */
> -	for_each_new_crtc_in_state(state, crtc, cstate, i)
> +	for_each_new_intel_crtc_in_state(state, crtc, crtc_state, i)
>  		(*changed) = true;
>  
>  	if (!*changed)
> @@ -5412,20 +5403,20 @@ skl_ddb_add_affected_pipes(struct drm_atomic_state *state, bool *changed)
>  	 */
>  	if (dev_priv->wm.distrust_bios_wm) {
>  		ret = drm_modeset_lock(&dev->mode_config.connection_mutex,
> -				       state->acquire_ctx);
> +				       state->base.acquire_ctx);
>  		if (ret)
>  			return ret;
>  
> -		intel_state->active_pipe_changes = ~0;
> +		state->active_pipe_changes = ~0;
>  
>  		/*
> -		 * We usually only initialize intel_state->active_crtcs if we
> +		 * We usually only initialize state->active_crtcs if we
>  		 * we're doing a modeset; make sure this field is always
>  		 * initialized during the sanitization process that happens
>  		 * on the first commit too.
>  		 */
> -		if (!intel_state->modeset)
> -			intel_state->active_crtcs = dev_priv->active_crtcs;
> +		if (!state->modeset)
> +			state->active_crtcs = dev_priv->active_crtcs;
>  	}
>  
>  	/*
> @@ -5441,21 +5432,19 @@ skl_ddb_add_affected_pipes(struct drm_atomic_state *state, bool *changed)
>  	 * any other display updates race with this transaction, so we need
>  	 * to grab the lock on *all* CRTC's.
>  	 */
> -	if (intel_state->active_pipe_changes || intel_state->modeset) {
> +	if (state->active_pipe_changes || state->modeset) {
>  		realloc_pipes = ~0;
> -		intel_state->wm_results.dirty_pipes = ~0;
> +		state->wm_results.dirty_pipes = ~0;
>  	}
>  
>  	/*
>  	 * We're not recomputing for the pipes not included in the commit, so
>  	 * make sure we start with the current state.
>  	 */
> -	for_each_intel_crtc_mask(dev, intel_crtc, realloc_pipes) {
> -		struct intel_crtc_state *cstate;
> -
> -		cstate = intel_atomic_get_crtc_state(state, intel_crtc);
> -		if (IS_ERR(cstate))
> -			return PTR_ERR(cstate);
> +	for_each_intel_crtc_mask(dev, crtc, realloc_pipes) {
> +		crtc_state = intel_atomic_get_crtc_state(&state->base, crtc);
> +		if (IS_ERR(crtc_state))
> +			return PTR_ERR(crtc_state);
>  	}
>  
>  	return 0;
> @@ -5522,12 +5511,11 @@ static int skl_wm_add_affected_planes(struct intel_atomic_state *state,
>  }
>  
>  static int
> -skl_compute_wm(struct drm_atomic_state *state)
> +skl_compute_wm(struct intel_atomic_state *state)
>  {
> -	struct drm_crtc *crtc;
> -	struct drm_crtc_state *cstate;
> -	struct intel_atomic_state *intel_state = to_intel_atomic_state(state);
> -	struct skl_ddb_values *results = &intel_state->wm_results;
> +	struct intel_crtc *crtc;
> +	struct intel_crtc_state *cstate;
> +	struct skl_ddb_values *results = &state->wm_results;
>  	struct skl_pipe_wm *pipe_wm;
>  	bool changed = false;
>  	int ret, i;
> @@ -5553,33 +5541,30 @@ skl_compute_wm(struct drm_atomic_state *state)
>  	 * should allow skl_update_pipe_wm() to return failure in cases where
>  	 * no suitable watermark values can be found.
>  	 */
> -	for_each_new_crtc_in_state(state, crtc, cstate, i) {
> -		struct intel_crtc_state *intel_cstate =
> -			to_intel_crtc_state(cstate);
> +	for_each_new_intel_crtc_in_state(state, crtc, cstate, i) {
>  		const struct skl_pipe_wm *old_pipe_wm =
> -			&to_intel_crtc_state(crtc->state)->wm.skl.optimal;
> +			&to_intel_crtc_state(crtc->base.state)->wm.skl.optimal;
>  
> -		pipe_wm = &intel_cstate->wm.skl.optimal;
> +		pipe_wm = &cstate->wm.skl.optimal;
>  		ret = skl_update_pipe_wm(cstate, old_pipe_wm, pipe_wm, &changed);
>  		if (ret)
>  			return ret;
>  
> -		ret = skl_wm_add_affected_planes(intel_state,
> -						 to_intel_crtc(crtc));
> +		ret = skl_wm_add_affected_planes(state, crtc);
>  		if (ret)
>  			return ret;
>  
>  		if (changed)
> -			results->dirty_pipes |= drm_crtc_mask(crtc);
> +			results->dirty_pipes |= drm_crtc_mask(&crtc->base);
>  
> -		if ((results->dirty_pipes & drm_crtc_mask(crtc)) == 0)
> +		if ((results->dirty_pipes & drm_crtc_mask(&crtc->base)) == 0)
>  			/* This pipe's WM's did not change */
>  			continue;
>  
> -		intel_cstate->update_wm_pre = true;
> +		cstate->update_wm_pre = true;
>  	}
>  
> -	skl_print_wm_changes(intel_state);
> +	skl_print_wm_changes(state);
>  
>  	return 0;
>  }
> @@ -5617,13 +5602,13 @@ static void skl_initial_wm(struct intel_atomic_state *state,
>  	mutex_unlock(&dev_priv->wm.wm_mutex);
>  }
>  
> -static void ilk_compute_wm_config(struct drm_device *dev,
> +static void ilk_compute_wm_config(struct drm_i915_private *dev_priv,
>  				  struct intel_wm_config *config)
>  {
>  	struct intel_crtc *crtc;
>  
>  	/* Compute the currently _active_ config */
> -	for_each_intel_crtc(dev, crtc) {
> +	for_each_intel_crtc(&dev_priv->drm, crtc) {
>  		const struct intel_pipe_wm *wm = &crtc->wm.active.ilk;
>  
>  		if (!wm->pipe_enabled)
> @@ -5637,25 +5622,24 @@ static void ilk_compute_wm_config(struct drm_device *dev,
>  
>  static void ilk_program_watermarks(struct drm_i915_private *dev_priv)
>  {
> -	struct drm_device *dev = &dev_priv->drm;
>  	struct intel_pipe_wm lp_wm_1_2 = {}, lp_wm_5_6 = {}, *best_lp_wm;
>  	struct ilk_wm_maximums max;
>  	struct intel_wm_config config = {};
>  	struct ilk_wm_values results = {};
>  	enum intel_ddb_partitioning partitioning;
>  
> -	ilk_compute_wm_config(dev, &config);
> +	ilk_compute_wm_config(dev_priv, &config);
>  
> -	ilk_compute_wm_maximums(dev, 1, &config, INTEL_DDB_PART_1_2, &max);
> -	ilk_wm_merge(dev, &config, &max, &lp_wm_1_2);
> +	ilk_compute_wm_maximums(dev_priv, 1, &config, INTEL_DDB_PART_1_2, &max);
> +	ilk_wm_merge(dev_priv, &config, &max, &lp_wm_1_2);
>  
>  	/* 5/6 split only in single pipe config on IVB+ */
>  	if (INTEL_GEN(dev_priv) >= 7 &&
>  	    config.num_pipes_active == 1 && config.sprites_enabled) {
> -		ilk_compute_wm_maximums(dev, 1, &config, INTEL_DDB_PART_5_6, &max);
> -		ilk_wm_merge(dev, &config, &max, &lp_wm_5_6);
> +		ilk_compute_wm_maximums(dev_priv, 1, &config, INTEL_DDB_PART_5_6, &max);
> +		ilk_wm_merge(dev_priv, &config, &max, &lp_wm_5_6);
>  
> -		best_lp_wm = ilk_find_best_result(dev, &lp_wm_1_2, &lp_wm_5_6);
> +		best_lp_wm = ilk_find_best_result(dev_priv, &lp_wm_1_2, &lp_wm_5_6);
>  	} else {
>  		best_lp_wm = &lp_wm_1_2;
>  	}
> @@ -5663,7 +5647,7 @@ static void ilk_program_watermarks(struct drm_i915_private *dev_priv)
>  	partitioning = (best_lp_wm == &lp_wm_1_2) ?
>  		       INTEL_DDB_PART_1_2 : INTEL_DDB_PART_5_6;
>  
> -	ilk_compute_wm_results(dev, best_lp_wm, partitioning, &results);
> +	ilk_compute_wm_results(dev_priv, best_lp_wm, partitioning, &results);
>  
>  	ilk_write_wm_values(dev_priv, &results);
>  }
> @@ -5703,19 +5687,18 @@ static inline void skl_wm_level_from_reg_val(uint32_t val,
>  		PLANE_WM_LINES_MASK;
>  }
>  
> -void skl_pipe_wm_get_hw_state(struct drm_crtc *crtc,
> +void skl_pipe_wm_get_hw_state(struct intel_crtc *crtc,
>  			      struct skl_pipe_wm *out)
>  {
> -	struct drm_i915_private *dev_priv = to_i915(crtc->dev);
> -	struct intel_crtc *intel_crtc = to_intel_crtc(crtc);
> -	enum pipe pipe = intel_crtc->pipe;
> +	struct drm_i915_private *dev_priv = to_i915(crtc->base.dev);
> +	enum pipe pipe = crtc->pipe;
>  	int level, max_level;
>  	enum plane_id plane_id;
>  	uint32_t val;
>  
>  	max_level = ilk_wm_max_level(dev_priv);
>  
> -	for_each_plane_id_on_crtc(intel_crtc, plane_id) {
> +	for_each_plane_id_on_crtc(crtc, plane_id) {
>  		struct skl_plane_wm *wm = &out->planes[plane_id];
>  
>  		for (level = 0; level <= max_level; level++) {
> @@ -5735,30 +5718,27 @@ void skl_pipe_wm_get_hw_state(struct drm_crtc *crtc,
>  		skl_wm_level_from_reg_val(val, &wm->trans_wm);
>  	}
>  
> -	if (!intel_crtc->active)
> +	if (!crtc->active)
>  		return;
>  
>  	out->linetime = I915_READ(PIPE_WM_LINETIME(pipe));
>  }
>  
> -void skl_wm_get_hw_state(struct drm_device *dev)
> +void skl_wm_get_hw_state(struct drm_i915_private *dev_priv)
>  {
> -	struct drm_i915_private *dev_priv = to_i915(dev);
>  	struct skl_ddb_values *hw = &dev_priv->wm.skl_hw;
>  	struct skl_ddb_allocation *ddb = &dev_priv->wm.skl_hw.ddb;
> -	struct drm_crtc *crtc;
> -	struct intel_crtc *intel_crtc;
> +	struct intel_crtc *crtc;
>  	struct intel_crtc_state *cstate;
>  
>  	skl_ddb_get_hw_state(dev_priv, ddb);
> -	list_for_each_entry(crtc, &dev->mode_config.crtc_list, head) {
> -		intel_crtc = to_intel_crtc(crtc);
> -		cstate = to_intel_crtc_state(crtc->state);
> +	for_each_intel_crtc(&dev_priv->drm, crtc) {
> +		cstate = to_intel_crtc_state(crtc->base.state);
>  
>  		skl_pipe_wm_get_hw_state(crtc, &cstate->wm.skl.optimal);
>  
> -		if (intel_crtc->active)
> -			hw->dirty_pipes |= drm_crtc_mask(crtc);
> +		if (crtc->active)
> +			hw->dirty_pipes |= drm_crtc_mask(&crtc->base);
>  	}
>  
>  	if (dev_priv->active_crtcs) {
> @@ -5767,15 +5747,14 @@ void skl_wm_get_hw_state(struct drm_device *dev)
>  	}
>  }
>  
> -static void ilk_pipe_wm_get_hw_state(struct drm_crtc *crtc)
> +static void ilk_pipe_wm_get_hw_state(struct intel_crtc *crtc)
>  {
> -	struct drm_device *dev = crtc->dev;
> +	struct drm_device *dev = crtc->base.dev;
>  	struct drm_i915_private *dev_priv = to_i915(dev);
>  	struct ilk_wm_values *hw = &dev_priv->wm.hw;
> -	struct intel_crtc *intel_crtc = to_intel_crtc(crtc);
> -	struct intel_crtc_state *cstate = to_intel_crtc_state(crtc->state);
> +	struct intel_crtc_state *cstate = to_intel_crtc_state(crtc->base.state);
>  	struct intel_pipe_wm *active = &cstate->wm.ilk.optimal;
> -	enum pipe pipe = intel_crtc->pipe;
> +	enum pipe pipe = crtc->pipe;
>  	static const i915_reg_t wm0_pipe_reg[] = {
>  		[PIPE_A] = WM0_PIPEA_ILK,
>  		[PIPE_B] = WM0_PIPEB_ILK,
> @@ -5788,7 +5767,7 @@ static void ilk_pipe_wm_get_hw_state(struct drm_crtc *crtc)
>  
>  	memset(active, 0, sizeof(*active));
>  
> -	active->pipe_enabled = intel_crtc->active;
> +	active->pipe_enabled = crtc->active;
>  
>  	if (active->pipe_enabled) {
>  		u32 tmp = hw->wm_pipe[pipe];
> @@ -5816,7 +5795,7 @@ static void ilk_pipe_wm_get_hw_state(struct drm_crtc *crtc)
>  			active->wm[level].enable = true;
>  	}
>  
> -	intel_crtc->wm.active.ilk = *active;
> +	crtc->wm.active.ilk = *active;
>  }
>  
>  #define _FW_WM(value, plane) \
> @@ -5926,9 +5905,8 @@ static void vlv_read_wm_values(struct drm_i915_private *dev_priv,
>  #undef _FW_WM
>  #undef _FW_WM_VLV
>  
> -void g4x_wm_get_hw_state(struct drm_device *dev)
> +void g4x_wm_get_hw_state(struct drm_i915_private *dev_priv)
>  {
> -	struct drm_i915_private *dev_priv = to_i915(dev);
>  	struct g4x_wm_values *wm = &dev_priv->wm.g4x;
>  	struct intel_crtc *crtc;
>  
> @@ -5936,7 +5914,7 @@ void g4x_wm_get_hw_state(struct drm_device *dev)
>  
>  	wm->cxsr = I915_READ(FW_BLC_SELF) & FW_BLC_SELF_EN;
>  
> -	for_each_intel_crtc(dev, crtc) {
> +	for_each_intel_crtc(&dev_priv->drm, crtc) {
>  		struct intel_crtc_state *crtc_state =
>  			to_intel_crtc_state(crtc->base.state);
>  		struct g4x_wm_state *active = &crtc->wm.active.g4x;
> @@ -6067,9 +6045,8 @@ void g4x_wm_sanitize(struct drm_i915_private *dev_priv)
>  	mutex_unlock(&dev_priv->wm.wm_mutex);
>  }
>  
> -void vlv_wm_get_hw_state(struct drm_device *dev)
> +void vlv_wm_get_hw_state(struct drm_i915_private *dev_priv)
>  {
> -	struct drm_i915_private *dev_priv = to_i915(dev);
>  	struct vlv_wm_values *wm = &dev_priv->wm.vlv;
>  	struct intel_crtc *crtc;
>  	u32 val;
> @@ -6113,7 +6090,7 @@ void vlv_wm_get_hw_state(struct drm_device *dev)
>  		mutex_unlock(&dev_priv->pcu_lock);
>  	}
>  
> -	for_each_intel_crtc(dev, crtc) {
> +	for_each_intel_crtc(&dev_priv->drm, crtc) {
>  		struct intel_crtc_state *crtc_state =
>  			to_intel_crtc_state(crtc->base.state);
>  		struct vlv_wm_state *active = &crtc->wm.active.vlv;
> @@ -6230,15 +6207,14 @@ static void ilk_init_lp_watermarks(struct drm_i915_private *dev_priv)
>  	 */
>  }
>  
> -void ilk_wm_get_hw_state(struct drm_device *dev)
> +void ilk_wm_get_hw_state(struct drm_i915_private *dev_priv)
>  {
> -	struct drm_i915_private *dev_priv = to_i915(dev);
>  	struct ilk_wm_values *hw = &dev_priv->wm.hw;
> -	struct drm_crtc *crtc;
> +	struct intel_crtc *crtc;
>  
>  	ilk_init_lp_watermarks(dev_priv);
>  
> -	for_each_crtc(dev, crtc)
> +	for_each_intel_crtc(&dev_priv->drm, crtc)
>  		ilk_pipe_wm_get_hw_state(crtc);
>  
>  	hw->wm_lp[0] = I915_READ(WM1_LP_ILK);
> -- 
> 2.14.4
> 
> _______________________________________________
> Intel-gfx mailing list
> Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
> https://lists.freedesktop.org/mailman/listinfo/intel-gfx

-- 
Ville Syrjälä
Intel
_______________________________________________
Intel-gfx mailing list
Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/intel-gfx




[Index of Archives]     [AMD Graphics]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux