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

[Kevin Brace]

 configure.ac      |    2 +-
 src/via_display.c |   13 ++++++++++++-
 src/via_outputs.c |    6 ------
 3 files changed, 13 insertions(+), 8 deletions(-)

New commits:
commit 9877810ccefc2258aafd1f1fe95654e5cb788496
Author: Kevin Brace <kevinbrace at gmx.com>
Date:   Fri Jun 10 19:07:50 2016 -0500

    Version bumped to 0.4.177
    
    Signed-off-by: Kevin Brace <kevinbrace at gmx.com>

diff --git a/configure.ac b/configure.ac
index 7bbde22..77d2017 100644
--- a/configure.ac
+++ b/configure.ac
@@ -23,7 +23,7 @@
 # Initialize Autoconf
 AC_PREREQ(2.57)
 AC_INIT([xf86-video-openchrome],
-        [0.4.176],
+        [0.4.177],
         [https://bugs.freedesktop.org/enter_bug.cgi?product=xorg&component=Driver/openchrome],
         [xf86-video-openchrome])
 
commit e60ca86f6b5465976458ba955a532580695a3ef8
Author: Kevin Brace <kevinbrace at gmx.com>
Date:   Fri Jun 10 19:01:44 2016 -0500

    Activating VT1632A external TMDS transmitter (DVI) detection
    
    While the code may not work, it was determined that the detection of
    it should be enabled at this point. It is unlikely that it will work
    for now, but if it does not, the code will be fixed in the near future.
    
    Signed-off-by: Kevin Brace <kevinbrace at gmx.com>

diff --git a/src/via_outputs.c b/src/via_outputs.c
index 8e48131..7e21e8c 100644
--- a/src/via_outputs.c
+++ b/src/via_outputs.c
@@ -1259,14 +1259,8 @@ viaOutputDetect(ScrnInfoPtr pScrn)
     /* TV */
     via_tv_init(pScrn);
 
-    /*
-     * via_dvi_init function to initialize VT1632A TMDS transmitter
-     * is disabled for now.
-     */
     /* External TMDS Transmitter (DVI) */
-/*
     via_dvi_init(pScrn);
-*/
 
     DEBUG(xf86DrvMsg(pScrn->scrnIndex, X_INFO,
                         "Exiting viaOutputDetect.\n"));
commit c768c86d044c84504f2b747db2b306c1f041ba44
Author: Kevin Brace <kevinbrace at gmx.com>
Date:   Fri Jun 10 02:04:11 2016 -0500

    Limiting IGA1 horizontal total to 2080.
    
    Previous code was faulty, so it was updated to keep hardware
    limitation in mind.
    
    Signed-off-by: Kevin Brace <kevinbrace at gmx.com>

diff --git a/src/via_display.c b/src/via_display.c
index 79163b2..e58d7dc 100644
--- a/src/via_display.c
+++ b/src/via_display.c
@@ -1222,7 +1222,18 @@ viaIGA1ModeValid(ScrnInfoPtr pScrn, DisplayModePtr mode)
     DEBUG(xf86DrvMsg(pScrn->scrnIndex, X_INFO,
                         "Entered viaIGA1ModeValid.\n"));
 
-    if (mode->CrtcHTotal > 4100)
+    /* Note that horizontal total being written to VGA registers is
+     * shifted to the right by 3 bit positions, and then 5 is subtracted
+     * from it. Hence, to check if the screen can even be valid,
+     * opposite of that needs to happen. That being said, to check if the
+     * number is within an acceptable range, 1 is subtracted from 5, hence,
+     * 4 (5 - 1) is multiplied with 8 (i.e., 1 is shifted 3 bit positions to
+     * the left) and the resulting 32 is added to 2048 to calculate the
+     * maximum horizontal total IGA1 can handle. Ultimately, 2080 is the
+     * largest number VIA IGP's IGA1 can handle safely. This is how
+     * VIA Technologies gets to claim that IGA1 (i.e., VGA) is capable of
+     * 1920 dots in the horizontal (Y) direction. */
+    if (mode->CrtcHTotal > (2048 + ((1 << 3) * 4)))
         return MODE_BAD_HVALUE;
 
     if (mode->CrtcHDisplay > 2048)