Bug 20684

Summary: [SDVO-TV] setting properties via xrandr doesn't work on intel D945GCLF2
Product: xorg Reporter: georgy@kibardin.name <george-kibardin>
Component: Driver/intelAssignee: Wang Zhenyu <zhenyu.z.wang>
Status: RESOLVED NOTABUG QA Contact: Xorg Project Team <xorg-team>
Severity: normal    
Priority: medium CC: haien.liu
Version: unspecified   
Hardware: Other   
OS: All   
i915 platform: i915 features:
Description Flags
new sdvo properties none

Description georgy@kibardin.name 2009-03-16 05:11:31 UTC
 xrandr --output TV-1 --set LEFT 0
X Error of failed request:  BadName (named color or font does not exist)
  Major opcode of failed request:  152 (RANDR)
  Minor opcode of failed request:  11 ()
  Serial number of failed request:  17
  Current serial number in output stream:  17
The same is for RIGHT, TOP, BOTTOM and TV_FORMAT.

xrandr --output TV-1 --mode ....

this happens with 2.6.1 with patch from bug #17776 applied
Comment 1 Wang Zhenyu 2009-03-16 20:13:18 UTC
It's not supported in driver.
Comment 2 georgy@kibardin.name 2009-03-16 23:06:15 UTC
"man intel" says that it is. Or do you mean that these properties are not supported by this chipset?
Comment 3 Wang Zhenyu 2009-03-19 00:32:40 UTC
manpage's TV attributes are for integrated TV only, and nothing applys to SDVO TV now.
Comment 4 georgy@kibardin.name 2009-03-19 04:31:49 UTC
I digged the code a bit, now I see.

Actually, I just want to solve the problem with the picture shifted to the right-bottom corner so I get two black stripes on the top and on the left. I've tried to play with modelines but they always get adjusted.

I think in this case it would be great to have an overscan mode when the picture is always a bit bigger than the screen so it would be possible to correct this later when palying movies etc.

Maybe it is also possible to emulate those properties when calculating modelines?

What do you think?
Comment 5 Wang Zhenyu 2009-03-19 19:04:55 UTC
yeah, SDVO does have many TV enhancement commands for tuning, but current xf-v-i driver doesn't support them yet.

Comment 6 georgy@kibardin.name 2009-03-21 12:21:38 UTC
Created attachment 24124 [details] [review]
new sdvo properties

I've created a patch that allows to set different sdvo properties. Could you please take a look? I tested at least positions, overscan and saturation.
Comment 7 georgy@kibardin.name 2009-03-21 12:23:24 UTC
Sorry, it looks like a patch proposed for bug# 20785 is also there :(
Comment 8 Wang Zhenyu 2009-03-22 18:08:48 UTC
cool, yeah, TV detect problem should be separate. And besides options from xorg.conf, we should make randr output properties too. I think psb's sdvo driver can be an example, would you pick it up and send future patches to intel-gfx@lists.freedesktop.org to review? Thanks.
Comment 9 georgy@kibardin.name 2009-03-30 23:57:47 UTC
I posted the patch yesterday, but it looks like nobody cares. May be I did something wrong?
Comment 10 Wang Zhenyu 2009-03-31 00:17:10 UTC
George, thanks for doing this, I haven't done a detail review of it, as I'm kind of busy with something else. I'll try to look into it more, or maybe some other one could pick it up.
Comment 11 georgy@kibardin.name 2009-03-31 00:19:28 UTC
No problem, I just want to be sure I did everything right :)

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.