From mboxrd@z Thu Jan 1 00:00:00 1970 From: bugzilla-daemon@freedesktop.org Subject: [Bug 101723] hdmi unplug not changing connector status Date: Fri, 07 Jul 2017 22:37:27 +0000 Message-ID: Mime-Version: 1.0 Content-Type: multipart/mixed; boundary="===============1867882622==" Return-path: Received: from culpepper.freedesktop.org (culpepper.freedesktop.org [IPv6:2610:10:20:722:a800:ff:fe98:4b55]) by gabe.freedesktop.org (Postfix) with ESMTP id 574376E2EE for ; Fri, 7 Jul 2017 22:37:27 +0000 (UTC) List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dri-devel-bounces@lists.freedesktop.org Sender: "dri-devel" To: dri-devel@lists.freedesktop.org List-Id: dri-devel@lists.freedesktop.org --===============1867882622== Content-Type: multipart/alternative; boundary="14994670470.ea2Ac8F.14381"; charset="UTF-8" --14994670470.ea2Ac8F.14381 Date: Fri, 7 Jul 2017 22:37:27 +0000 MIME-Version: 1.0 Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable X-Bugzilla-URL: http://bugs.freedesktop.org/ Auto-Submitted: auto-generated https://bugs.freedesktop.org/show_bug.cgi?id=3D101723 Bug ID: 101723 Summary: hdmi unplug not changing connector status Product: DRI Version: unspecified Hardware: All OS: Linux (All) Status: NEW Severity: minor Priority: medium Component: DRM/Radeon Assignee: dri-devel@lists.freedesktop.org Reporter: aki.lemmetyinen@gmail.com Created attachment 132558 --> https://bugs.freedesktop.org/attachment.cgi?id=3D132558&action=3Dedit Small patch to fix the problem I made a quick script for my HP Pavilion dv7 to automatically switch audio = to hdmi and back at hotplug event, but I noticed that the driver do not react properly to plug/unplug. /sys/class/drm/card0-HDMI-A-1/status was "connecte= d" even after unplugging the cable. After looking at the code it seemed that in case of cable disconnected ther= e is not point where hotplug state is polled. Function radeon_connector_hotplug do not do much with hdmi and radeon_hotplug_work_func cals drm_helper_hpd_irq_event which calls radeon_dvi_detect where, in case of disconnected cable, it return with connector_status_connected, just throwing an error for missing EDID. It looks like there would maybe be a need for a bigger rewrite of hotplug c= ode, but just to get connector state working properly, I added these few lines to radeon_connectors.c. --=20 You are receiving this mail because: You are the assignee for the bug.= --14994670470.ea2Ac8F.14381 Date: Fri, 7 Jul 2017 22:37:27 +0000 MIME-Version: 1.0 Content-Type: text/html; charset="UTF-8" Content-Transfer-Encoding: quoted-printable X-Bugzilla-URL: http://bugs.freedesktop.org/ Auto-Submitted: auto-generated
Bug ID 101723
Summary hdmi unplug not changing connector status
Product DRI
Version unspecified
Hardware All
OS Linux (All)
Status NEW
Severity minor
Priority medium
Component DRM/Radeon
Assignee dri-devel@lists.freedesktop.org
Reporter aki.lemmetyinen@gmail.com

Created attachmen=
t 132558 [details] [review]
Small patch to fix the problem

I made a quick script for my HP Pavilion dv7 to automatically switch audio =
to
hdmi and back at hotplug event, but I noticed that the driver do not react
properly to plug/unplug. /sys/class/drm/card0-HDMI-A-1/status was "con=
nected"
even after unplugging the cable.

After looking at the code it seemed that in case of cable disconnected ther=
e is
not point where hotplug state is polled.

Function radeon_connector_hotplug do not do much with hdmi and
radeon_hotplug_work_func cals drm_helper_hpd_irq_event which calls
radeon_dvi_detect where, in case of disconnected cable, it return with
connector_status_connected, just throwing an error for missing EDID.

It looks like there would maybe be a need for a bigger rewrite of hotplug c=
ode,
but just to get connector state working properly, I added these few lines to
radeon_connectors.c.


You are receiving this mail because:
  • You are the assignee for the bug.
= --14994670470.ea2Ac8F.14381-- --===============1867882622== Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: base64 Content-Disposition: inline X19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX18KZHJpLWRldmVs IG1haWxpbmcgbGlzdApkcmktZGV2ZWxAbGlzdHMuZnJlZWRlc2t0b3Aub3JnCmh0dHBzOi8vbGlz dHMuZnJlZWRlc2t0b3Aub3JnL21haWxtYW4vbGlzdGluZm8vZHJpLWRldmVsCg== --===============1867882622==--