xf86-video-ati: Branch 'master'

Alex Deucher agd5f at kemper.freedesktop.org
Thu Apr 23 08:50:21 PDT 2009


 src/radeon_crtc.c   |    2 +-
 src/radeon_cursor.c |   14 +++-----------
 src/radeon_pm.c     |   14 +++-----------
 3 files changed, 7 insertions(+), 23 deletions(-)

New commits:
commit 0da80f34b5aa5999d2030ffbc5187328fa4e4ae5
Author: Alex Deucher <alexdeucher at gmail.com>
Date:   Wed Apr 22 18:42:36 2009 -0400

    radeon: minor cleanup and warning fix

diff --git a/src/radeon_crtc.c b/src/radeon_crtc.c
index a67d374..51f361d 100644
--- a/src/radeon_crtc.c
+++ b/src/radeon_crtc.c
@@ -181,7 +181,7 @@ RADEONComputePLL(RADEONPLLPtr pll,
 	}
 
 	for (ref_div = min_ref_div; ref_div <= max_ref_div; ++ref_div) {
-	    uint32_t feedback_div, current_freq, error, vco_diff;
+	    uint32_t feedback_div, current_freq = 0, error, vco_diff;
 	    uint32_t pll_in = pll->reference_freq / ref_div;
 	    uint32_t min_feed_div = pll->min_feedback_div;
 	    uint32_t max_feed_div = pll->max_feedback_div+1;
diff --git a/src/radeon_cursor.c b/src/radeon_cursor.c
index 30bee50..980b81f 100644
--- a/src/radeon_cursor.c
+++ b/src/radeon_cursor.c
@@ -194,6 +194,7 @@ void
 radeon_crtc_set_cursor_position (xf86CrtcPtr crtc, int x, int y)
 {
     ScrnInfoPtr pScrn = crtc->scrn;
+    RADEONEntPtr pRADEONEnt = RADEONEntPriv(pScrn);
     RADEONCrtcPrivatePtr radeon_crtc = crtc->driver_private;
     int crtc_id = radeon_crtc->crtc_id;
     RADEONInfoPtr      info       = RADEONPTR(pScrn);
@@ -208,9 +209,7 @@ radeon_crtc_set_cursor_position (xf86CrtcPtr crtc, int x, int y)
     if (yorigin >= CURSOR_HEIGHT) yorigin = CURSOR_HEIGHT - 1;
 
     if (IS_AVIVO_VARIANT) {
-	xf86CrtcConfigPtr xf86_config = XF86_CRTC_CONFIG_PTR(pScrn);
 	int w = CURSOR_WIDTH;
-	int i;
 
 	/* avivo cursor spans the full fb width */
 	if (crtc->rotatedData == NULL) {
@@ -218,15 +217,8 @@ radeon_crtc_set_cursor_position (xf86CrtcPtr crtc, int x, int y)
 	    y += crtc->y;
 	}
 
-	for (i = 0; i < xf86_config->num_crtc; i++) {
-	    xf86CrtcPtr crtc = xf86_config->crtc[i];
-	    RADEONCrtcPrivatePtr radeon_crtc = crtc->driver_private;
-
-	    if (!radeon_crtc->enabled)
-		break;
-	}
-
-	if (i == xf86_config->num_crtc) {
+	if (pRADEONEnt->Controller[0]->enabled &&
+	    pRADEONEnt->Controller[1]->enabled) {
 	    int cursor_end, frame_end;
 
 	    cursor_end = x - xorigin + w;
diff --git a/src/radeon_pm.c b/src/radeon_pm.c
index faf7af9..5267fbc 100644
--- a/src/radeon_pm.c
+++ b/src/radeon_pm.c
@@ -798,18 +798,10 @@ void RADEONPMFini(ScrnInfoPtr pScrn)
 
 void RADEONPMBlockHandler(ScrnInfoPtr pScrn)
 {
-    xf86CrtcConfigPtr xf86_config = XF86_CRTC_CONFIG_PTR(pScrn);
-    int i;
-
-    for (i = 0; i < xf86_config->num_crtc; i++) {
-	xf86CrtcPtr crtc = xf86_config->crtc[i];
-	RADEONCrtcPrivatePtr radeon_crtc = crtc->driver_private;
-
-	if (radeon_crtc->enabled)
-	    break;
-    }
+    RADEONEntPtr pRADEONEnt = RADEONEntPriv(pScrn);
 
-    if (i == xf86_config->num_crtc)
+    if ((!pRADEONEnt->Controller[0]->enabled) &&
+	(!pRADEONEnt->Controller[1]->enabled))
 	RADEONSetStaticPowerMode(pScrn, POWER_LOW);
     else
 	RADEONSetStaticPowerMode(pScrn, POWER_HIGH);


More information about the xorg-commit mailing list