Bug 16631

Summary: [965 SDVO] Unable to write to SDVOCTRL_E for SDVOB Slave
Product: xorg Reporter: liuhaien <haien.liu>
Component: Driver/intelAssignee: Wang Zhenyu <zhenyu.z.wang>
Status: VERIFIED FIXED QA Contact: Xorg Project Team <xorg-team>
Severity: critical    
Priority: high CC: hasso, nian.wu
Version: git   
Hardware: Other   
OS: Linux (All)   
Whiteboard:
i915 platform: i915 features:
Bug Depends on:    
Bug Blocks: 16926    
Attachments:
Description Flags
xorg.0.log
none
xorg conf file
none
sdvo probe debug patch
none
updated patch
none
force sdvo detect patch none

Description liuhaien 2008-07-06 23:26:16 UTC
Created attachment 17559 [details]
xorg.0.log

System Environment:
--------------------------

--Platform:q965

--Architecture:32-bit

--2D driver: master f2ec3fd8f2c63b1bc0745802dcf737eda4bb42b6

--3D driver:master 2b2cb566563b9f1f9739327ef9874143af838850

--DRM: master 401f77a2ef1d55feaccc13cdae1e5379f4cfec7e

--Xserver:master d78bebb20a00e8519788c75c90b467a5750c78be

--Kernel: 2.6.26-rc8

Bug detailed description:
--------------------------
VGA-monitor connected by SDVO ,it will display below error info:"(EE) intel(0): Unable to read from SDVOCTRL_E for SDVOB slave 0x70." and X cannot start .

Reproduce steps:
----------------
using SDVO connected monitor


Current result:
----------------
 X starts failed with above info.

Expected result:
----------------
start X normally.
Comment 1 liuhaien 2008-07-06 23:29:16 UTC
Created attachment 17560 [details]
xorg conf file
Comment 2 liuhaien 2008-07-06 23:54:05 UTC
we connected two monitor,one is connected by VGA,and the other is connected by SDVO.and it also happened on xf86-video-intel-2.4-branch.
Comment 3 Wang Zhenyu 2008-07-28 00:08:44 UTC
Details on the ADD2 card you use?
Comment 4 WuNian 2008-07-28 19:47:22 UTC
My Add2 card is: WINTEC ZDDVI8SI-A
Comment 5 Michael Fu 2008-07-28 19:56:57 UTC
a quick test, how about if you use another ADD2 card instead? this can help us quickly rule out if there is a HW issue here...thanks.
Comment 6 WuNian 2008-07-28 22:02:04 UTC
We have tested with other ADD2 card, also had this problem.
We used this ADD2 case on our Q965 machine before. So this is a regression.
Comment 7 Michael Fu 2008-08-02 22:00:20 UTC
(In reply to comment #6)
> We have tested with other ADD2 card, also had this problem.
> We used this ADD2 case on our Q965 machine before. So this is a regression.
> 

could be a HW issue,too. here HW mean the motherboard...Can you confirm if this 965 the "Q965" that you said works before?
Comment 8 WuNian 2008-08-03 18:08:40 UTC
(In reply to comment #7)
> could be a HW issue,too. here HW mean the motherboard...Can you confirm if this
> 965 the "Q965" that you said works before?
> 
I used to play the ADD2 card on the "Q965" to test dual head. It always worked.

Comment 9 Gordon Jin 2008-09-12 21:24:43 UTC
*** Bug 17240 has been marked as a duplicate of this bug. ***
Comment 10 Gordon Jin 2008-09-12 21:26:53 UTC
I'm increasing priority, since 2 users (from bug#17240) could confirm this is not hw issue.
Comment 11 Wang Zhenyu 2008-09-16 01:01:25 UTC
Created attachment 18913 [details] [review]
sdvo probe debug patch

Please try if this patch restores the normal behavior back.
Comment 12 Hasso Tepper 2008-09-16 01:29:52 UTC
Yes, it does.
Comment 13 Norman Yarvin 2008-09-16 18:32:41 UTC
The patch works for me, too.
Comment 14 Gordon Jin 2008-09-17 20:18:43 UTC
Haien, please try Zhenyu's patch.
Comment 15 Wang Zhenyu 2008-09-17 23:28:29 UTC
Created attachment 18969 [details] [review]
updated patch

Try detect bits before init HDMI
Comment 16 Norman Yarvin 2008-09-18 14:28:57 UTC
The updated patch also works for me.  (I still get the error messages "(EE) intel(0): Unable to write to SDVOCTRL_E for SDVOB Slave 0x70." and "(EE) intel(0): Unable to write to SDVOCTRL_E for SDVOC Slave 0x72."), but they don't cause any loss of functionality.)
Comment 17 Wang Zhenyu 2008-09-26 00:05:32 UTC
Created attachment 19230 [details] [review]
force sdvo detect patch

This is the one I'd like to push. Use an option to enable SDVO force detect, as it will cost much in X startup time, so normal user shouldn't be affected by this probe.
Comment 18 Wang Zhenyu 2008-09-27 19:19:14 UTC
Pushed patch. Close.

commit 11d304e99c0e11c28901ec28115d9c8b81a2b9cc
Author: Zhenyu Wang <zhenyu.z.wang@intel.com>
Date:   Sun Sep 28 10:08:26 2008 +0800

    Bug #16631: add option for SDVO force detect
    
    Some ADD2 card doesn't get SDVO detect status setup right,
    which disabled outputs on those cards. This adds a new
    option "ForceSDVODetect" to probe all SDVO ports anyway.
Comment 19 liuhaien 2008-10-12 18:59:45 UTC
verified.

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.