drm/i915/bios: reduce indent in parse_general_features
authorJani Nikula <jani.nikula@intel.com>
Wed, 16 Dec 2015 13:04:21 +0000 (15:04 +0200)
committerJani Nikula <jani.nikula@intel.com>
Wed, 16 Dec 2015 16:02:05 +0000 (18:02 +0200)
Slightly cleaner with early exit.

Reviewed-by: Chris Wilson <chris@chris-wilson.co.uk>
Signed-off-by: Jani Nikula <jani.nikula@intel.com>
Link: http://patchwork.freedesktop.org/patch/msgid/1450271061-32646-4-git-send-email-jani.nikula@intel.com
drivers/gpu/drm/i915/intel_bios.c

index 25edfc062f9491e5b7b051693b08927c1b79ba04..eba3e0f87181edbf3b4749f24709a35f2c83a8a9 100644 (file)
@@ -353,25 +353,26 @@ parse_general_features(struct drm_i915_private *dev_priv,
        const struct bdb_general_features *general;
 
        general = find_section(bdb, BDB_GENERAL_FEATURES);
-       if (general) {
-               dev_priv->vbt.int_tv_support = general->int_tv_support;
-               /* int_crt_support can't be trusted on earlier platforms */
-               if (bdb->version >= 155 &&
-                   (HAS_DDI(dev_priv) || IS_VALLEYVIEW(dev_priv)))
-                       dev_priv->vbt.int_crt_support = general->int_crt_support;
-               dev_priv->vbt.lvds_use_ssc = general->enable_ssc;
-               dev_priv->vbt.lvds_ssc_freq =
-                       intel_bios_ssc_frequency(dev_priv, general->ssc_freq);
-               dev_priv->vbt.display_clock_mode = general->display_clock_mode;
-               dev_priv->vbt.fdi_rx_polarity_inverted = general->fdi_rx_polarity_inverted;
-               DRM_DEBUG_KMS("BDB_GENERAL_FEATURES int_tv_support %d int_crt_support %d lvds_use_ssc %d lvds_ssc_freq %d display_clock_mode %d fdi_rx_polarity_inverted %d\n",
-                             dev_priv->vbt.int_tv_support,
-                             dev_priv->vbt.int_crt_support,
-                             dev_priv->vbt.lvds_use_ssc,
-                             dev_priv->vbt.lvds_ssc_freq,
-                             dev_priv->vbt.display_clock_mode,
-                             dev_priv->vbt.fdi_rx_polarity_inverted);
-       }
+       if (!general)
+               return;
+
+       dev_priv->vbt.int_tv_support = general->int_tv_support;
+       /* int_crt_support can't be trusted on earlier platforms */
+       if (bdb->version >= 155 &&
+           (HAS_DDI(dev_priv) || IS_VALLEYVIEW(dev_priv)))
+               dev_priv->vbt.int_crt_support = general->int_crt_support;
+       dev_priv->vbt.lvds_use_ssc = general->enable_ssc;
+       dev_priv->vbt.lvds_ssc_freq =
+               intel_bios_ssc_frequency(dev_priv, general->ssc_freq);
+       dev_priv->vbt.display_clock_mode = general->display_clock_mode;
+       dev_priv->vbt.fdi_rx_polarity_inverted = general->fdi_rx_polarity_inverted;
+       DRM_DEBUG_KMS("BDB_GENERAL_FEATURES int_tv_support %d int_crt_support %d lvds_use_ssc %d lvds_ssc_freq %d display_clock_mode %d fdi_rx_polarity_inverted %d\n",
+                     dev_priv->vbt.int_tv_support,
+                     dev_priv->vbt.int_crt_support,
+                     dev_priv->vbt.lvds_use_ssc,
+                     dev_priv->vbt.lvds_ssc_freq,
+                     dev_priv->vbt.display_clock_mode,
+                     dev_priv->vbt.fdi_rx_polarity_inverted);
 }
 
 static void