On Wed, 24 Aug 2005 16:26:58 -0400
Alex Deucher <[EMAIL PROTECTED]> wrote:

> On 8/24/05, Aapo Tahkola <[EMAIL PROTECTED]> wrote:
> > On Fri, 15 Jul 2005 12:04:43 +0200
> > Rune Petersen <[EMAIL PROTECTED]> wrote:
> > 
> > > Just to add that login (KDM) is more or less fine (cursor is corrupted
> > > untill its moved).
> > 
> > Attached diff should fix this temporarily.
> > Im still unsure why MaxSurfaceWidth ends up affecting piches and stuff...
> > 
> 
> Odd... I never saw any problems like that.  However, if you set your
> virtual desktop greater than 3968, the desktop gets VERY corrupt.  I
> don't see why that would affect pitches either.

That check doesnt currently seem to work because some setup(at preinit) with 
color tiling enabled will get done earlier.

Does this version break r200s or radeons?

-- 
Aapo Tahkola
Index: radeon_driver.c
===================================================================
RCS file: /cvs/xorg/xc/programs/Xserver/hw/xfree86/drivers/ati/radeon_driver.c,v
retrieving revision 1.69
diff -u -b -B -u -r1.69 radeon_driver.c
--- radeon_driver.c     8 Aug 2005 23:42:36 -0000       1.69
+++ radeon_driver.c     25 Aug 2005 12:53:31 -0000
@@ -3684,7 +3684,6 @@
                              NULL,                  /* linePitches */
                              8 * 64,                /* minPitch */
                              8 * 1024,              /* maxPitch */
-                             info->allowColorTiling ? info->MaxSurfaceWidth :
                                  64 * pScrn1->bitsPerPixel, /* pitchInc */
                              128,                   /* minHeight */
                              8 * 1024, /*2048,*/    /* maxHeight */
@@ -3747,7 +3746,6 @@
                                          NULL,                  /* linePitches 
*/
                                          8 * 64,                /* minPitch */
                                          8 * 1024,              /* maxPitch */
-                                         info->allowColorTiling ? 
info->MaxSurfaceWidth :
                                              64 * pScrn1->bitsPerPixel, /* 
pitchInc */
                                          128,                   /* minHeight */
                                          8 * 1024, /*2048,*/    /* maxHeight */
@@ -3949,7 +3947,6 @@
                              NULL,                  /* linePitches */
                              8 * 64,                /* minPitch */
                              8 * 1024,              /* maxPitch */
-                             info->allowColorTiling ? info->MaxSurfaceWidth :
                                  64 * pScrn->bitsPerPixel, /* pitchInc */
                              128,                   /* minHeight */
                              info->MaxLines,      /* maxHeight */
@@ -4018,7 +4015,6 @@
                                          NULL,                  /* linePitches 
*/
                                          8 * 64,                /* minPitch */
                                          8 * 1024,              /* maxPitch */
-                                         info->allowColorTiling ? 
info->MaxSurfaceWidth :
                                              64 * pScrn->bitsPerPixel, /* 
pitchInc */
                                          128,                   /* minHeight */
                                          info->MaxLines,      /* maxHeight */

Reply via email to