Looks correct and consistent with the other DCE drivers.

Reviewed-by: Tom St Denis <[email protected]>


________________________________
From: amd-gfx <[email protected]> on behalf of Alex Deucher 
<[email protected]>
Sent: Monday, September 19, 2016 12:15
To: [email protected]
Cc: Deucher, Alexander
Subject: [PATCH] drm/amdgpu/dce6: fix off by one in interrupt setup

Reported-by: Christian Inci <[email protected]>
Signed-off-by: Alex Deucher <[email protected]>
---
 drivers/gpu/drm/amd/amdgpu/dce_v6_0.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/drivers/gpu/drm/amd/amdgpu/dce_v6_0.c 
b/drivers/gpu/drm/amd/amdgpu/dce_v6_0.c
index fb6dafd..c535f43 100644
--- a/drivers/gpu/drm/amd/amdgpu/dce_v6_0.c
+++ b/drivers/gpu/drm/amd/amdgpu/dce_v6_0.c
@@ -185,7 +185,7 @@ static void dce_v6_0_pageflip_interrupt_init(struct 
amdgpu_device *adev)
         unsigned i;

         /* Enable pflip interrupts */
-       for (i = 0; i <= adev->mode_info.num_crtc; i++)
+       for (i = 0; i < adev->mode_info.num_crtc; i++)
                 amdgpu_irq_get(adev, &adev->pageflip_irq, i);
 }

@@ -194,7 +194,7 @@ static void dce_v6_0_pageflip_interrupt_fini(struct 
amdgpu_device *adev)
         unsigned i;

         /* Disable pflip interrupts */
-       for (i = 0; i <= adev->mode_info.num_crtc; i++)
+       for (i = 0; i < adev->mode_info.num_crtc; i++)
                 amdgpu_irq_put(adev, &adev->pageflip_irq, i);
 }

--
2.5.5

_______________________________________________
amd-gfx mailing list
[email protected]
https://lists.freedesktop.org/mailman/listinfo/amd-gfx
amd-gfx Info Page - 
lists.freedesktop.org<https://lists.freedesktop.org/mailman/listinfo/amd-gfx>
lists.freedesktop.org
To see the collection of prior postings to the list, visit the amd-gfx 
Archives. Using amd-gfx: To post a message to all the list members, send email 
...



_______________________________________________
amd-gfx mailing list
[email protected]
https://lists.freedesktop.org/mailman/listinfo/amd-gfx

Reply via email to