Skip to content

Commit 09fa8bb

Browse files
committed
drm/i915: Add some curly braces
intel_enable_pipe() looks rather confusing when one side doesn't have the curly braces, and the other one does. And what's even worse, there's another if-else inside the braceless side. Let's put braces around it to make it clear which branch goes where. Signed-off-by: Ville Syrjälä <ville.syrjala@linux.intel.com> Link: http://patchwork.freedesktop.org/patch/msgid/1470418894-1249-1-git-send-email-ville.syrjala@linux.intel.com Reviewed-by: Chris Wilson <chris@chris-wilson.co.uk>
1 parent 5a198b8 commit 09fa8bb

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

drivers/gpu/drm/i915/intel_display.c

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1959,12 +1959,12 @@ static void intel_enable_pipe(struct intel_crtc *crtc)
19591959
* a plane. On ILK+ the pipe PLLs are integrated, so we don't
19601960
* need the check.
19611961
*/
1962-
if (HAS_GMCH_DISPLAY(dev_priv))
1962+
if (HAS_GMCH_DISPLAY(dev_priv)) {
19631963
if (intel_crtc_has_type(crtc->config, INTEL_OUTPUT_DSI))
19641964
assert_dsi_pll_enabled(dev_priv);
19651965
else
19661966
assert_pll_enabled(dev_priv, pipe);
1967-
else {
1967+
} else {
19681968
if (crtc->config->has_pch_encoder) {
19691969
/* if driving the PCH, we need FDI enabled */
19701970
assert_fdi_rx_pll_enabled(dev_priv, pch_transcoder);

0 commit comments

Comments
 (0)