[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

xserver-xorg-video-ati: Changes to 'debian-unstable'



 debian/changelog   |    7 +++++++
 src/radeon_video.c |   21 ---------------------
 2 files changed, 7 insertions(+), 21 deletions(-)

New commits:
commit 8398efc5fd8e9c82fe337030a51c48c5a6087d28
Author: Brice Goglin <bgoglin@debian.org>
Date:   Fri Sep 5 18:43:35 2008 +0200

    Prepare changelog for upload

diff --git a/debian/changelog b/debian/changelog
index 37e79b5..d67de20 100644
--- a/debian/changelog
+++ b/debian/changelog
@@ -1,3 +1,10 @@
+xserver-xorg-video-ati (1:6.9.0-1+lenny4) unstable; urgency=low
+
+  * Revert "Make sure video offerlay offsets don't exceed the hardware
+            limit of 128 MB", closes: #496979.
+
+ -- Brice Goglin <bgoglin@debian.org>  Fri, 05 Sep 2008 18:42:32 +0200
+
 xserver-xorg-video-ati (1:6.9.0-1+lenny3) unstable; urgency=low
 
   * Cherry-pick patches from the upstream git repository:

commit bf613a7aa07e0e625583b9fa69f2c9123e506379
Author: Brice Goglin <bgoglin@debian.org>
Date:   Fri Sep 5 18:40:46 2008 +0200

    Revert "Make sure video offerlay offsets don't exceed the hardware limit of 128 MB."
    
    This reverts commit 15536739b0e68e25dbd89d5d517680d855077fab.
    
    This commit breaks more (see #496979 and #497612) than it helps.

diff --git a/src/radeon_video.c b/src/radeon_video.c
index 57dcd8a..ac60166 100644
--- a/src/radeon_video.c
+++ b/src/radeon_video.c
@@ -2586,7 +2586,6 @@ RADEONDisplayVideo(
     RADEONOutputPrivatePtr radeon_output;
     xf86OutputPtr output;
     RADEONCrtcPrivatePtr radeon_crtc = crtc->driver_private;
-    int base_offset;
 
     is_rgb=0; is_planar=0;
     switch(id){
@@ -2716,22 +2715,6 @@ RADEONDisplayVideo(
         }
 #endif
 
-    /* Make the overlay base address as close to the buffers as possible to
-     * prevent the buffer offsets from exceeding the hardware limit of 128 MB.
-     * The base address must be aligned to a multiple of 4 MB.
-     */
-    base_offset = ((info->fbLocation +
-		    min(offset1, min(offset2, min(offset3, min(offset4,
-			min(offset5, offset6)))))) & (~0 << 22)) -
-	info->fbLocation;
-
-    offset1 -= base_offset;
-    offset2 -= base_offset;
-    offset3 -= base_offset;
-    offset4 -= base_offset;
-    offset5 -= base_offset;
-    offset6 -= base_offset;
-
     /* keep everything in 16.16 */
 
     if (is_planar) {
@@ -2863,10 +2846,6 @@ RADEONDisplayVideo(
 	src_w >>= 1;
     OUTREG(RADEON_OV0_P2_X_START_END, (src_w + leftuv - 1) | (leftuv << 16));
     OUTREG(RADEON_OV0_P3_X_START_END, (src_w + leftuv - 1) | (leftuv << 16));
-    if (info->ModeReg->ov0_base_addr != (info->fbLocation + base_offset)) {
-	info->ModeReg->ov0_base_addr = info->fbLocation + base_offset;
-	OUTREG(RADEON_OV0_BASE_ADDR, info->ModeReg->ov0_base_addr);
-    }
     OUTREG(RADEON_OV0_VID_BUF0_BASE_ADRS, offset1);
     OUTREG(RADEON_OV0_VID_BUF1_BASE_ADRS, offset2);
     OUTREG(RADEON_OV0_VID_BUF2_BASE_ADRS, offset3);


Reply to: