Am 17.05.2018 um 20:03 schrieb Alex Deucher:
Take the encode rings into account in the idle work handler.

Signed-off-by: Alex Deucher <alexander.deuc...@amd.com>
---
  drivers/gpu/drm/amd/amdgpu/amdgpu_vcn.c | 5 +++++
  1 file changed, 5 insertions(+)

diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_vcn.c 
b/drivers/gpu/drm/amd/amdgpu/amdgpu_vcn.c
index e5d234cf804f..60468385e6b4 100644
--- a/drivers/gpu/drm/amd/amdgpu/amdgpu_vcn.c
+++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_vcn.c
@@ -205,6 +205,11 @@ static void amdgpu_vcn_idle_work_handler(struct 
work_struct *work)
        struct amdgpu_device *adev =
                container_of(work, struct amdgpu_device, vcn.idle_work.work);
        unsigned fences = amdgpu_fence_count_emitted(&adev->vcn.ring_dec);
+       unsigned i;
+
+       for (i = 0; i < adev->vcn.num_enc_rings; ++i) {
+               fences += amdgpu_fence_count_emitted(&adev->vcn.ring_enc[i]);
+       }

"{" and "}" should be dropped here if I'm not completely mistaken.

But either way Reviewed-by: Christian König <christian.koe...@amd.com> for the whole series.

Christian.

if (fences == 0) {
                if (adev->pm.dpm_enabled) {

_______________________________________________
amd-gfx mailing list
amd-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/amd-gfx

Reply via email to