Remove the DPMS code in creator_blank_display(), as it causes some

LCDs to blink in the V_DISPLAY_ON case, at least in combination with
some 13W3-VGA-adaptors (what's exactly going on is unclear though,
as it happens when all of H-sync, V-sync and video output are enabled
and not touching the sync bits from the preset fixes it). Thus
creator_blank_display() now is reduced to turning the video output
on/off.
Although that DPMS code did what the XFree86/Xorg sunffb(4x) does,
it was questionable in the first place, as both implementations
also turn(ed) off the video output on standby and suspend, thus most
likely causing the monitor to turn off instead of entering standby
or suspend as intended (at least my monitors don't).

Reported and tested by:	Patrick Reich
MFC after:		3 days
This commit is contained in:
Marius Strobl 2006-08-24 22:00:24 +00:00
parent 8788b6eb7a
commit 2afacffb3a
Notes: svn2git 2020-12-20 02:59:44 +00:00
svn path=/head/; revision=161590

View File

@ -551,20 +551,11 @@ creator_blank_display(video_adapter_t *adp, int mode)
v = FFB_READ(sc, FFB_DAC, FFB_DAC_VALUE);
switch (mode) {
case V_DISPLAY_ON:
v &= ~(FFB_DAC_CFG_TGEN_VSD | FFB_DAC_CFG_TGEN_HSD);
v |= FFB_DAC_CFG_TGEN_VIDE;
break;
case V_DISPLAY_BLANK:
v |= (FFB_DAC_CFG_TGEN_VSD | FFB_DAC_CFG_TGEN_HSD);
v &= ~FFB_DAC_CFG_TGEN_VIDE;
break;
case V_DISPLAY_STAND_BY:
v &= ~FFB_DAC_CFG_TGEN_VSD;
v &= ~FFB_DAC_CFG_TGEN_VIDE;
break;
case V_DISPLAY_SUSPEND:
v |= FFB_DAC_CFG_TGEN_VSD;
v &= ~FFB_DAC_CFG_TGEN_HSD;
v &= ~FFB_DAC_CFG_TGEN_VIDE;
break;
}