diff options
| author | Colin Ian King <colin.king@canonical.com> | 2017-12-15 10:53:43 +0000 |
|---|---|---|
| committer | Alex Deucher <alexander.deucher@amd.com> | 2017-12-15 17:15:09 -0500 |
| commit | a89ff457d6f6200cd58db568c0138988bb60579c (patch) | |
| tree | 553c0a7d72d8603073a64451e7b01bf1c5f26a6b /drivers/gpu/drm/amd/display | |
| parent | a7ea6548a5d3e458cd0ad0c41cea04eee64f1fb8 (diff) | |
| download | lwn-a89ff457d6f6200cd58db568c0138988bb60579c.tar.gz lwn-a89ff457d6f6200cd58db568c0138988bb60579c.zip | |
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/display')
| -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) }; struct edid *edid; - if (!aconnector->base.edid_blob_ptr || - !aconnector->base.edid_blob_ptr->data) { + if (!aconnector->base.edid_blob_ptr) { DRM_ERROR("No EDID firmware found on connector: %s ,forcing to OFF!\n", aconnector->base.name); |