drm/amdgpu: fix coding style in amdgpu_job_free
authorChristian König <christian.koenig@amd.com>
Wed, 18 May 2016 11:09:47 +0000 (13:09 +0200)
committerAlex Deucher <alexander.deucher@amd.com>
Thu, 7 Jul 2016 18:50:52 +0000 (14:50 -0400)
Ther should be a new line between code and decleration.
Also use amdgpu_ib_free() instead of releasing the member manually.

Signed-off-by: Christian König <christian.koenig@amd.com>
Reviewed-by: Monk.Liu <monk.liu@amd.com>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
drivers/gpu/drm/amd/amdgpu/amdgpu_job.c

index 001030b6822ba0a90acc74a507c7e1c55b847ccf..f0fa48511f410ba62753326c35e6a246fb615b10 100644 (file)
@@ -88,13 +88,14 @@ int amdgpu_job_alloc_with_ib(struct amdgpu_device *adev, unsigned size,
 
 void amdgpu_job_free(struct amdgpu_job *job)
 {
-       unsigned i;
        struct fence *f;
+       unsigned i;
+
        /* use sched fence if available */
-       f = (job->base.s_fence)? &job->base.s_fence->base : job->fence;
+       f = job->base.s_fence ? &job->base.s_fence->base : job->fence;
 
        for (i = 0; i < job->num_ibs; ++i)
-               amdgpu_sa_bo_free(job->adev, &job->ibs[i].sa_bo, f);
+               amdgpu_ib_free(job->adev, &job->ibs[i], f);
        fence_put(job->fence);
 
        amdgpu_bo_unref(&job->uf_bo);