From b05e9e4c9d5d0d32357468b64d72101b0fb2b650 Mon Sep 17 00:00:00 2001 From: =?utf8?q?Christian=20K=C3=B6nig?= Date: Fri, 19 Apr 2013 16:14:19 +0200 Subject: [PATCH] drm/radeon: raise UVD clocks on init v3 MIME-Version: 1.0 Content-Type: text/plain; charset=utf8 Content-Transfer-Encoding: 8bit v2: not only raise the clocks on VCPU boot, but also on IB test. v3: agd5f: fix r600_uvd_init return value. fixes: https://bugs.freedesktop.org/show_bug.cgi?id=63730 Signed-off-by: Christian König Signed-off-by: Alex Deucher --- drivers/gpu/drm/radeon/r600.c | 32 ++++++++++++++++++++++++-------- 1 file changed, 24 insertions(+), 8 deletions(-) diff --git a/drivers/gpu/drm/radeon/r600.c b/drivers/gpu/drm/radeon/r600.c index 5fe9e74..709f3e4 100644 --- a/drivers/gpu/drm/radeon/r600.c +++ b/drivers/gpu/drm/radeon/r600.c @@ -2645,6 +2645,9 @@ int r600_uvd_init(struct radeon_device *rdev) { int i, j, r; + /* raise clocks while booting up the VCPU */ + radeon_set_uvd_clocks(rdev, 53300, 40000); + /* disable clock gating */ WREG32(UVD_CGC_GATE, 0); @@ -2715,19 +2718,24 @@ int r600_uvd_init(struct radeon_device *rdev) mdelay(10); r = -1; } + if (r) { DRM_ERROR("UVD not responding, giving up!!!\n"); + radeon_set_uvd_clocks(rdev, 0, 0); return r; } + /* enable interupt */ WREG32_P(UVD_MASTINT_EN, 3<<1, ~(3 << 1)); r = r600_uvd_rbc_start(rdev); - if (r) - return r; + if (!r) + DRM_INFO("UVD initialized successfully.\n"); - DRM_INFO("UVD initialized successfully.\n"); - return 0; + /* lower clocks again */ + radeon_set_uvd_clocks(rdev, 0, 0); + + return r; } /* @@ -3566,28 +3574,36 @@ int r600_dma_ib_test(struct radeon_device *rdev, struct radeon_ring *ring) int r600_uvd_ib_test(struct radeon_device *rdev, struct radeon_ring *ring) { - struct radeon_fence *fence; + struct radeon_fence *fence = NULL; int r; + r = radeon_set_uvd_clocks(rdev, 53300, 40000); + if (r) { + DRM_ERROR("radeon: failed to raise UVD clocks (%d).\n", r); + return r; + } + r = radeon_uvd_get_create_msg(rdev, ring->idx, 1, NULL); if (r) { DRM_ERROR("radeon: failed to get create msg (%d).\n", r); - return r; + goto error; } r = radeon_uvd_get_destroy_msg(rdev, ring->idx, 1, &fence); if (r) { DRM_ERROR("radeon: failed to get destroy ib (%d).\n", r); - return r; + goto error; } r = radeon_fence_wait(fence, false); if (r) { DRM_ERROR("radeon: fence wait failed (%d).\n", r); - return r; + goto error; } DRM_INFO("ib test on ring %d succeeded\n", ring->idx); +error: radeon_fence_unref(&fence); + radeon_set_uvd_clocks(rdev, 0, 0); return r; } -- 2.7.4