[2/2] drm/i915: Remove the double handling of 'flags from intel_mode_from_pipe_config()

Submitted by Ville Syrjala on Dec. 22, 2016, 2:04 p.m.

Details

Message ID 20161222140442.7204-2-ville.syrjala@linux.intel.com
State Accepted
Commit e96128235b0fbddff1228b37033077397dbd21d9
Headers show
Series "Series without cover letter" ( rev: 1 ) in Intel GFX

Not browsing as part of any series.

Commit Message

Ville Syrjala Dec. 22, 2016, 2:04 p.m. 
From: Ville Syrjälä <ville.syrjala@linux.intel.com>

We're trying to copy the flags from the adjusted mode to the
passed in mode twice. Once is enough.

Signed-off-by: Ville Syrjälä <ville.syrjala@linux.intel.com>
---
 drivers/gpu/drm/i915/intel_display.c | 1 -
 1 file changed, 1 deletion(-)

Patch hide | download patch | download mbox 

diff --git a/drivers/gpu/drm/i915/intel_display.c b/drivers/gpu/drm/i915/intel_display.c
index 091c192c6c5c..e4c573322b6c 100644
--- a/drivers/gpu/drm/i915/intel_display.c
+++ b/drivers/gpu/drm/i915/intel_display.c
@@ -8404,7 +8404,6 @@  void intel_mode_from_pipe_config(struct drm_display_mode *mode,
 	mode->type = DRM_MODE_TYPE_DRIVER;
 
 	mode->clock = pipe_config->base.adjusted_mode.crtc_clock;
-	mode->flags |= pipe_config->base.adjusted_mode.flags;
 
 	mode->hsync = drm_mode_hsync(mode);
 	mode->vrefresh = drm_mode_vrefresh(mode);

Comments

Chris Wilson Dec. 22, 2016, 2:09 p.m. 
On Thu, Dec 22, 2016 at 04:04:42PM +0200, ville.syrjala@linux.intel.com wrote:
> From: Ville Syrjälä <ville.syrjala@linux.intel.com>
> 
> We're trying to copy the flags from the adjusted mode to the
> passed in mode twice. Once is enough.
> 
> Signed-off-by: Ville Syrjälä <ville.syrjala@linux.intel.com>
Reviewed-by: Chris Wilson <chris@chris-wilson.co.uk>
-Chris