drm/i915: Remove pointless if-else from sdvo code
authorVille Syrjälä <ville.syrjala@linux.intel.com>
Thu, 21 Jun 2018 17:46:58 +0000 (20:46 +0300)
committerVille Syrjälä <ville.syrjala@linux.intel.com>
Thu, 5 Jul 2018 19:16:56 +0000 (22:16 +0300)
The return value is a bool so we can just return the result of
the biwise AND. The compiler will take care of the rest.

Signed-off-by: Ville Syrjälä <ville.syrjala@linux.intel.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20180621174658.18823-1-ville.syrjala@linux.intel.com
Reviewed-by: Chris Wilson <chris@chris-wilson.co.uk>
drivers/gpu/drm/i915/intel_sdvo.c

index 36f81a9..5417c54 100644 (file)
@@ -1401,10 +1401,7 @@ static bool intel_sdvo_connector_get_hw_state(struct intel_connector *connector)
 
        intel_sdvo_get_active_outputs(intel_sdvo, &active_outputs);
 
-       if (active_outputs & intel_sdvo_connector->output_flag)
-               return true;
-       else
-               return false;
+       return active_outputs & intel_sdvo_connector->output_flag;
 }
 
 bool intel_sdvo_port_enabled(struct drm_i915_private *dev_priv,