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

[Kevin Brace]

Hi Luc,

> 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.
> 

The commit appears to be working.
I tested it with CLE266, VX700, and VN896 chipsets.
SR01[5] does turn off IGA1 (display controller 1).
I also experimented this with via_regs_dump tool.

Regards,

Kevin Brace