From mboxrd@z Thu Jan 1 00:00:00 1970 From: Dave Airlie Subject: [PATCH 3/3] drm/radeon/kms: fix dac detect stealing in-use encoder. Date: Tue, 14 Jun 2011 16:13:56 +1000 Message-ID: <1308032036-23115-3-git-send-email-airlied@gmail.com> References: <1308032036-23115-1-git-send-email-airlied@gmail.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Received: from mx1.redhat.com (mx1.redhat.com [209.132.183.28]) by gabe.freedesktop.org (Postfix) with ESMTP id 843619E70A for ; Mon, 13 Jun 2011 23:14:01 -0700 (PDT) Received: from int-mx12.intmail.prod.int.phx2.redhat.com (int-mx12.intmail.prod.int.phx2.redhat.com [10.5.11.25]) by mx1.redhat.com (8.14.4/8.14.4) with ESMTP id p5E6E1QD007744 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-SHA bits=256 verify=OK) for ; Tue, 14 Jun 2011 02:14:01 -0400 Received: from clockmaker-el6.bne.redhat.com (dhcp-0-222.bne.redhat.com [10.64.0.222]) by int-mx12.intmail.prod.int.phx2.redhat.com (8.14.4/8.14.4) with ESMTP id p5E6DvLv008542 for ; Tue, 14 Jun 2011 02:14:00 -0400 In-Reply-To: <1308032036-23115-1-git-send-email-airlied@gmail.com> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: dri-devel-bounces+sf-dri-devel=m.gmane.org@lists.freedesktop.org Errors-To: dri-devel-bounces+sf-dri-devel=m.gmane.org@lists.freedesktop.org To: dri-devel@lists.freedesktop.org List-Id: dri-devel@lists.freedesktop.org From: Dave Airlie On my RS690 the VGA and TV-out share the same DAC, so when xrandr is called, tv-out steals the encoder and tries to load detect on it, however this causes flicker on the VGA. This blocks this by testing if the encoder is connected to anything and if so whether its connected to the probing connector. Signed-off-by: Dave Airlie --- drivers/gpu/drm/radeon/radeon_encoders.c | 12 ++++++++++++ 1 files changed, 12 insertions(+), 0 deletions(-) diff --git a/drivers/gpu/drm/radeon/radeon_encoders.c b/drivers/gpu/drm/radeon/radeon_encoders.c index 03f124d..eb07e74 100644 --- a/drivers/gpu/drm/radeon/radeon_encoders.c +++ b/drivers/gpu/drm/radeon/radeon_encoders.c @@ -1971,6 +1971,18 @@ radeon_atom_dac_detect(struct drm_encoder *encoder, struct drm_connector *connec struct radeon_encoder *radeon_encoder = to_radeon_encoder(encoder); struct radeon_connector *radeon_connector = to_radeon_connector(connector); uint32_t bios_0_scratch; + struct drm_connector *test_connector; + + /* + * if we are passed a connector and it doesn't match what the encoder + * is connected to do, don't do load detect as it might cause flicker. + */ + list_for_each_entry(test_connector, &dev->mode_config.connector_list, head) { + if ((test_connector->encoder == encoder) && (test_connector != connector)) { + DRM_DEBUG_KMS("load detect failed: encoder in use\n"); + return connector_status_disconnected; + } + } if (!atombios_dac_load_detect(encoder, connector)) { DRM_DEBUG_KMS("detect returned false \n"); -- 1.7.5.2