[ros-diffs] [aandrejevic] 59738: [NTVDM] Fix 16 color modes. Implement 4-bit interleaved shift mode.

Author: aandrejevic
Date: Wed Aug 14 17:17:43 2013
New Revision: 59738

URL: http://svn.reactos.org/svn/reactos?rev=59738&view=rev
Log:
[NTVDM]
Fix 16 color modes.
Implement 4-bit interleaved shift mode.


Modified:
    branches/ntvdm/subsystems/ntvdm/vga.c

Modified: branches/ntvdm/subsystems/ntvdm/vga.c
URL:
http://svn.reactos.org/svn/reactos/branches/ntvdm/subsystems/ntvdm/vga.c?re…
==============================================================================
--- branches/ntvdm/subsystems/ntvdm/vga.c	[iso-8859-1] (original)
+++ branches/ntvdm/subsystems/ntvdm/vga.c	[iso-8859-1] Wed Aug 14 17:17:43 2013
@@ -572,13 +572,30 @@
                 }
                 else if (VgaGcRegisters[VGA_GC_MODE_REG] & VGA_GC_MODE_SHIFTREG)
                 {
-                    /*
-                     * 2 bits shifted from plane 0 and 2 for the first 4 pixels,
-                     * then 2 bits shifted from plane 1 and 3 for the next 4
-                     */
-
-                    // TODO: NOT IMPLEMENTED!
-                    DPRINT1("Interleaved shift mode is not implemented!\n");
+                    /* Check if this is 16 or 256 color mode */
+                    if (VgaAcRegisters[VGA_AC_CONTROL_REG] & VGA_AC_CONTROL_8BIT)
+                    {
+                        /*
+                         * 2 bits shifted from plane 0 and 2 for the first 4 pixels,
+                         * then 2 bits shifted from plane 1 and 3 for the next 4
+                         */
+                        BYTE LowPlaneData = VgaMemory[((j / 4) % 2) * VGA_BANK_SIZE
+                                                      + (Address + (j / 4)) *
AddressSize];
+                        BYTE HighPlaneData = VgaMemory[(((j / 4) % 2) + 1) *
VGA_BANK_SIZE
+                                                       + (Address + (j / 4)) *
AddressSize];
+
+                        /* Extract the two bits from each plane */
+                        LowPlaneData = (LowPlaneData >> (6 - ((j % 4) * 2))) &
3;
+                        HighPlaneData = (HighPlaneData >> (6 - ((j % 4) * 2)))
& 3;
+
+                        /* Combine them into the pixel */
+                        PixelData = LowPlaneData | (HighPlaneData << 2);
+                    }
+                    else
+                    {
+                        // TODO: NOT IMPLEMENTED
+                        DPRINT1("8-bit interleaved mode is not
implemented!\n");
+                    }
                 }
                 else
                 {
@@ -618,6 +635,12 @@
                             if (PlaneData & (1 << (7 - (j % 8)))) PixelData |=
1 << k;
                         }
                     }
+                }
+
+                if (!(VgaAcRegisters[VGA_AC_CONTROL_REG] & VGA_AC_CONTROL_8BIT))
+                {
+                    /* In 16 color mode, the value is an index to the AC registers */
+                    PixelData = VgaAcRegisters[PixelData];
                 }
 
                 /* Now check if the resulting pixel data has changed */