diff options
author | Himanshu Jha <himanshujha199640@gmail.com> | 2017-08-29 09:21:27 -0400 |
---|---|---|
committer | Alex Deucher <alexander.deucher@amd.com> | 2017-08-29 15:57:32 -0400 |
commit | c5927537dd5706b17affa8aeea28c7b19c8fbb42 (patch) | |
tree | 4bcb1c9b199f032be613f02b6790dc113a8bf854 /drivers/gpu/drm/amd/amdgpu/amdgpu_connectors.c | |
parent | 3f3333f8a0e90ac26f84ed7b0aa344efce695c08 (diff) |
drm/amd: Remove null check before kfree
Kfree on NULL pointer is a no-op and therefore checking is redundant.
Reviewed-by: Christian König <christian.koenig@amd.com>
Signed-off-by: Himanshu Jha <himanshujha199640@gmail.com>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
Diffstat (limited to 'drivers/gpu/drm/amd/amdgpu/amdgpu_connectors.c')
-rw-r--r-- | drivers/gpu/drm/amd/amdgpu/amdgpu_connectors.c | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_connectors.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_connectors.c index 8d1cf2d3e663..f51b41f094ef 100644 --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_connectors.c +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_connectors.c | |||
@@ -346,10 +346,8 @@ static void amdgpu_connector_free_edid(struct drm_connector *connector) | |||
346 | { | 346 | { |
347 | struct amdgpu_connector *amdgpu_connector = to_amdgpu_connector(connector); | 347 | struct amdgpu_connector *amdgpu_connector = to_amdgpu_connector(connector); |
348 | 348 | ||
349 | if (amdgpu_connector->edid) { | 349 | kfree(amdgpu_connector->edid); |
350 | kfree(amdgpu_connector->edid); | 350 | amdgpu_connector->edid = NULL; |
351 | amdgpu_connector->edid = NULL; | ||
352 | } | ||
353 | } | 351 | } |
354 | 352 | ||
355 | static int amdgpu_connector_ddc_get_modes(struct drm_connector *connector) | 353 | static int amdgpu_connector_ddc_get_modes(struct drm_connector *connector) |