Skip to content

Commit e4c4139

Browse files
Alex Hungalexdeucher
authored andcommitted
Revert "drm/amd/display: Return drm_connector from find_first_crtc_matching_connector"
This reverts commit ca2b37a. [WHY & HOW] The writeback series cause a regression in thunderbolt display. Signed-off-by: Alex Hung <alex.hung@amd.com> Reviewed-by: Harry Wentland <harry.wentland@amd.com> Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
1 parent e4bd165 commit e4c4139

3 files changed

Lines changed: 5 additions & 9 deletions

File tree

drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c

Lines changed: 3 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -2679,7 +2679,7 @@ static int dm_suspend(void *handle)
26792679
return 0;
26802680
}
26812681

2682-
struct drm_connector *
2682+
struct amdgpu_dm_connector *
26832683
amdgpu_dm_find_first_crtc_matching_connector(struct drm_atomic_state *state,
26842684
struct drm_crtc *crtc)
26852685
{
@@ -2692,7 +2692,7 @@ amdgpu_dm_find_first_crtc_matching_connector(struct drm_atomic_state *state,
26922692
crtc_from_state = new_con_state->crtc;
26932693

26942694
if (crtc_from_state == crtc)
2695-
return connector;
2695+
return to_amdgpu_dm_connector(connector);
26962696
}
26972697

26982698
return NULL;
@@ -9393,7 +9393,6 @@ static int dm_update_crtc_state(struct amdgpu_display_manager *dm,
93939393
* update changed items
93949394
*/
93959395
struct amdgpu_crtc *acrtc = NULL;
9396-
struct drm_connector *connector = NULL;
93979396
struct amdgpu_dm_connector *aconnector = NULL;
93989397
struct drm_connector_state *drm_new_conn_state = NULL, *drm_old_conn_state = NULL;
93999398
struct dm_connector_state *dm_new_conn_state = NULL, *dm_old_conn_state = NULL;
@@ -9403,8 +9402,7 @@ static int dm_update_crtc_state(struct amdgpu_display_manager *dm,
94039402
dm_old_crtc_state = to_dm_crtc_state(old_crtc_state);
94049403
dm_new_crtc_state = to_dm_crtc_state(new_crtc_state);
94059404
acrtc = to_amdgpu_crtc(crtc);
9406-
connector = amdgpu_dm_find_first_crtc_matching_connector(state, crtc);
9407-
aconnector = to_amdgpu_dm_connector(connector);
9405+
aconnector = amdgpu_dm_find_first_crtc_matching_connector(state, crtc);
94089406

94099407
/* TODO This hack should go away */
94109408
if (aconnector && enable) {

drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.h

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -834,7 +834,7 @@ struct dc_stream_state *
834834
int dm_atomic_get_state(struct drm_atomic_state *state,
835835
struct dm_atomic_state **dm_state);
836836

837-
struct drm_connector *
837+
struct amdgpu_dm_connector *
838838
amdgpu_dm_find_first_crtc_matching_connector(struct drm_atomic_state *state,
839839
struct drm_crtc *crtc);
840840

drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_mst_types.c

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1500,16 +1500,14 @@ int pre_validate_dsc(struct drm_atomic_state *state,
15001500
int ind = find_crtc_index_in_state_by_stream(state, stream);
15011501

15021502
if (ind >= 0) {
1503-
struct drm_connector *connector;
15041503
struct amdgpu_dm_connector *aconnector;
15051504
struct drm_connector_state *drm_new_conn_state;
15061505
struct dm_connector_state *dm_new_conn_state;
15071506
struct dm_crtc_state *dm_old_crtc_state;
15081507

1509-
connector =
1508+
aconnector =
15101509
amdgpu_dm_find_first_crtc_matching_connector(state,
15111510
state->crtcs[ind].ptr);
1512-
aconnector = to_amdgpu_dm_connector(connector);
15131511
drm_new_conn_state =
15141512
drm_atomic_get_new_connector_state(state,
15151513
&aconnector->base);

0 commit comments

Comments
 (0)