Bug 23677 – XFree patch to enable dpms support for ati radeon dvi

• Product: Gentoo Linux
• Status: RESOLVED INVALID

[Description Arthur Britto 2003-06-29 01:09:02 UTC]

This patch adds support for dpms for dvi connections with ati radeon.

I have been successfully using this patch for many months.

Please add this patch to the xfree ebuild.

The source of this patch is:
http://www.spinics.net/lists/xf-xpert/msg08120.html

This is the patch applied to xfree-4.3.0-r3:
---
/var/tmp/portage/xfree-4.3.0-r3/work/xc/programs/Xserver/hw/xfree86/drivers/ati/radeon_driver.c.orig
       2003-06-28 21:51:51.000000000 -0700
+++
/var/tmp/portage/xfree-4.3.0-r3/work/xc/programs/Xserver/hw/xfree86/drivers/ati/radeon_driver.c
    2003-06-28 21:48:18.000000000 -0700
@@ -6624,6 +6624,19 @@
        }
     }

+    if (info->DisplayType == MT_DFP) {
+       switch (PowerManagementMode) {
+       case DPMSModeOn:
+           OUTREG(RADEON_FP_GEN_CNTL, INREG(RADEON_FP_GEN_CNTL) |
(RADEON_FP_FPON | RADEON_FP_TMDS_EN));
+           break;
+       case DPMSModeStandby:
+       case DPMSModeSuspend:
+       case DPMSModeOff:
+           OUTREG(RADEON_FP_GEN_CNTL, INREG(RADEON_FP_GEN_CNTL) &
~(RADEON_FP_FPON | RADEON_FP_TMDS_EN));
+           break;
+       }
+    }
+
 #ifdef XF86DRI
     if (info->CPStarted) DRIUnlock(pScrn->pScreen);
 #endif


Reproducible: Always
Steps to Reproduce:
Using dvi connection with ati radeon, this command fails:
   xset dpms force off

Actual Results:  
Nothing.

Expected Results:  
Monitor should power down.

[Comment 1 Donnie Berkholz (RETIRED) 2003-08-11 23:37:29 UTC]

Created attachment 15959 [details, diff]
Current patch with the same purpose

Hi Arthur,
We apply a patch that's quite similar to yours now. Do things work currently
without using your patch, just ours?

[Comment 2 Arthur Britto 2003-08-20 14:56:38 UTC]

I tested xfree-4.3.0-r3 and dpms for dvi with ati radeon appears to work just fine without the patch I suggested.

Thanks! :)

P.S. I am fairly sure at the time I submited this bug, this was not the case.

[Comment 3 Donnie Berkholz (RETIRED) 2003-08-20 20:37:55 UTC]

Closing since we already have a working patch.