drm/amd/display: Remove redundant condition.
authorAndrey Grodzovsky <Andrey.Grodzovsky@amd.com>
Thu, 18 May 2017 17:35:22 +0000 (13:35 -0400)
committerAlex Deucher <alexander.deucher@amd.com>
Tue, 26 Sep 2017 22:07:25 +0000 (18:07 -0400)
You cannot have modeset and flip in the same call for
same CRTC, in such case it will be set mode and set plane,
not a flip.

Signed-off-by: Andrey Grodzovsky <Andrey.Grodzovsky@amd.com>
Reviewed-by: Andrey Grodzovsky <Andrey.Grodzovsky@amd.com>
Acked-by: Harry Wentland <Harry.Wentland@amd.com>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm_types.c

index a5b7c05..b3a4605 100644 (file)
@@ -3115,15 +3115,9 @@ int amdgpu_dm_atomic_check(struct drm_device *dev,
 
                        action = get_dm_commit_action(crtc->state);
 
-                       /* Surfaces are created under two scenarios:
-                        * 1. This commit is not a page flip.
-                        * 2. This commit is a page flip, and streams are created.
-                        */
                        crtc_state = drm_atomic_get_crtc_state(state, crtc);
                        pflip_needed = !state->allow_modeset;
-                       if (!pflip_needed ||
-                               action == DM_COMMIT_ACTION_DPMS_ON ||
-                               action == DM_COMMIT_ACTION_SET) {
+                       if (!pflip_needed) {
                                struct dc_surface *surface;
 
                                list_for_each_entry(connector,