From: Jerry Zuo <Jerry.Zuo@amd.com>
Date: Thu, 28 Sep 2017 16:20:30 -0400
Subject: drm/amd/display: Do not access fbdev helper members
Git-commit: 4bd29c34988be0c24759d5d58df424f861a8bf24
Patch-mainline: v4.15-rc1
References: FATE#326289 FATE#326079 FATE#326049 FATE#322398 FATE#326166
Original code is no longer needed and tested without loop.
Signed-off-by: Harry Wentland <harry.wentland@amd.com>
Signed-off-by: Jerry Zuo <Jerry.Zuo@amd.com>
Reviewed-by: Harry Wentland <Harry.Wentland@amd.com>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
Acked-by: Petr Tesarik <ptesarik@suse.com>
---
drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_mst_types.c | 13 ------------
1 file changed, 1 insertion(+), 12 deletions(-)
--- a/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_mst_types.c
+++ b/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_mst_types.c
@@ -397,21 +397,10 @@ static void dm_dp_mst_register_connector
{
struct drm_device *dev = connector->dev;
struct amdgpu_device *adev = dev->dev_private;
- int i;
drm_modeset_lock_all(dev);
- if (adev->mode_info.rfbdev) {
- /*Do not add if already registered in past*/
- for (i = 0; i < adev->mode_info.rfbdev->helper.connector_count; i++) {
- if (adev->mode_info.rfbdev->helper.connector_info[i]->connector
- == connector) {
- drm_modeset_unlock_all(dev);
- return;
- }
- }
-
+ if (adev->mode_info.rfbdev)
drm_fb_helper_add_one_connector(&adev->mode_info.rfbdev->helper, connector);
- }
else
DRM_ERROR("adev->mode_info.rfbdev is NULL\n");