diff options
author | Ander Conselvan de Oliveira <ander.conselvan.de.oliveira@intel.com> | 2015-04-21 10:12:57 -0400 |
---|---|---|
committer | Daniel Vetter <daniel.vetter@ffwll.ch> | 2015-05-08 07:03:57 -0400 |
commit | e5d958ef42ff99ab2cf7980f4b7317d3bbd8793f (patch) | |
tree | 1ffd527f6325832fdfa9b537efafd4134f96eb5b /drivers/gpu/drm/i915/intel_display.c | |
parent | ce22dba92de22e951dee2ff89937a39754d2dd91 (diff) |
drm/i915: Don't check for NULL before freeing state
This is not necessary after the below commit.
commit a0211bb482c346820506c546a6a58b8357999a99
Author: Ander Conselvan de Oliveira <ander.conselvan.de.oliveira@intel.com>
Date: Mon Mar 30 14:05:43 2015 +0300
drm/atomic: Don't try to free a NULL state
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>
Diffstat (limited to 'drivers/gpu/drm/i915/intel_display.c')
-rw-r--r-- | drivers/gpu/drm/i915/intel_display.c | 9 |
1 files changed, 3 insertions, 6 deletions
diff --git a/drivers/gpu/drm/i915/intel_display.c b/drivers/gpu/drm/i915/intel_display.c index d5d648030af0..1375f53cdc95 100644 --- a/drivers/gpu/drm/i915/intel_display.c +++ b/drivers/gpu/drm/i915/intel_display.c | |||
@@ -9860,10 +9860,8 @@ retry: | |||
9860 | fail: | 9860 | fail: |
9861 | intel_crtc->new_enabled = crtc->state->enable; | 9861 | intel_crtc->new_enabled = crtc->state->enable; |
9862 | fail_unlock: | 9862 | fail_unlock: |
9863 | if (state) { | 9863 | drm_atomic_state_free(state); |
9864 | drm_atomic_state_free(state); | 9864 | state = NULL; |
9865 | state = NULL; | ||
9866 | } | ||
9867 | 9865 | ||
9868 | if (ret == -EDEADLK) { | 9866 | if (ret == -EDEADLK) { |
9869 | drm_modeset_backoff(ctx); | 9867 | drm_modeset_backoff(ctx); |
@@ -13003,8 +13001,7 @@ fail: | |||
13003 | } | 13001 | } |
13004 | 13002 | ||
13005 | out_config: | 13003 | out_config: |
13006 | if (state) | 13004 | drm_atomic_state_free(state); |
13007 | drm_atomic_state_free(state); | ||
13008 | 13005 | ||
13009 | intel_set_config_free(config); | 13006 | intel_set_config_free(config); |
13010 | return ret; | 13007 | return ret; |