diff options
| author | David (Ming Qiang) Wu <David.Wu3@amd.com> | 2024-08-02 14:29:41 -0400 |
|---|---|---|
| committer | Alex Deucher <alexander.deucher@amd.com> | 2024-08-13 12:13:03 -0400 |
| commit | 6a28a072d90e4543b5e07a904e3a6afad0117e26 (patch) | |
| tree | e1744340af2658e0e456297663a5ea4c657cbdf5 /drivers/gpu/drm/amd/amdgpu/amdgpu_uvd.c | |
| parent | a7f670d5d8e77b092404ca8a35bb0f8f89ed3117 (diff) | |
| download | linux-6a28a072d90e4543b5e07a904e3a6afad0117e26.tar.gz linux-6a28a072d90e4543b5e07a904e3a6afad0117e26.tar.bz2 linux-6a28a072d90e4543b5e07a904e3a6afad0117e26.zip | |
drm/amd/amdgpu: cleanup parse_cs callbacks
Because gpu_addr is updated in the calling routine
(amdgpu_cs_patch_ibs()),it is removed in the callback.
Use .patch_cs_in_place instead of .parse_cs for
amdgpu_vce_ring_parse_cs_vm() as there is no need for keeping
a temporary IB, therefore ib->sa_bo is NULL and amdgpu_ib_free()
is removed.
Reviewed-by: Alex Deucher <alexander.deucher@amd.com>
Signed-off-by: David (Ming Qiang) Wu <David.Wu3@amd.com>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
Diffstat (limited to 'drivers/gpu/drm/amd/amdgpu/amdgpu_uvd.c')
| -rw-r--r-- | drivers/gpu/drm/amd/amdgpu/amdgpu_uvd.c | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_uvd.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_uvd.c index 775c09d57222..31fd30dcd593 100644 --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_uvd.c +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_uvd.c @@ -1088,7 +1088,6 @@ int amdgpu_uvd_ring_parse_cs(struct amdgpu_cs_parser *parser, int r; job->vm = NULL; - ib->gpu_addr = amdgpu_sa_bo_gpu_addr(ib->sa_bo); if (ib->length_dw % 16) { DRM_ERROR("UVD IB length (%d) not 16 dwords aligned!\n", |