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

1 files changed, 0 insertions, 9 deletions

diff --git a/sys/dev/fb/creator.c b/sys/dev/fb/creator.c
index 288e4dd..bfcf2bc 100644
--- a/sys/dev/fb/creator.c
+++ b/sys/dev/fb/creator.c
@@ -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;
 	}