[Intel-gfx] TV Out strobing regression bisected

[Ma, Ling]

hi Rob,
Thanks for your help,  please try the debug patch  on your machine.

Regards
Ma Ling

-----Original Message-----
From: Robert Lowery [mailto:rglowery at exemail.com.au] 
Sent: Monday, March 23, 2009 9:51 PM
To: intel-gfx at lists.freedesktop.org
Cc: Ma, Ling
Subject: TV Out strobing regression bisected

Folks,

I have an AOpen MP965-DR mini PC that I run a projector over DVI at
1920x1080 and a TV over component at 576p.  Updating to the master intel
driver has made the TV Out flash like a strobe light as if the picture is
too big for the screen.

I have bisected it to the below commit and confirmed that master minus
this commit works fine for me.

Any suggestions?

-Rob

commit id b9f5915ce812144ffd9d2aa42e8ba856129c35e,
which resolved bug #17235, but generate new regression-bug #19715.
This patch intends to resolve bug #17235, and avoid regression as well.
We have successfully re-tested it for bug #17235 and #19715 respectively.

diff --git a/src/i830_display.c b/src/i830_display.c
index 7a8e96d..8ecb5e5 100644
--- a/src/i830_display.c
+++ b/src/i830_display.c
@@ -808,10 +808,6 @@ i830_disable_vga_plane (xf86CrtcPtr crtc)

     vgacntrl |= VGA_DISP_DISABLE;

-    /* disable center mode */
-    if (IS_I9XX(pI830))
-	vgacntrl &= ~(3 << 24);
-
     OUTREG(VGACNTRL, vgacntrl);
     i830WaitForVblank(pScrn);




-------------- next part --------------
A non-text attachment was scrubbed...
Name: vgadisable.patch
Type: application/octet-stream
Size: 447 bytes
Desc: vgadisable.patch
URL: <http://lists.freedesktop.org/archives/intel-gfx/attachments/20090324/4d42aca7/attachment.obj>