]> git.itanic.dy.fi Git - linux-stable/commitdiff
drm/amdgpu: use adev_to_drm for consistency when accessing drm_device
authorGuchun Chen <guchun.chen@amd.com>
Fri, 8 Oct 2021 17:21:45 +0000 (13:21 -0400)
committerAlex Deucher <alexander.deucher@amd.com>
Fri, 8 Oct 2021 17:22:13 +0000 (13:22 -0400)
adev_to_drm is used everywhere, so improve recent changes
when accessing drm_device pointer from amdgpu_device.

Signed-off-by: Guchun Chen <guchun.chen@amd.com>
Acked-by: Christian König <christian.koenig@amd.com>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
15 files changed:
drivers/gpu/drm/amd/amdgpu/amdgpu_device.c
drivers/gpu/drm/amd/amdgpu/amdgpu_fence.c
drivers/gpu/drm/amd/amdgpu/amdgpu_gart.c
drivers/gpu/drm/amd/amdgpu/amdgpu_gmc.c
drivers/gpu/drm/amd/amdgpu/amdgpu_job.c
drivers/gpu/drm/amd/amdgpu/amdgpu_psp.c
drivers/gpu/drm/amd/amdgpu/amdgpu_uvd.c
drivers/gpu/drm/amd/amdgpu/amdgpu_vce.c
drivers/gpu/drm/amd/amdgpu/amdgpu_vcn.c
drivers/gpu/drm/amd/amdgpu/amdgpu_vm.c
drivers/gpu/drm/amd/amdgpu/psp_v11_0.c
drivers/gpu/drm/amd/amdgpu/vce_v4_0.c
drivers/gpu/drm/amd/amdgpu/vcn_v2_0.c
drivers/gpu/drm/amd/amdgpu/vcn_v2_5.c
drivers/gpu/drm/amd/amdgpu/vcn_v3_0.c

index 5ea36c1951f36745f2242f2f55d1535249ea8cc6..239e7117485500df8030004218278d7013f48dac 100644 (file)
@@ -306,7 +306,7 @@ void amdgpu_device_mm_access(struct amdgpu_device *adev, loff_t pos,
        uint64_t last;
        int idx;
 
-       if (!drm_dev_enter(&adev->ddev, &idx))
+       if (!drm_dev_enter(adev_to_drm(adev), &idx))
                return;
 
        BUG_ON(!IS_ALIGNED(pos, 4) || !IS_ALIGNED(size, 4));
index a25e192e8a3feba5d8475b9197878327e809bcb4..3b7e86ea71672b9fff2ab6e455449905d99c7d37 100644 (file)
@@ -550,7 +550,7 @@ void amdgpu_fence_driver_hw_fini(struct amdgpu_device *adev)
                        drm_sched_stop(&ring->sched, NULL);
 
                /* You can't wait for HW to signal if it's gone */
-               if (!drm_dev_is_unplugged(&adev->ddev))
+               if (!drm_dev_is_unplugged(adev_to_drm(adev)))
                        r = amdgpu_fence_wait_empty(ring);
                else
                        r = -ENODEV;
index d7e4f4660acf230a4d8d600e3d4ffbbe47da977e..d3e4203f62174edc4144b50fb93d609c6ef91acb 100644 (file)
@@ -238,7 +238,7 @@ int amdgpu_gart_unbind(struct amdgpu_device *adev, uint64_t offset,
                return -EINVAL;
        }
 
-       if (!drm_dev_enter(&adev->ddev, &idx))
+       if (!drm_dev_enter(adev_to_drm(adev), &idx))
                return 0;
 
        t = offset / AMDGPU_GPU_PAGE_SIZE;
@@ -289,7 +289,7 @@ int amdgpu_gart_map(struct amdgpu_device *adev, uint64_t offset,
                return -EINVAL;
        }
 
-       if (!drm_dev_enter(&adev->ddev, &idx))
+       if (!drm_dev_enter(adev_to_drm(adev), &idx))
                return 0;
 
        t = offset / AMDGPU_GPU_PAGE_SIZE;
index a0dec7f211f05d1d7f787f713fe1bfbd44a77201..08478fce00f2d1a40fabefe09d44500e2f5296b5 100644 (file)
@@ -745,7 +745,7 @@ void amdgpu_gmc_init_pdb0(struct amdgpu_device *adev)
        u64 gart_ptb_gpu_pa = amdgpu_gmc_vram_pa(adev, adev->gart.bo);
        int idx;
 
-       if (!drm_dev_enter(&adev->ddev, &idx))
+       if (!drm_dev_enter(adev_to_drm(adev), &idx))
                return;
 
        flags |= AMDGPU_PTE_VALID | AMDGPU_PTE_READABLE;
index 4e30a09bc887978949196a584e1e8653eab7a373..bfc47bea23db2972e358dd449f683106d6255cde 100644 (file)
@@ -38,7 +38,7 @@ static enum drm_gpu_sched_stat amdgpu_job_timedout(struct drm_sched_job *s_job)
        struct amdgpu_device *adev = ring->adev;
        int idx;
 
-       if (!drm_dev_enter(&adev->ddev, &idx)) {
+       if (!drm_dev_enter(adev_to_drm(adev), &idx)) {
                DRM_INFO("%s - device unplugged skipping recovery on scheduler:%s",
                         __func__, s_job->sched->name);
 
index fd6e38ad493c284ea213e559f1f1d919fd1b59ad..6b39e6c02dd8e896638657d40b5be11d0f5b2bab 100644 (file)
@@ -440,7 +440,7 @@ psp_cmd_submit_buf(struct psp_context *psp,
        if (psp->adev->no_hw_access)
                return 0;
 
-       if (!drm_dev_enter(&psp->adev->ddev, &idx))
+       if (!drm_dev_enter(adev_to_drm(psp->adev), &idx))
                return 0;
 
        memset(psp->cmd_buf_mem, 0, PSP_CMD_BUFFER_SIZE);
@@ -3272,7 +3272,7 @@ void psp_copy_fw(struct psp_context *psp, uint8_t *start_addr, uint32_t bin_size
 {
        int idx;
 
-       if (!drm_dev_enter(&psp->adev->ddev, &idx))
+       if (!drm_dev_enter(adev_to_drm(psp->adev), &idx))
                return;
 
        memset(psp->fw_pri_buf, 0, PSP_1_MEG);
index 8a26459bd80bced141448c90b9f12872fa3e7f08..6f8de11a17f12e3cde1f2a16fa25ecae2b914bbf 100644 (file)
@@ -454,7 +454,7 @@ int amdgpu_uvd_suspend(struct amdgpu_device *adev)
                if (!adev->uvd.inst[j].saved_bo)
                        return -ENOMEM;
 
-               if (drm_dev_enter(&adev->ddev, &idx)) {
+               if (drm_dev_enter(adev_to_drm(adev), &idx)) {
                        /* re-write 0 since err_event_athub will corrupt VCPU buffer */
                        if (in_ras_intr)
                                memset(adev->uvd.inst[j].saved_bo, 0, size);
@@ -487,7 +487,7 @@ int amdgpu_uvd_resume(struct amdgpu_device *adev)
                ptr = adev->uvd.inst[i].cpu_addr;
 
                if (adev->uvd.inst[i].saved_bo != NULL) {
-                       if (drm_dev_enter(&adev->ddev, &idx)) {
+                       if (drm_dev_enter(adev_to_drm(adev), &idx)) {
                                memcpy_toio(ptr, adev->uvd.inst[i].saved_bo, size);
                                drm_dev_exit(idx);
                        }
@@ -500,7 +500,7 @@ int amdgpu_uvd_resume(struct amdgpu_device *adev)
                        hdr = (const struct common_firmware_header *)adev->uvd.fw->data;
                        if (adev->firmware.load_type != AMDGPU_FW_LOAD_PSP) {
                                offset = le32_to_cpu(hdr->ucode_array_offset_bytes);
-                               if (drm_dev_enter(&adev->ddev, &idx)) {
+                               if (drm_dev_enter(adev_to_drm(adev), &idx)) {
                                        memcpy_toio(adev->uvd.inst[i].cpu_addr, adev->uvd.fw->data + offset,
                                                    le32_to_cpu(hdr->ucode_size_bytes));
                                        drm_dev_exit(idx);
index caa4d3420e00641ceba20c366e55f5a7862254fd..688bef1649b52e8700f2b9cb2d2f07612063910c 100644 (file)
@@ -313,7 +313,7 @@ int amdgpu_vce_resume(struct amdgpu_device *adev)
        hdr = (const struct common_firmware_header *)adev->vce.fw->data;
        offset = le32_to_cpu(hdr->ucode_array_offset_bytes);
 
-       if (drm_dev_enter(&adev->ddev, &idx)) {
+       if (drm_dev_enter(adev_to_drm(adev), &idx)) {
                memcpy_toio(cpu_addr, adev->vce.fw->data + offset,
                            adev->vce.fw->size - offset);
                drm_dev_exit(idx);
index 4d3f2386ef531887c7a3cc44169163a3102ae225..c7d316850570b681324caadd304e0f7165aa2e83 100644 (file)
@@ -325,7 +325,7 @@ int amdgpu_vcn_suspend(struct amdgpu_device *adev)
                if (!adev->vcn.inst[i].saved_bo)
                        return -ENOMEM;
 
-               if (drm_dev_enter(&adev->ddev, &idx)) {
+               if (drm_dev_enter(adev_to_drm(adev), &idx)) {
                        memcpy_fromio(adev->vcn.inst[i].saved_bo, ptr, size);
                        drm_dev_exit(idx);
                }
@@ -349,7 +349,7 @@ int amdgpu_vcn_resume(struct amdgpu_device *adev)
                ptr = adev->vcn.inst[i].cpu_addr;
 
                if (adev->vcn.inst[i].saved_bo != NULL) {
-                       if (drm_dev_enter(&adev->ddev, &idx)) {
+                       if (drm_dev_enter(adev_to_drm(adev), &idx)) {
                                memcpy_toio(ptr, adev->vcn.inst[i].saved_bo, size);
                                drm_dev_exit(idx);
                        }
@@ -362,7 +362,7 @@ int amdgpu_vcn_resume(struct amdgpu_device *adev)
                        hdr = (const struct common_firmware_header *)adev->vcn.fw->data;
                        if (adev->firmware.load_type != AMDGPU_FW_LOAD_PSP) {
                                offset = le32_to_cpu(hdr->ucode_array_offset_bytes);
-                               if (drm_dev_enter(&adev->ddev, &idx)) {
+                               if (drm_dev_enter(adev_to_drm(adev), &idx)) {
                                        memcpy_toio(adev->vcn.inst[i].cpu_addr, adev->vcn.fw->data + offset,
                                                    le32_to_cpu(hdr->ucode_size_bytes));
                                        drm_dev_exit(idx);
index a1ddf74bbdbafb221236e726cf123159273a3eea..0e7dc23f78e7ff767ece9a031c39a0fe39e54a95 100644 (file)
@@ -845,7 +845,7 @@ static int amdgpu_vm_clear_bo(struct amdgpu_device *adev,
                        return r;
        }
 
-       if (!drm_dev_enter(&adev->ddev, &idx))
+       if (!drm_dev_enter(adev_to_drm(adev), &idx))
                return -ENODEV;
 
        r = vm->update_funcs->map_table(vmbo);
@@ -1395,7 +1395,7 @@ int amdgpu_vm_update_pdes(struct amdgpu_device *adev,
        if (list_empty(&vm->relocated))
                return 0;
 
-       if (!drm_dev_enter(&adev->ddev, &idx))
+       if (!drm_dev_enter(adev_to_drm(adev), &idx))
                return -ENODEV;
 
        memset(&params, 0, sizeof(params));
@@ -1718,7 +1718,7 @@ int amdgpu_vm_bo_update_mapping(struct amdgpu_device *adev,
        enum amdgpu_sync_mode sync_mode;
        int r, idx;
 
-       if (!drm_dev_enter(&adev->ddev, &idx))
+       if (!drm_dev_enter(adev_to_drm(adev), &idx))
                return -ENODEV;
 
        memset(&params, 0, sizeof(params));
index aaf200ec982b9b8fe402b091c12fd65bc8eb8068..21a325ea49cb304932b24e58ebcd2cff3596493d 100644 (file)
@@ -698,7 +698,7 @@ static int psp_v11_0_memory_training(struct psp_context *psp, uint32_t ops)
                        return -ENOMEM;
                }
 
-               if (drm_dev_enter(&adev->ddev, &idx)) {
+               if (drm_dev_enter(adev_to_drm(adev), &idx)) {
                        memcpy_fromio(buf, adev->mman.aper_base_kaddr, sz);
                        ret = psp_v11_0_memory_training_send_msg(psp, PSP_BL__DRAM_LONG_TRAIN);
                        if (ret) {
index 226b79254db851e6faa0c9e1ce1fdb64c09769c6..d1fc4e0b8265d01d7ae91ba106dca5b06a88ae45 100644 (file)
@@ -565,7 +565,7 @@ static int vce_v4_0_suspend(void *handle)
        if (adev->vce.vcpu_bo == NULL)
                return 0;
 
-       if (drm_dev_enter(&adev->ddev, &idx)) {
+       if (drm_dev_enter(adev_to_drm(adev), &idx)) {
                if (adev->firmware.load_type == AMDGPU_FW_LOAD_PSP) {
                        unsigned size = amdgpu_bo_size(adev->vce.vcpu_bo);
                        void *ptr = adev->vce.cpu_addr;
@@ -615,7 +615,7 @@ static int vce_v4_0_resume(void *handle)
 
        if (adev->firmware.load_type == AMDGPU_FW_LOAD_PSP) {
 
-               if (drm_dev_enter(&adev->ddev, &idx)) {
+               if (drm_dev_enter(adev_to_drm(adev), &idx)) {
                        unsigned size = amdgpu_bo_size(adev->vce.vcpu_bo);
                        void *ptr = adev->vce.cpu_addr;
 
index 43f46ab07ddaa12e5ef991a682c24634b757fb19..091d8c0f680168f357d60848e5456d74f6f4b626 100644 (file)
@@ -198,7 +198,7 @@ static int vcn_v2_0_sw_fini(void *handle)
        struct amdgpu_device *adev = (struct amdgpu_device *)handle;
        volatile struct amdgpu_fw_shared *fw_shared = adev->vcn.inst->fw_shared_cpu_addr;
 
-       if (drm_dev_enter(&adev->ddev, &idx)) {
+       if (drm_dev_enter(adev_to_drm(adev), &idx)) {
                fw_shared->present_flag_0 = 0;
                drm_dev_exit(idx);
        }
index f4a7a30ecebd2399171138ac4366dd16dc8bad63..59f469bab005eaa24e008ac83da92eaf65ec2bac 100644 (file)
@@ -240,7 +240,7 @@ static int vcn_v2_5_sw_fini(void *handle)
        struct amdgpu_device *adev = (struct amdgpu_device *)handle;
        volatile struct amdgpu_fw_shared *fw_shared;
 
-       if (drm_dev_enter(&adev->ddev, &idx)) {
+       if (drm_dev_enter(adev_to_drm(adev), &idx)) {
                for (i = 0; i < adev->vcn.num_vcn_inst; i++) {
                        if (adev->vcn.harvest_config & (1 << i))
                                continue;
index 135a9c553af8a37ca3688738d6a3f9573e9d6fe1..dbfd929846553967c16723af3344551ffdbd37bf 100644 (file)
@@ -263,7 +263,7 @@ static int vcn_v3_0_sw_fini(void *handle)
        struct amdgpu_device *adev = (struct amdgpu_device *)handle;
        int i, r, idx;
 
-       if (drm_dev_enter(&adev->ddev, &idx)) {
+       if (drm_dev_enter(adev_to_drm(adev), &idx)) {
                for (i = 0; i < adev->vcn.num_vcn_inst; i++) {
                        volatile struct amdgpu_fw_shared *fw_shared;