projects
/
linux.git
/ commitdiff
commit
grep
author
committer
pickaxe
?
search:
re
summary
|
shortlog
|
log
|
commit
| commitdiff |
tree
raw
|
patch
| inline |
side by side
(parent:
fb0a84e
)
drm/mst: delete unnecessary case in drm_dp_add_payload_part2()
author
Wayne Lin
<Wayne.Lin@amd.com>
Mon, 7 Aug 2023 02:56:37 +0000
(10:56 +0800)
committer
Lyude Paul
<lyude@redhat.com>
Fri, 8 Sep 2023 21:00:56 +0000
(17:00 -0400)
[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
drivers/gpu/drm/display/drm_dp_mst_topology.c
patch
|
blob
|
history
diff --git
a/drivers/gpu/drm/display/drm_dp_mst_topology.c
b/drivers/gpu/drm/display/drm_dp_mst_topology.c
index ed96cfcfa304013158dbaa3629529d32ed07cc7d..4d80426757abc3bfce61a28a7d60e86c02094d32 100644
(file)
--- a/
drivers/gpu/drm/display/drm_dp_mst_topology.c
+++ b/
drivers/gpu/drm/display/drm_dp_mst_topology.c
@@
-3411,12
+3411,8
@@
int drm_dp_add_payload_part2(struct drm_dp_mst_topology_mgr *mgr,
ret = drm_dp_create_payload_step2(mgr, payload);
if (ret < 0) {
- if (!payload->delete)
- drm_err(mgr->dev, "Step 2 of creating MST payload for %p failed: %d\n",
- payload->port, ret);
- else
- drm_dbg_kms(mgr->dev, "Step 2 of removing MST payload for %p failed: %d\n",
- payload->port, ret);
+ drm_err(mgr->dev, "Step 2 of creating MST payload for %p failed: %d\n",
+ payload->port, ret);
}
return ret;