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

Bug#180938: marked as done (xserver-xfree86: [ati/radeon] no DPMS support on digital flat panels)



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.
branden@debian.org                 |      -- Napoleon Bonaparte
http://people.debian.org/~branden/ |

Attachment: signature.asc
Description: Digital signature


Reply to: