drm/i915: invert the log inside intel_prepare_ddi

Do an early return in case we don't have DDI instead of having the
whole function inside an "if" statement.

Signed-off-by: Paulo Zanoni <paulo.r.zanoni@intel.com>
Reviewed-by: Imre Deak <imre.deak@intel.com>
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>

1 files changed, 10 insertions, 9 deletions

diff --git a/drivers/gpu/drm/i915/intel_ddi.c b/drivers/gpu/drm/i915/intel_ddi.c
index 197dede1870a..ad936c68e4cb 100644
--- a/drivers/gpu/drm/i915/intel_ddi.c
+++ b/drivers/gpu/drm/i915/intel_ddi.c
@@ -115,16 +115,17 @@ void intel_prepare_ddi(struct drm_device *dev)
 {
         int port;
 
-        if (HAS_DDI(dev)) {
-                for (port = PORT_A; port < PORT_E; port++)
-                        intel_prepare_ddi_buffers(dev, port, false);
+        if (!HAS_DDI(dev))
+                return;
 
-                /* DDI E is the suggested one to work in FDI mode, so program is as such by
-                 * default. It will have to be re-programmed in case a digital DP output
-                 * will be detected on it
-                 */
-                intel_prepare_ddi_buffers(dev, PORT_E, true);
-        }
+        for (port = PORT_A; port < PORT_E; port++)
+                intel_prepare_ddi_buffers(dev, port, false);
+
+        /* DDI E is the suggested one to work in FDI mode, so program is as such
+         * by default. It will have to be re-programmed in case a digital DP
+         * output will be detected on it
+         */
+        intel_prepare_ddi_buffers(dev, PORT_E, true);
 }
 
 static const long hsw_ddi_buf_ctl_values[] = {