Commit 620fd73e authored by Dan Carpenter's avatar Dan Carpenter Committed by Alex Deucher

drm/amd/display: small cleanup in destruct()

Static analysis tools get annoyed that we don't indent this if
statement.  Actually, the if statement isn't required because kfree()
can handle NULL pointers just fine.  The DCE110STRENC_FROM_STRENC()
macro is a wrapper around container_of() but it's basically a no-op or a
cast.  Anyway, it's not really appropriate here so it should be removed
as well.
Acked-by: default avatarChristian König <christian.koenig@amd.com>
Signed-off-by: default avatarDan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: default avatarAlex Deucher <alexander.deucher@amd.com>
parent 26c860d5
...@@ -725,10 +725,8 @@ static void destruct(struct dcn10_resource_pool *pool) ...@@ -725,10 +725,8 @@ static void destruct(struct dcn10_resource_pool *pool)
} }
} }
for (i = 0; i < pool->base.stream_enc_count; i++) { for (i = 0; i < pool->base.stream_enc_count; i++)
if (pool->base.stream_enc[i] != NULL) kfree(pool->base.stream_enc[i]);
kfree(DCE110STRENC_FROM_STRENC(pool->base.stream_enc[i]));
}
for (i = 0; i < pool->base.audio_count; i++) { for (i = 0; i < pool->base.audio_count; i++) {
if (pool->base.audios[i]) if (pool->base.audios[i])
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment