On Thu, Mar 25, 2004 at 07:48:05AM -0800, Debian Bug Tracking System wrote:
> *** xc/programs/Xserver/hw/xfree86/drivers/ati/radeon_driver.c.orig   Sun Feb 
>  9 20:19:34 2003
> --- xc/programs/Xserver/hw/xfree86/drivers/ati/radeon_driver.c        Sun Feb 
>  9 21:50:06 2003
> ***************
> *** 4579,4582 ****
> --- 4579,4594 ----
>       OUTREGP(RADEON_CRTC_EXT_CNTL, mask, ~mask);
>       break;
>       }
> +     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;
> +       }
> +     }
>   }
[...]
> This bug report was fixed in very early versions of 4.3.x -
> 4.3.0-0ds3v2, I believe.

The DPMS code in radeon_driver.c doesn't look much like the above
anymore.  However, it does appear to handle DPMSModeOn and DPMSModeOff
for info->DisplayType == MT_DFP, as the above does, so I concur with the
closing.

It does continue to have a comment:

/* TODO: additional handling for LCD ? */

Bug submitter, please feel free to followup to this bug and confirm that
you see the fix.  You will need to use xserver-xfree86 4.3.0-7 from
unstable, of course.

-- 
G. Branden Robinson                |      Religion is excellent stuff for
Debian GNU/Linux                   |      keeping common people quiet.
[EMAIL PROTECTED]                 |      -- Napoleon Bonaparte
http://people.debian.org/~branden/ |

Attachment: signature.asc
Description: Digital signature

Reply via email to