[Intel-gfx] [PATCH 4/7] drm/i915: Adding atomic fitting mode property for GEN9

[Nabendu Maiti]

Pipe scaler Fitting mode property is added as crtc atomic property.

Signed-off-by: Nabendu Maiti <nabendu.bikash.maiti at intel.com>
---
 drivers/gpu/drm/drm_atomic.c         |  5 ++++
 drivers/gpu/drm/drm_crtc.c           |  8 ++++++
 drivers/gpu/drm/i915/intel_display.c | 47 ++++++++++--------------------------
 include/drm/drm_crtc.h               |  2 ++
 4 files changed, 28 insertions(+), 34 deletions(-)

diff --git a/drivers/gpu/drm/drm_atomic.c b/drivers/gpu/drm/drm_atomic.c
index 066f010..3fddfa4 100644
--- a/drivers/gpu/drm/drm_atomic.c
+++ b/drivers/gpu/drm/drm_atomic.c
@@ -499,6 +499,9 @@ int drm_atomic_crtc_set_property(struct drm_crtc *crtc,
 	} else if (property == config->prop_pipe_src_h) {
 		state->src_h = val;
 		state->pipescaler_changed = true;
+	} else if (property == config->prop_fitting_mode) {
+		state->fitting_mode = val;
+		state->pipescaler_changed = true;
 	} else if (crtc->funcs->atomic_set_property)
 		return crtc->funcs->atomic_set_property(crtc, state, property, val);
 	else
@@ -545,6 +548,8 @@ drm_atomic_crtc_get_property(struct drm_crtc *crtc,
 		*val =  state->src_w;
 	else if (property == config->prop_pipe_src_h)
 		*val =  state->src_h;
+	else if (property == config->prop_fitting_mode)
+		*val = state->fitting_mode;
 	else if (crtc->funcs->atomic_get_property)
 		return crtc->funcs->atomic_get_property(crtc, state, property, val);
 	else
diff --git a/drivers/gpu/drm/drm_crtc.c b/drivers/gpu/drm/drm_crtc.c
index 6fd6dd8..b24e839 100644
--- a/drivers/gpu/drm/drm_crtc.c
+++ b/drivers/gpu/drm/drm_crtc.c
@@ -381,6 +381,8 @@ int drm_crtc_init_with_planes(struct drm_device *dev, struct drm_crtc *crtc,
 					   config->prop_pipe_src_w, 0);
 		drm_object_attach_property(&crtc->base,
 					   config->prop_pipe_src_h, 0);
+		drm_object_attach_property(&crtc->base,
+					   config->prop_fitting_mode, 0);
 	}
 
 	return 0;
@@ -959,6 +961,12 @@ static int drm_mode_create_standard_properties(struct drm_device *dev)
 		return -ENOMEM;
 	dev->mode_config.prop_pipe_src_h = prop;
 
+	prop = drm_property_create_range(dev, DRM_MODE_PROP_ATOMIC,
+			"SCALER_MODE", 0, INT_MAX);
+	if (!prop)
+		return -ENOMEM;
+	dev->mode_config.prop_fitting_mode = prop;
+
 	return 0;
 }
 
diff --git a/drivers/gpu/drm/i915/intel_display.c b/drivers/gpu/drm/i915/intel_display.c
index ff5dea8..e3ecb8b 100644
--- a/drivers/gpu/drm/i915/intel_display.c
+++ b/drivers/gpu/drm/i915/intel_display.c
@@ -12551,48 +12551,25 @@ static bool check_single_encoder_cloning(struct drm_atomic_state *state,
 static int skylake_pfiter_calculate(struct drm_crtc *crtc,
 				   struct drm_crtc_state *crtc_state)
 {
-	struct drm_device *dev = crtc->dev;
 	struct intel_crtc *intel_crtc = to_intel_crtc(crtc);
 	struct intel_crtc_state *pipe_config =
 		to_intel_crtc_state(crtc_state);
 	bool mode_changed = needs_modeset(crtc_state);
-	struct intel_connector *intel_connector;
 	int ret = 0;
-	struct drm_display_mode *adjusted_mode =
-		&intel_crtc->config->base.adjusted_mode;
-
-	for_each_intel_connector(dev, intel_connector) {
-		if (!(intel_connector) || !(intel_connector->encoder) ||
-		    (intel_connector->encoder->base.crtc  != crtc))
-			continue;
 
-		if (((pipe_config->pipe_src_w !=
-		      intel_crtc->config->pipe_src_w) ||
-		     (pipe_config->pipe_src_h !=
-		      intel_crtc->config->pipe_src_h)) && (!mode_changed))
-			pipe_config->update_pipe = true;
-
-		if ((pipe_config->pipe_scaling_mode !=
-		    intel_connector->panel.fitting_mode) &&
-		    (!mode_changed)) {
-			if  ((adjusted_mode->hdisplay !=
-			      pipe_config->pipe_src_w) ||
-			     (adjusted_mode->vdisplay !=
-			      pipe_config->pipe_src_h)) {
-				pipe_config->pipe_scaling_mode =
-					intel_connector->panel.fitting_mode;
-				pipe_config->update_pipe = true;
-			}
-		}
+	if (((pipe_config->pipe_scaling_mode !=
+	      intel_crtc->config->pipe_scaling_mode) ||
+	     (pipe_config->pipe_src_w != intel_crtc->config->pipe_src_w) ||
+	     (pipe_config->pipe_src_h != intel_crtc->config->pipe_src_h)) &&
+	    (!mode_changed)) {
+		pipe_config->update_pipe = true;
+	}
 
-		if ((mode_changed) || (pipe_config->update_pipe)) {
-			ret = skl_update_scaler_crtc(pipe_config);
-			if (ret)
-				break;
+	if ((mode_changed) || (pipe_config->update_pipe)) {
+		ret = skl_update_scaler_crtc(pipe_config);
+		if (!ret)
 			intel_pch_panel_fitting(intel_crtc, pipe_config,
-					intel_connector->panel.fitting_mode);
-			break;
-		}
+						pipe_config->pipe_scaling_mode);
 	}
 	return ret;
 }
@@ -14054,6 +14031,8 @@ static int intel_atomic_check(struct drm_device *dev,
 			}
 			pipe_config->pipe_src_w = crtc_state->src_w;
 			pipe_config->pipe_src_h = crtc_state->src_h;
+			pipe_config->pipe_scaling_mode =
+				crtc_state->fitting_mode;
 			crtc_state->pipescaler_changed = false;
 		}
 
diff --git a/include/drm/drm_crtc.h b/include/drm/drm_crtc.h
index 4d6b580..e245746 100644
--- a/include/drm/drm_crtc.h
+++ b/include/drm/drm_crtc.h
@@ -200,6 +200,7 @@ struct drm_crtc_state {
 
 	bool pipescaler_changed;
 	u32 src_w, src_h;
+	u32 fitting_mode;
 };
 
 /**
@@ -1948,6 +1949,7 @@ struct drm_mode_config {
 	/* pipe scaler properties */
 	struct drm_property *prop_pipe_src_w;
 	struct drm_property *prop_pipe_src_h;
+	struct drm_property *prop_fitting_mode;
 
 	/**
 	 * @dvi_i_subconnector_property: Optional DVI-I property to
-- 
1.9.1