drm/i915: Add "missing" break to haswell_get_ddi_pll()

vsyrjala · vsyrjala · commit 79bd23dadf61 · 2015-12-02T11:24:13.000+02:00
While not technically needed on the last case in the switch statement, the 'break' makes it look better IMO. v2: Fixed a typo in the commit message (Paulo) Signed-off-by: Ville Syrjälä <ville.syrjala@linux.intel.com> Reviewed-by: Paulo Zanoni <paulo.r.zanoni@intel.com> Link: http://patchwork.freedesktop.org/patch/msgid/1449005527-15617-1-git-send-email-ville.syrjala@linux.intel.com
diff --git a/drivers/gpu/drm/i915/intel_display.c b/drivers/gpu/drm/i915/intel_display.c
@@ -9808,6 +9808,7 @@ static void haswell_get_ddi_pll(struct drm_i915_private *dev_priv,
 		break;
 	case PORT_CLK_SEL_SPLL:
 		pipe_config->shared_dpll = DPLL_ID_SPLL;
+		break;
 	}
 }
 

Original file line number	Diff line number	Diff line change
`@@ -9808,6 +9808,7 @@ static void haswell_get_ddi_pll(struct drm_i915_private *dev_priv,`
`9808`	`9808`	`break;`
`9809`	`9809`	`case PORT_CLK_SEL_SPLL:`
`9810`	`9810`	`pipe_config->shared_dpll = DPLL_ID_SPLL;`
	`9811`	`+ break;`
`9811`	`9812`	`}`
`9812`	`9813`	`}`
`9813`	`9814`