]> www.infradead.org Git - users/jedix/linux-maple.git/commit
drm/amd/display: Remove a redundant check in authenticated_dp
authorWenjing Liu <wenjing.liu@amd.com>
Fri, 14 Jun 2024 15:01:59 +0000 (11:01 -0400)
committerAlex Deucher <alexander.deucher@amd.com>
Thu, 27 Jun 2024 21:10:38 +0000 (17:10 -0400)
commit4b22869f76563ce1e10858d2ae3305affa8d4a6a
tree1dacea2e0f1bed8f0afb4954f211d54d31acc49e
parent0b8de7a04f7c14abd28bd8b9f3e1e5737a3702e2
drm/amd/display: Remove a redundant check in authenticated_dp

[WHY]
mod_hdcp_execute_and_set returns (*status == MOD_HDCP_STATUS_SUCCESS).
When it return 0, it is guaranteed that status == MOD_HDCP_STATUS_SUCCESS
will be evaluated as false. Since now we are using goto out already, all 3
if (status == MOD_HDCP_STATUS_SUCCESS) clauses are guaranteed to enter.
Therefore we are removing the if statements due to redundancy.

Reviewed-by: Rodrigo Siqueira <rodrigo.siqueira@amd.com>
Signed-off-by: Alex Hung <alex.hung@amd.com>
Signed-off-by: Wenjing Liu <wenjing.liu@amd.com>
Tested-by: Daniel Wheeler <daniel.wheeler@amd.com>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
drivers/gpu/drm/amd/display/modules/hdcp/hdcp1_execution.c