diff mbox

drm/radeon/kms: fix regression in hotplug

Message ID 1310395768-32343-1-git-send-email-alexdeucher@gmail.com (mailing list archive)
State New, archived
Headers show

Commit Message

Alex Deucher July 11, 2011, 2:49 p.m. UTC
Skip connectors that do not have an HPD pin.

Should fix:
https://bugs.freedesktop.org/show_bug.cgi?id=39027

Signed-off-by: Alex Deucher <alexdeucher@gmail.com>
---
 drivers/gpu/drm/radeon/radeon_connectors.c |    6 ++++++
 1 files changed, 6 insertions(+), 0 deletions(-)

Comments

Rafał Miłecki July 11, 2011, 9:02 p.m. UTC | #1
2011/7/11 Alex Deucher <alexdeucher@gmail.com>:
> Skip connectors that do not have an HPD pin.
>
> Should fix:
> https://bugs.freedesktop.org/show_bug.cgi?id=39027
>
> Signed-off-by: Alex Deucher <alexdeucher@gmail.com>

Tested-by: Rafa? Mi?ecki <zajec5@gmail.com>

Fixes the regression.
diff mbox

Patch

diff --git a/drivers/gpu/drm/radeon/radeon_connectors.c b/drivers/gpu/drm/radeon/radeon_connectors.c
index cbfca3a..9792d4f 100644
--- a/drivers/gpu/drm/radeon/radeon_connectors.c
+++ b/drivers/gpu/drm/radeon/radeon_connectors.c
@@ -52,6 +52,12 @@  void radeon_connector_hotplug(struct drm_connector *connector)
 	struct radeon_device *rdev = dev->dev_private;
 	struct radeon_connector *radeon_connector = to_radeon_connector(connector);
 
+	/* bail if the connector does not have hpd pin, e.g.,
+	 * VGA, TV, etc.
+	 */
+	if (radeon_connector->hpd.hpd == RADEON_HPD_NONE)
+		return;
+
 	radeon_hpd_set_polarity(rdev, radeon_connector->hpd.hpd);
 
 	/* powering up/down the eDP panel generates hpd events which