Skip to content

Commit ae4d231

Browse files
Wayne LinLyude
authored andcommitted
drm/mst: delete unnecessary case in drm_dp_add_payload_part2()
[Why] There is no need to consider payload->delete case since we won't call drm_dp_add_payload_part2() to create a payload when we're about to remove it. [How] Delete unnecessary case to simplify the code. Signed-off-by: Wayne Lin <Wayne.Lin@amd.com> Signed-off-by: Lyude Paul <lyude@redhat.com> Link: https://patchwork.freedesktop.org/patch/msgid/20230807025639.1612361-2-Wayne.Lin@amd.com
1 parent fb0a84e commit ae4d231

1 file changed

Lines changed: 2 additions & 6 deletions

File tree

drivers/gpu/drm/display/drm_dp_mst_topology.c

Lines changed: 2 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -3411,12 +3411,8 @@ int drm_dp_add_payload_part2(struct drm_dp_mst_topology_mgr *mgr,
34113411

34123412
ret = drm_dp_create_payload_step2(mgr, payload);
34133413
if (ret < 0) {
3414-
if (!payload->delete)
3415-
drm_err(mgr->dev, "Step 2 of creating MST payload for %p failed: %d\n",
3416-
payload->port, ret);
3417-
else
3418-
drm_dbg_kms(mgr->dev, "Step 2 of removing MST payload for %p failed: %d\n",
3419-
payload->port, ret);
3414+
drm_err(mgr->dev, "Step 2 of creating MST payload for %p failed: %d\n",
3415+
payload->port, ret);
34203416
}
34213417

34223418
return ret;

0 commit comments

Comments
 (0)