diff options
author | Colin Ian King <colin.king@canonical.com> | 2017-12-15 05:53:43 -0500 |
---|---|---|
committer | Alex Deucher <alexander.deucher@amd.com> | 2017-12-15 17:15:09 -0500 |
commit | a89ff457d6f6200cd58db568c0138988bb60579c (patch) | |
tree | 553c0a7d72d8603073a64451e7b01bf1c5f26a6b /drivers/gpu/drm/amd | |
parent | a7ea6548a5d3e458cd0ad0c41cea04eee64f1fb8 (diff) |
drm/amd/display: remove redundant null check of array 'data'
The null check on aconnector->base.edid_blob_ptr->data is redundant
since data is an array and can never be null. Remove it.
Detected by CoverityScan, CID#1460369 ("Array compared against 0")
Acked-by: Christian König <christian.koenig@amd.com>
Reviewed-by: Harry Wentland <harry.wentland@amd.com>
Signed-off-by: Colin Ian King <colin.king@canonical.com>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
Diffstat (limited to 'drivers/gpu/drm/amd')
-rw-r--r-- | drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c b/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c index 8e7621931b92..1cf51bcbbb4a 100644 --- a/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c +++ b/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c | |||
@@ -2725,8 +2725,7 @@ static void create_eml_sink(struct amdgpu_dm_connector *aconnector) | |||
2725 | }; | 2725 | }; |
2726 | struct edid *edid; | 2726 | struct edid *edid; |
2727 | 2727 | ||
2728 | if (!aconnector->base.edid_blob_ptr || | 2728 | if (!aconnector->base.edid_blob_ptr) { |
2729 | !aconnector->base.edid_blob_ptr->data) { | ||
2730 | DRM_ERROR("No EDID firmware found on connector: %s ,forcing to OFF!\n", | 2729 | DRM_ERROR("No EDID firmware found on connector: %s ,forcing to OFF!\n", |
2731 | aconnector->base.name); | 2730 | aconnector->base.name); |
2732 | 2731 | ||