drm/i915: Simplify error handling in __intel_set_mode()
authorAnder Conselvan de Oliveira <ander.conselvan.de.oliveira@intel.com>
Tue, 21 Apr 2015 14:13:07 +0000 (17:13 +0300)
committerDaniel Vetter <daniel.vetter@ffwll.ch>
Fri, 8 May 2015 11:04:02 +0000 (13:04 +0200)
The remaining parts of the failure path could only be reached if the
allocation of crtc_state_copy would fail. In that case, there is nothing
to undo, so just get rid of the label for error handling and return an
error code immediately.

We also always allocate a pipe_config, even if the pipe is being
disabled, so the remaining part of what was the error/done case can be
simplified a little too.

v2: Ignore return value from drm_plane_helper_update(). (Ander)

Signed-off-by: Ander Conselvan de Oliveira <ander.conselvan.de.oliveira@intel.com>
Reviewed-by: Maarten Lankhorst <maarten.lankhorst@linux.intel.com>
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
drivers/gpu/drm/i915/intel_display.c

index 9655f1f..9aa90ab 100644 (file)
@@ -12278,10 +12278,8 @@ static int __intel_set_mode(struct drm_crtc *modeset_crtc,
                return ret;
 
        crtc_state_copy = kmalloc(sizeof(*crtc_state_copy), GFP_KERNEL);
-       if (!crtc_state_copy) {
-               ret = -ENOMEM;
-               goto done;
-       }
+       if (!crtc_state_copy)
+               return -ENOMEM;
 
        for_each_crtc_in_state(state, crtc, crtc_state, i) {
                if (!needs_modeset(crtc_state))
@@ -12337,6 +12335,7 @@ static int __intel_set_mode(struct drm_crtc *modeset_crtc,
                                              hdisplay, vdisplay,
                                              x << 16, y << 16,
                                              hdisplay << 16, vdisplay << 16);
+               WARN_ON(ret != 0);
        }
 
        /* Now enable the clocks, plane, pipe, and connectors that we set up. */
@@ -12351,21 +12350,16 @@ static int __intel_set_mode(struct drm_crtc *modeset_crtc,
        }
 
        /* FIXME: add subpixel order */
-done:
-       if (ret == 0 && pipe_config) {
-               struct intel_crtc *intel_crtc = to_intel_crtc(modeset_crtc);
-
-               /* The pipe_config will be freed with the atomic state, so
-                * make a copy. */
-               memcpy(crtc_state_copy, intel_crtc->config,
-                      sizeof *crtc_state_copy);
-               intel_crtc->config = crtc_state_copy;
-               intel_crtc->base.state = &crtc_state_copy->base;
-       } else {
-               kfree(crtc_state_copy);
-       }
 
-       return ret;
+       intel_crtc = to_intel_crtc(modeset_crtc);
+
+       /* The pipe_config will be freed with the atomic state, so
+        * make a copy. */
+       memcpy(crtc_state_copy, intel_crtc->config, sizeof *crtc_state_copy);
+       intel_crtc->config = crtc_state_copy;
+       intel_crtc->base.state = &crtc_state_copy->base;
+
+       return 0;
 }
 
 static int intel_set_mode_with_config(struct drm_crtc *crtc,