[PATCH] drm: Remove the EDID blob stored in the EDID property when it is disconnected

2010-03-04 Thread Zhenyu Wang 
From: Zhao Yakui yakui.z...@intel.com

Now the EDID property will be updated when the corresponding EDID can be
obtained from the external display device. But after the external device
is plugged-out, the EDID property is not updated. In such case we still
get the corresponding EDID property although it is already detected as
disconnected.

https://bugs.freedesktop.org/show_bug.cgi?id=26743

Signed-off-by: Zhao Yakui yakui.z...@intel.com
Signed-off-by: Zhenyu Wang zhen...@linux.intel.com
---
 drivers/gpu/drm/drm_crtc_helper.c |1 +
 1 files changed, 1 insertions(+), 0 deletions(-)

diff --git a/drivers/gpu/drm/drm_crtc_helper.c 
b/drivers/gpu/drm/drm_crtc_helper.c
index f2aaf39..51103aa 100644
--- a/drivers/gpu/drm/drm_crtc_helper.c
+++ b/drivers/gpu/drm/drm_crtc_helper.c
@@ -104,6 +104,7 @@ int drm_helper_probe_single_connector_modes(struct 
drm_connector *connector,
if (connector-status == connector_status_disconnected) {
DRM_DEBUG_KMS(%s is disconnected\n,
  drm_get_connector_name(connector));
+   drm_mode_connector_update_edid_property(connector, NULL);
goto prune;
}
 
-- 
1.6.3.3


--
Download Intel#174; Parallel Studio Eval
Try the new software tools for yourself. Speed compiling, find bugs
proactively, and fine-tune applications for parallel performance.
See why Intel Parallel Studio got high marks during beta.
http://p.sf.net/sfu/intel-sw-dev
--
___
Dri-devel mailing list
Dri-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dri-devel

Re: [PATCH] drm: Remove the EDID blob stored in the EDID property when it is disconnected

2010-03-04 Thread Jesse Barnes 
On Thu,  4 Mar 2010 16:25:55 +0800
Zhenyu Wang zhen...@linux.intel.com wrote:

 From: Zhao Yakui yakui.z...@intel.com
 
 Now the EDID property will be updated when the corresponding EDID can be
 obtained from the external display device. But after the external device
 is plugged-out, the EDID property is not updated. In such case we still
 get the corresponding EDID property although it is already detected as
 disconnected.
 
 https://bugs.freedesktop.org/show_bug.cgi?id=26743
 
 Signed-off-by: Zhao Yakui yakui.z...@intel.com
 Signed-off-by: Zhenyu Wang zhen...@linux.intel.com
 ---
  drivers/gpu/drm/drm_crtc_helper.c |1 +
  1 files changed, 1 insertions(+), 0 deletions(-)
 
 diff --git a/drivers/gpu/drm/drm_crtc_helper.c 
 b/drivers/gpu/drm/drm_crtc_helper.c
 index f2aaf39..51103aa 100644
 --- a/drivers/gpu/drm/drm_crtc_helper.c
 +++ b/drivers/gpu/drm/drm_crtc_helper.c
 @@ -104,6 +104,7 @@ int drm_helper_probe_single_connector_modes(struct 
 drm_connector *connector,
   if (connector-status == connector_status_disconnected) {
   DRM_DEBUG_KMS(%s is disconnected\n,
 drm_get_connector_name(connector));
 + drm_mode_connector_update_edid_property(connector, NULL);
   goto prune;
   }
  

I think this should be safe, and does fix a real bug.  If so, it should
also be cc: sta...@kernel.org.

Dave?

-- 
Jesse Barnes, Intel Open Source Technology Center

--
Download Intel#174; Parallel Studio Eval
Try the new software tools for yourself. Speed compiling, find bugs
proactively, and fine-tune applications for parallel performance.
See why Intel Parallel Studio got high marks during beta.
http://p.sf.net/sfu/intel-sw-dev
--
___
Dri-devel mailing list
Dri-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dri-devel