drm/i915: Simplify the intel_color_init_hooks() if ladder
authorVille Syrjälä <ville.syrjala@linux.intel.com>
Thu, 29 Sep 2022 07:15:14 +0000 (10:15 +0300)
committerVille Syrjälä <ville.syrjala@linux.intel.com>
Fri, 30 Sep 2022 12:58:39 +0000 (15:58 +0300)
Get rid of the funny hsw vs. ivb extra indentation level in
intel_color_init_hooks().

Signed-off-by: Ville Syrjälä <ville.syrjala@linux.intel.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20220929071521.26612-4-ville.syrjala@linux.intel.com
Reviewed-by: Jani Nikula <jani.nikula@intel.com>
drivers/gpu/drm/i915/display/intel_color.c

index ddfe7c2..ce8a4be 100644 (file)
@@ -2238,12 +2238,11 @@ void intel_color_init_hooks(struct drm_i915_private *dev_priv)
                        dev_priv->display.funcs.color = &skl_color_funcs;
                else if (DISPLAY_VER(dev_priv) == 8)
                        dev_priv->display.funcs.color = &bdw_color_funcs;
-               else if (DISPLAY_VER(dev_priv) == 7) {
-                       if (IS_HASWELL(dev_priv))
-                               dev_priv->display.funcs.color = &hsw_color_funcs;
-                       else
-                               dev_priv->display.funcs.color = &ivb_color_funcs;
-               } else
+               else if (IS_HASWELL(dev_priv))
+                       dev_priv->display.funcs.color = &hsw_color_funcs;
+               else if (DISPLAY_VER(dev_priv) == 7)
+                       dev_priv->display.funcs.color = &ivb_color_funcs;
+               else
                        dev_priv->display.funcs.color = &ilk_color_funcs;
        }
 }