Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit dd57602e authored by Ville Syrjälä's avatar Ville Syrjälä
Browse files

drm/i915: Switch fbc over to for_each_new_intel_plane_in_state() 



Stop using the old for_each_intel_plane_in_state() type iteration
macro and replace it with for_each_new_intel_plane_in_state().
And similarly replace drm_atomic_get_existing_crtc_state() with
intel_atomic_get_new_crtc_state(). Switch over to intel_ types
as well to make the code less cluttered.

v2: s/plane/i9xx_plane/ etc. (James)

Cc: James Ausmus <james.ausmus@intel.com>
Cc: Maarten Lankhorst <maarten.lankhorst@linux.intel.com>
Reviewed-by: default avatarDaniel Vetter <daniel.vetter@ffwll.ch>
Link: https://patchwork.freedesktop.org/patch/msgid/20171117191917.11506-8-ville.syrjala@linux.intel.com


Signed-off-by: default avatarVille Syrjälä <ville.syrjala@linux.intel.com>
parent 81894b2f

drivers/gpu/drm/i915/i915_drv.h

+3 −4
Original line number Diff line number Diff line
@@ -560,13 +560,13 @@ struct i915_hotplug { 
	for_each_power_well_rev(__dev_priv, __power_well)		        \

		for_each_if ((__power_well)->domains & (__domain_mask))



#define for_each_intel_plane_in_state(__state, plane, plane_state, __i) \

#define for_each_new_intel_plane_in_state(__state, plane, new_plane_state, __i) \

	for ((__i) = 0; \

	     (__i) < (__state)->base.dev->mode_config.num_total_plane && \

		     ((plane) = to_intel_plane((__state)->base.planes[__i].ptr), \

		      (plane_state) = to_intel_plane_state((__state)->base.planes[__i].state), 1); \

		      (new_plane_state) = to_intel_plane_state((__state)->base.planes[__i].new_state), 1); \

	     (__i)++) \

		for_each_if (plane_state)

		for_each_if (plane)



#define for_each_new_intel_crtc_in_state(__state, crtc, new_crtc_state, __i) \

	for ((__i) = 0; \
@@ -576,7 +576,6 @@ struct i915_hotplug { 
	     (__i)++) \

		for_each_if (crtc)





#define for_each_oldnew_intel_plane_in_state(__state, plane, old_plane_state, new_plane_state, __i) \

	for ((__i) = 0; \

	     (__i) < (__state)->base.dev->mode_config.num_total_plane && \

drivers/gpu/drm/i915/intel_display.c

+1 −1
Original line number Diff line number Diff line
@@ -12080,7 +12080,7 @@ static int intel_atomic_check(struct drm_device *dev, 
	if (ret)

		return ret;



	intel_fbc_choose_crtc(dev_priv, state);

	intel_fbc_choose_crtc(dev_priv, intel_state);

	return calc_watermark_data(state);

}

drivers/gpu/drm/i915/intel_drv.h

+1 −1
Original line number Diff line number Diff line
@@ -1652,7 +1652,7 @@ static inline void intel_fbdev_restore_mode(struct drm_device *dev) 


/* intel_fbc.c */

void intel_fbc_choose_crtc(struct drm_i915_private *dev_priv,

			   struct drm_atomic_state *state);

			   struct intel_atomic_state *state);

bool intel_fbc_is_active(struct drm_i915_private *dev_priv);

void intel_fbc_pre_update(struct intel_crtc *crtc,

			  struct intel_crtc_state *crtc_state,

drivers/gpu/drm/i915/intel_fbc.c

+10 −13
Original line number Diff line number Diff line
@@ -1054,11 +1054,11 @@ void intel_fbc_flush(struct drm_i915_private *dev_priv, 
 * enable FBC for the chosen CRTC. If it does, it will set dev_priv->fbc.crtc.

 */

void intel_fbc_choose_crtc(struct drm_i915_private *dev_priv,

			   struct drm_atomic_state *state)

			   struct intel_atomic_state *state)

{

	struct intel_fbc *fbc = &dev_priv->fbc;

	struct drm_plane *plane;

	struct drm_plane_state *plane_state;

	struct intel_plane *plane;

	struct intel_plane_state *plane_state;

	bool crtc_chosen = false;

	int i;
@@ -1066,7 +1066,7 @@ void intel_fbc_choose_crtc(struct drm_i915_private *dev_priv, 


	/* Does this atomic commit involve the CRTC currently tied to FBC? */

	if (fbc->crtc &&

	    !drm_atomic_get_existing_crtc_state(state, &fbc->crtc->base))

	    !intel_atomic_get_new_crtc_state(state, fbc->crtc))

		goto out;



	if (!intel_fbc_can_enable(dev_priv))
@@ -1076,13 +1076,11 @@ void intel_fbc_choose_crtc(struct drm_i915_private *dev_priv, 
	 * plane. We could go for fancier schemes such as checking the plane

	 * size, but this would just affect the few platforms that don't tie FBC

	 * to pipe or plane A. */

	for_each_new_plane_in_state(state, plane, plane_state, i) {

		struct intel_plane_state *intel_plane_state =

			to_intel_plane_state(plane_state);

		struct intel_crtc_state *intel_crtc_state;

		struct intel_crtc *crtc = to_intel_crtc(plane_state->crtc);

	for_each_new_intel_plane_in_state(state, plane, plane_state, i) {

		struct intel_crtc_state *crtc_state;

		struct intel_crtc *crtc = to_intel_crtc(plane_state->base.crtc);



		if (!intel_plane_state->base.visible)

		if (!plane_state->base.visible)

			continue;



		if (fbc_on_pipe_a_only(dev_priv) && crtc->pipe != PIPE_A)
@@ -1091,10 +1089,9 @@ void intel_fbc_choose_crtc(struct drm_i915_private *dev_priv, 
		if (fbc_on_plane_a_only(dev_priv) && crtc->i9xx_plane != PLANE_A)

			continue;



		intel_crtc_state = to_intel_crtc_state(

			drm_atomic_get_existing_crtc_state(state, &crtc->base));

		crtc_state = intel_atomic_get_new_crtc_state(state, crtc);



		intel_crtc_state->enable_fbc = true;

		crtc_state->enable_fbc = true;

		crtc_chosen = true;

		break;

	}