[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 06:11:56PM +0200, Luc Verhaegen wrote:
> On Tue, Jun 28, 2016 at 05:18:35PM +0200, Kevin Brace wrote:
> > 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.
> 
> It does not.
> 
> It turns off the RAM readout of IGA1.
> 
> It does not turn off IGA1 (is that even possible?).
> It definitely does not turn off the DAC.
> 
> Luc Verhaegen.

And now include the ml as well.

Luc Verhaegen.