All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] drm/radeon/kms: bail on BTC parts if MC ucode is missing
@ 2011-12-21 16:58 alexdeucher
  2011-12-21 16:58 ` [PATCH] drm/radeon/kms: add missing ring ready check in sync tests alexdeucher
  0 siblings, 1 reply; 2+ messages in thread
From: alexdeucher @ 2011-12-21 16:58 UTC (permalink / raw)
  To: airlied, dri-devel; +Cc: Alex Deucher, Jean Delvare, stable

From: Alex Deucher <alexander.deucher@amd.com>

We already do this for cayman, need to also do it for
BTC parts.  The default memory and voltage setup is not
adequate for advanced operation.  Continuing will
result in an unusable display.

Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
Cc: stable@kernel.org
Cc: Jean Delvare <khali@linux-fr.org>
---
 drivers/gpu/drm/radeon/evergreen.c |   12 ++++++++++++
 1 files changed, 12 insertions(+), 0 deletions(-)

diff --git a/drivers/gpu/drm/radeon/evergreen.c b/drivers/gpu/drm/radeon/evergreen.c
index 1934728..c2e493d 100644
--- a/drivers/gpu/drm/radeon/evergreen.c
+++ b/drivers/gpu/drm/radeon/evergreen.c
@@ -3337,6 +3337,18 @@ int evergreen_init(struct radeon_device *rdev)
 		evergreen_pcie_gart_fini(rdev);
 		rdev->accel_working = false;
 	}
+
+	/* Don't start up if the MC ucode is missing on BTC parts.
+	 * The default clocks and voltages before the MC ucode
+	 * is loaded are not suffient for advanced operations.
+	 */
+	if (ASIC_IS_DCE5(rdev)) {
+		if (!rdev->mc_fw && !(rdev->flags & RADEON_IS_IGP)) {
+			DRM_ERROR("radeon: MC ucode required for NI+.\n");
+			return -EINVAL;
+		}
+	}
+
 	return 0;
 }
 
-- 
1.7.3.4

^ permalink raw reply related	[flat|nested] 2+ messages in thread

* [PATCH] drm/radeon/kms: add missing ring ready check in sync tests
  2011-12-21 16:58 [PATCH] drm/radeon/kms: bail on BTC parts if MC ucode is missing alexdeucher
@ 2011-12-21 16:58 ` alexdeucher
  0 siblings, 0 replies; 2+ messages in thread
From: alexdeucher @ 2011-12-21 16:58 UTC (permalink / raw)
  To: airlied, dri-devel; +Cc: Alex Deucher

From: Alex Deucher <alexander.deucher@amd.com>

Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
---
 drivers/gpu/drm/radeon/radeon_test.c |    2 ++
 1 files changed, 2 insertions(+), 0 deletions(-)

diff --git a/drivers/gpu/drm/radeon/radeon_test.c b/drivers/gpu/drm/radeon/radeon_test.c
index 210d99f..dc5dcf4 100644
--- a/drivers/gpu/drm/radeon/radeon_test.c
+++ b/drivers/gpu/drm/radeon/radeon_test.c
@@ -471,6 +471,8 @@ void radeon_test_syncing(struct radeon_device *rdev)
 
 			for (k = 0; k < j; ++k) {
 				struct radeon_ring *ringC = &rdev->ring[k];
+				if (!ringC->ready)
+					continue;
 
 				DRM_INFO("Testing syncing between rings %d, %d and %d...\n", i, j, k);
 				radeon_test_ring_sync2(rdev, ringA, ringB, ringC);
-- 
1.7.3.4

^ permalink raw reply related	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2011-12-21 16:58 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2011-12-21 16:58 [PATCH] drm/radeon/kms: bail on BTC parts if MC ucode is missing alexdeucher
2011-12-21 16:58 ` [PATCH] drm/radeon/kms: add missing ring ready check in sync tests alexdeucher

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.