Attachment 35067 Details for Bug 27649 – fix

[patch] fix

0001-drm-radeon-kms-fix-tv-dac-conflict-resolver.patch (text/plain), 1.86 KB, created by Alex Deucher on 2010-04-15 10:42:10 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Alex Deucher

Created: 2010-04-15 10:42:10 UTC

Size: 1.86 KB

patch

obsolete

>From 837feb147c531219c7315857a6f17e145af8f750 Mon Sep 17 00:00:00 2001
>From: Alex Deucher <alexdeucher@gmail.com>
>Date: Thu, 15 Apr 2010 13:31:12 -0400
>Subject: [PATCH] drm/radeon/kms: fix tv dac conflict resolver
>
>On systems with the tv dac shared between DVI and TV,
>we can only use the dac for one of the connectors.
>However, when using a digital monitor on the DVI port,
>you can use the dac for the TV connector just fine.
>Check the use_digital status when resolving the conflict.
>
>Fixes fdo bug 27649, possibly others.
>
>Signed-off-by: Alex Deucher <alexdeucher@gmail.com>
>Cc: stable <stable@kernel.org>
>---
> drivers/gpu/drm/radeon/radeon_connectors.c |    5 +++++
> 1 files changed, 5 insertions(+), 0 deletions(-)
>
>diff --git a/drivers/gpu/drm/radeon/radeon_connectors.c b/drivers/gpu/drm/radeon/radeon_connectors.c
>index 5c5776d..c489346 100644
>--- a/drivers/gpu/drm/radeon/radeon_connectors.c
>+++ b/drivers/gpu/drm/radeon/radeon_connectors.c
>@@ -162,12 +162,14 @@ radeon_connector_analog_encoder_conflict_solve(struct drm_connector *connector,
> {
> 	struct drm_device *dev = connector->dev;
> 	struct drm_connector *conflict;
>+	struct radeon_connector *radeon_conflict;
> 	int i;
> 
> 	list_for_each_entry(conflict, &dev->mode_config.connector_list, head) {
> 		if (conflict == connector)
> 			continue;
> 
>+		radeon_conflict = to_radeon_connector(conflict);
> 		for (i = 0; i < DRM_CONNECTOR_MAX_ENCODER; i++) {
> 			if (conflict->encoder_ids[i] == 0)
> 				break;
>@@ -177,6 +179,9 @@ radeon_connector_analog_encoder_conflict_solve(struct drm_connector *connector,
> 				if (conflict->status != connector_status_connected)
> 					continue;
> 
>+				if (radeon_conflict->use_digital)
>+					continue;
>+
> 				if (priority == true) {
> 					DRM_INFO("1: conflicting encoders switching off %s\n", drm_get_connector_name(conflict));
> 					DRM_INFO("in favor of %s\n", drm_get_connector_name(connector));
>-- 
>1.5.6.3
>

Actions: View | Splinter Review

Attachments on bug 27649: 35034 | 35035 | 35037 | 35038 | 35039 | 35041 | 35067

Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.