[Openchrome-devel] xf86-video-openchrome: 9 commits - configure.ac src/via_display.c src/via_outputs.c src/via_vt1632.c

[Luc Verhaegen]

On Tue, Jun 28, 2016 at 04:08:05AM +0000, Kevin Brace wrote:
> commit ed68b1e9428b96b2c3ec4dc56625579bf531bc63
> Author: Kevin Brace <kevinbrace at gmx.com>
> Date:   Mon Jun 27 22:58:54 2016 -0500
> 
>     Using SR01[5] to turn off analog VGA output rather than CR47[2]
>     
>     CR47[2] (3X5.47[2]) is the ???back door??? register to turn off analog
>     VGA output. Will rather use the ???front door??? version of it located
>     at SR01[5] (3C5.01[5]).
>     
>     Signed-off-by: Kevin Brace <kevinbrace at gmx.com>
> 
> diff --git a/src/via_outputs.c b/src/via_outputs.c
> index bc516c8..e7f2ab7 100644
> --- a/src/via_outputs.c
> +++ b/src/via_outputs.c
> @@ -520,11 +520,10 @@ viaAnalogOutput(ScrnInfoPtr pScrn, Bool displayState)
>      DEBUG(xf86DrvMsg(pScrn->scrnIndex, X_INFO,
>                          "Entered viaAnalogOutput.\n"));
>  
> -    /* This register controls analog VGA DAC output state. */
> -    /* 3X5.47[2] - DACOFF Backdoor Register
> +    /* 3C5.01[5] - DACOFF Register
>       *             0: DAC on
>       *             1: DAC off */
> -    ViaCrtcMask(hwp, 0x47, displayState ? 0x00 : 0x04, 0x04);
> +    ViaSeqMask(hwp, 0x01, displayState ? 0x00 : 0x20, 0x20);
>      xf86DrvMsg(pScrn->scrnIndex, X_INFO,
>                  "Analog VGA Output: %s\n",
>                  displayState ? "On" : "Off");

You definitely are confusing the VGA chip standard with the VGA 
connection standard here.

Read up on what the vga sequencer does.

Luc Verhaegen.