From: Lu Yao <[email protected]>

The else-if branch in dm_gpureset_toggle_interrupts() condition was
incorrectly checking plane_count != 0, which is the same as the
preceding if-branch and can never be true in the else path.

Fixes: 8382cd234981 ("drm/amd/display: consolidate DCN vblank/flip handling 
onto vupdate_no_lock")
Signed-off-by: Lu Yao <[email protected]>
---
 drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c 
b/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c
index 9c564cd5edee..a11c64277d1d 100644
--- a/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c
+++ b/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c
@@ -3353,7 +3353,7 @@ static void dm_gpureset_toggle_interrupts(struct 
amdgpu_device *adev,
                        if (!dc_interrupt_set(adev->dm.dc, irq_source, enable))
                                drm_warn(adev_to_drm(adev), "Failed to %sable 
vblank interrupt\n", enable ? "en" : "dis");
 
-               } else if (acrtc && state->stream_status[i].plane_count != 0) {
+               } else if (acrtc && state->stream_status[i].plane_count == 0) {
                        /* DCN only needs to toggle VUPDATE_NO_LOCK */
                        rc = amdgpu_dm_crtc_set_vupdate_irq(&acrtc->base, 
enable);
                        if (rc)
-- 
2.25.1

Reply via email to