From: Mario Limonciello <mario.limoncie...@amd.com> Messages emitted in amdgpu_vpe are currently device scope, but as they are drm driver messages they should be drm scope. Adjust accordingly.
Signed-off-by: Mario Limonciello <mario.limoncie...@amd.com> --- drivers/gpu/drm/amd/amdgpu/amdgpu_vpe.c | 18 ++++++++++-------- 1 file changed, 10 insertions(+), 8 deletions(-) diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_vpe.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_vpe.c index 121ee17b522bd..25dfe47dc2af0 100644 --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_vpe.c +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_vpe.c @@ -263,7 +263,7 @@ int amdgpu_vpe_init_microcode(struct amdgpu_vpe *vpe) return 0; out: - dev_err(adev->dev, "fail to initialize vpe microcode\n"); + drm_err(adev_to_drm(adev), "fail to initialize vpe microcode\n"); release_firmware(adev->vpe.fw); adev->vpe.fw = NULL; return ret; @@ -317,7 +317,8 @@ static int vpe_early_init(struct amdgpu_ip_block *ip_block) vpe_set_ring_funcs(adev); vpe_set_regs(vpe); - dev_info(adev->dev, "VPE: collaborate mode %s", vpe->collaborate_mode ? "true" : "false"); + drm_info(adev_to_drm(adev), "VPE: collaborate mode %s", + vpe->collaborate_mode ? "true" : "false"); return 0; } @@ -347,7 +348,7 @@ static int vpe_common_init(struct amdgpu_vpe *vpe) &adev->vpe.cmdbuf_gpu_addr, (void **)&adev->vpe.cmdbuf_cpu_addr); if (r) { - dev_err(adev->dev, "VPE: failed to allocate cmdbuf bo %d\n", r); + drm_err(adev_to_drm(adev), "VPE: failed to allocate cmdbuf bo %d\n", r); return r; } @@ -635,7 +636,7 @@ static int vpe_ring_preempt_ib(struct amdgpu_ring *ring) if (i >= adev->usec_timeout) { r = -EINVAL; - dev_err(adev->dev, "ring %d failed to be preempted\n", ring->idx); + drm_err(adev_to_drm(adev), "ring %d failed to be preempted\n", ring->idx); } /* deassert IB preemption */ @@ -660,7 +661,7 @@ static int vpe_set_powergating_state(struct amdgpu_ip_block *ip_block, struct amdgpu_vpe *vpe = &adev->vpe; if (!adev->pm.dpm_enabled) - dev_err(adev->dev, "Without PM, cannot support powergating\n"); + drm_err(adev_to_drm(adev), "Without PM, cannot support powergating\n"); dev_dbg(adev->dev, "%s: %s!\n", __func__, (state == AMD_PG_STATE_GATE) ? "GATE":"UNGATE"); @@ -756,7 +757,7 @@ static int vpe_ring_test_ring(struct amdgpu_ring *ring) ret = amdgpu_device_wb_get(adev, &index); if (ret) { - dev_err(adev->dev, "(%d) failed to allocate wb slot\n", ret); + drm_err(adev_to_drm(adev), "(%d) failed to allocate wb slot\n", ret); return ret; } @@ -765,7 +766,8 @@ static int vpe_ring_test_ring(struct amdgpu_ring *ring) ret = amdgpu_ring_alloc(ring, 4); if (ret) { - dev_err(adev->dev, "amdgpu: dma failed to lock ring %d (%d).\n", ring->idx, ret); + drm_err(adev_to_drm(adev), + "amdgpu: dma failed to lock ring %d (%d).\n", ring->idx, ret); goto out; } @@ -800,7 +802,7 @@ static int vpe_ring_test_ib(struct amdgpu_ring *ring, long timeout) ret = amdgpu_device_wb_get(adev, &index); if (ret) { - dev_err(adev->dev, "(%d) failed to allocate wb slot\n", ret); + drm_err(adev_to_drm(adev), "(%d) failed to allocate wb slot\n", ret); return ret; } -- 2.43.0