diff options
author | Christian König <christian.koenig@amd.com> | 2016-05-18 07:09:47 -0400 |
---|---|---|
committer | Alex Deucher <alexander.deucher@amd.com> | 2016-07-07 14:50:52 -0400 |
commit | 1ab0d211f3b81464f82caccd58f3f8204ad72c97 (patch) | |
tree | 688863038236b293b670733b9ed2c11f2fbd2c1f /drivers/gpu/drm/amd/amdgpu/amdgpu_job.c | |
parent | 20df080da27e368eea04e5d69d4c7292f8425281 (diff) |
drm/amdgpu: fix coding style in amdgpu_job_free
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>
Diffstat (limited to 'drivers/gpu/drm/amd/amdgpu/amdgpu_job.c')
-rw-r--r-- | drivers/gpu/drm/amd/amdgpu/amdgpu_job.c | 7 |
1 files changed, 4 insertions, 3 deletions
diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_job.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_job.c index 001030b6822b..f0fa48511f41 100644 --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_job.c +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_job.c | |||
@@ -88,13 +88,14 @@ int amdgpu_job_alloc_with_ib(struct amdgpu_device *adev, unsigned size, | |||
88 | 88 | ||
89 | void amdgpu_job_free(struct amdgpu_job *job) | 89 | void amdgpu_job_free(struct amdgpu_job *job) |
90 | { | 90 | { |
91 | unsigned i; | ||
92 | struct fence *f; | 91 | struct fence *f; |
92 | unsigned i; | ||
93 | |||
93 | /* use sched fence if available */ | 94 | /* use sched fence if available */ |
94 | f = (job->base.s_fence)? &job->base.s_fence->base : job->fence; | 95 | f = job->base.s_fence ? &job->base.s_fence->base : job->fence; |
95 | 96 | ||
96 | for (i = 0; i < job->num_ibs; ++i) | 97 | for (i = 0; i < job->num_ibs; ++i) |
97 | amdgpu_sa_bo_free(job->adev, &job->ibs[i].sa_bo, f); | 98 | amdgpu_ib_free(job->adev, &job->ibs[i], f); |
98 | fence_put(job->fence); | 99 | fence_put(job->fence); |
99 | 100 | ||
100 | amdgpu_bo_unref(&job->uf_bo); | 101 | amdgpu_bo_unref(&job->uf_bo); |