[ros-diffs] [jgardou] 60066: [VGA_NEW] - Lessons learnt: the more trivial, the more suspicious. Fix build, and remind that MSDN is not always right.

Author: jgardou
Date: Thu Sep 12 21:22:42 2013
New Revision: 60066

URL: http://svn.reactos.org/svn/reactos?rev=60066&view=rev
Log:
[VGA_NEW]
 - Lessons learnt: the more trivial, the more suspicious.
Fix build, and remind that MSDN is not always right.

Modified:
    trunk/reactos/win32ss/drivers/miniport/vga_new/modeset.c
    trunk/reactos/win32ss/drivers/miniport/vga_new/vbe.c
    trunk/reactos/win32ss/drivers/miniport/vga_new/vbemodes.c
    trunk/reactos/win32ss/drivers/miniport/vga_new/vga.c

Modified: trunk/reactos/win32ss/drivers/miniport/vga_new/modeset.c
URL:
http://svn.reactos.org/svn/reactos/trunk/reactos/win32ss/drivers/miniport/v…
==============================================================================
--- trunk/reactos/win32ss/drivers/miniport/vga_new/modeset.c	[iso-8859-1] (original)
+++ trunk/reactos/win32ss/drivers/miniport/vga_new/modeset.c	[iso-8859-1] Thu Sep 12
21:22:42 2013
@@ -470,7 +470,7 @@
     
     if (pRequestedMode->hres == 320)
     {
-        VideoDebugPrint(0, "ModeX not support!!!\n");
+        VideoDebugPrint((0, "ModeX not support!!!\n"));
         return ERROR_INVALID_PARAMETER;
     }
 // eVb: 2.7 [END]
@@ -481,7 +481,7 @@
     if (!(pRequestedMode->fbType & VIDEO_MODE_GRAPHICS))
     {
 // eVb: 2.8 [TODO] - This code path is not implemented yet
-        VideoDebugPrint(0, "Text-mode not support!!!\n");
+        VideoDebugPrint((0, "Text-mode not support!!!\n"));
         return ERROR_INVALID_PARAMETER;
 // eVb: 2.8 [END]
     }

Modified: trunk/reactos/win32ss/drivers/miniport/vga_new/vbe.c
URL:
http://svn.reactos.org/svn/reactos/trunk/reactos/win32ss/drivers/miniport/v…
==============================================================================
--- trunk/reactos/win32ss/drivers/miniport/vga_new/vbe.c	[iso-8859-1] (original)
+++ trunk/reactos/win32ss/drivers/miniport/vga_new/vbe.c	[iso-8859-1] Thu Sep 12 21:22:42
2013
@@ -44,7 +44,7 @@
     CHAR Version[21];
 
     /* If the broken VESA bios found, turn VESA off */
-    VideoDebugPrint(0, "Vendor: %s Product: %s Revision: %s (%lx)\n", Vendor,
Product, Revision, OemRevision);
+    VideoDebugPrint((0, "Vendor: %s Product: %s Revision: %s (%lx)\n", Vendor,
Product, Revision, OemRevision));
     for (i = 0; i < (sizeof(BrokenVesaBiosList) / sizeof(PCHAR)); i++)
     {
         if (!strncmp(Product, BrokenVesaBiosList[i], strlen(BrokenVesaBiosList[i])))
return FALSE;
@@ -69,7 +69,7 @@
     }
 
     /* VESA ok */
-    VideoDebugPrint(0, "Vesa ok\n");
+    VideoDebugPrint((0, "Vesa ok\n"));
     return TRUE;
 }
 
@@ -153,7 +153,7 @@
 
     Entries = ClutBuffer->NumEntries;
     
-    VideoDebugPrint(0, "Setting %lu entries.\n", Entries);
+    VideoDebugPrint((0, "Setting %lu entries.\n", Entries));
     
     /* 
      * For Vga compatible modes, write them directly.
@@ -209,7 +209,7 @@
     Status = VgaExtension->Int10Interface.Int10CallBios(Context, &BiosArguments);
     if (Status != NO_ERROR) return ERROR_INVALID_PARAMETER;
     VideoPortFreePool(VgaExtension, VesaClut);
-    VideoDebugPrint(Error, "VBE Status: %lx\n", BiosArguments.Eax);
+    VideoDebugPrint((Error, "VBE Status: %lx\n", BiosArguments.Eax));
     if (VBE_GETRETURNCODE(BiosArguments.Eax) == VBE_SUCCESS)
         return NO_ERROR;
     return ERROR_INVALID_PARAMETER;

Modified: trunk/reactos/win32ss/drivers/miniport/vga_new/vbemodes.c
URL:
http://svn.reactos.org/svn/reactos/trunk/reactos/win32ss/drivers/miniport/v…
==============================================================================
--- trunk/reactos/win32ss/drivers/miniport/vga_new/vbemodes.c	[iso-8859-1] (original)
+++ trunk/reactos/win32ss/drivers/miniport/vga_new/vbemodes.c	[iso-8859-1] Thu Sep 12
21:22:42 2013
@@ -98,12 +98,12 @@
     ModeIndex = VgaMode->Mode;
     BiosArguments.Eax = VBE_SET_VBE_MODE;
     BiosArguments.Ebx = HIWORD(ModeIndex);
-    VideoDebugPrint(0, "Switching to %lx %lx\n", BiosArguments.Eax,
BiosArguments.Ebx);
+    VideoDebugPrint((0, "Switching to %lx %lx\n", BiosArguments.Eax,
BiosArguments.Ebx));
     Status = VideoPortInt10(VgaDeviceExtension, &BiosArguments);
     if (Status != NO_ERROR) return Status;
     if(VBE_GETRETURNCODE(BiosArguments.Eax) != VBE_SUCCESS)
     {
-        VideoDebugPrint(0, "Changing VBE mode failed, Eax %lx",
BiosArguments.Eax);
+        VideoDebugPrint((0, "Changing VBE mode failed, Eax %lx",
BiosArguments.Eax));
         return ERROR_INVALID_PARAMETER;
     }
 
@@ -202,7 +202,7 @@
     }
 
     /* Add ref */
-    VideoDebugPrint(0, "have int10 iface\n");
+    VideoDebugPrint((0, "have int10 iface\n"));
    
VgaExtension->Int10Interface.InterfaceReference(VgaExtension->Int10Interface.Context);
     Context = VgaExtension->Int10Interface.Context;
 
@@ -217,7 +217,7 @@
     if (!VbeInfo) return;
 
     /* Init VBE data and write to card buffer */
-    VideoDebugPrint(0, "have int10 data\n");
+    VideoDebugPrint((0, "have int10 data\n"));
     VbeInfo->ModeArray[128] = 0xFFFF;
     VbeInfo->Info.Signature = VBE2_MAGIC;
     Status = VgaExtension->Int10Interface.Int10WriteMemory(Context,
@@ -236,7 +236,7 @@
     if (Status != NO_ERROR) return;
     if(VBE_GETRETURNCODE(BiosArguments.Eax) != VBE_SUCCESS)
     {
-        VideoDebugPrint(0, "BiosArguments.Eax %lx\n", BiosArguments.Eax);
+        VideoDebugPrint((0, "BiosArguments.Eax %lx\n", BiosArguments.Eax));
         return;
     }
     Status = VgaExtension->Int10Interface.Int10ReadMemory(Context,
@@ -247,21 +247,21 @@
     if (Status != NO_ERROR) return;
 
     /* Check correct VBE BIOS */
-    VideoDebugPrint(0, "have vbe data\n");
+    VideoDebugPrint((0, "have vbe data\n"));
     TotalMemory = VbeInfo->Info.TotalMemory << 16;
     VbeVersion = VbeInfo->Info.Version;
-    VideoDebugPrint(0, "vbe version %lx memory %lx\n", VbeVersion,
TotalMemory);
+    VideoDebugPrint((0, "vbe version %lx memory %lx\n", VbeVersion,
TotalMemory));
     if (!ValidateVbeInfo(VgaExtension, VbeInfo)) return;
 
     /* Read modes */
-    VideoDebugPrint(0, "read modes from %p\n", VbeInfo->Info.VideoModePtr);
+    VideoDebugPrint((0, "read modes from %p\n",
VbeInfo->Info.VideoModePtr));
     Status = VgaExtension->Int10Interface.Int10ReadMemory(Context,
                                                          
HIWORD(VbeInfo->Info.VideoModePtr),
                                                          
LOWORD(VbeInfo->Info.VideoModePtr),
                                                           VbeInfo->ModeArray,
                                                           128 * sizeof(USHORT));
     if (Status != NO_ERROR) return;
-    VideoDebugPrint(0, "Read modes at: %p\n", VbeInfo->ModeArray);
+    VideoDebugPrint((0, "Read modes at: %p\n", VbeInfo->ModeArray));
 
     /* Count modes, check for new 4bpp SVGA modes */
     ThisMode = VbeInfo->ModeArray;
@@ -269,7 +269,7 @@
     while (ModeResult != 0xFFFF)
     {
         Mode = ModeResult & 0x1FF;
-        VideoDebugPrint(0, "Mode found: %lx\n", Mode);
+        VideoDebugPrint((0, "Mode found: %lx\n", Mode));
         if ((Mode == 0x102) || (Mode == 0x6A)) FourBppModeFound = TRUE;
         ModeResult = *++ThisMode;
         NewModes++;
@@ -287,19 +287,19 @@
     /* Apply fixup for Intel Brookdale */
     if (g_bIntelBrookdaleBIOS)
     {
-        VideoDebugPrint(0, "Intel Brookdale-G Video BIOS Not Support!\n");
+        VideoDebugPrint((0, "Intel Brookdale-G Video BIOS Not Support!\n"));
         while (TRUE);
     }
 
     /* Scan SVGA modes */
-    VideoDebugPrint(0, "Static modes: %d\n", NumVideoModes);
+    VideoDebugPrint((0, "Static modes: %d\n", NumVideoModes));
     VgaMode = &VgaModeList[NumVideoModes];
     ThisMode = VbeInfo->ModeArray;
-    VideoDebugPrint(0, "new modes: %d\n", NewModes);
+    VideoDebugPrint((0, "new modes: %d\n", NewModes));
     while (NewModes--)
     {
         /* Get info on mode */
-        VideoDebugPrint(0, "Getting info of mode %lx.\n", *ThisMode);
+        VideoDebugPrint((0, "Getting info of mode %lx.\n", *ThisMode));
         VideoPortZeroMemory(&BiosArguments, sizeof(BiosArguments));
         BiosArguments.Eax = VBE_GET_MODE_INFORMATION;
         BiosArguments.Ecx = *ThisMode;
@@ -316,7 +316,7 @@
         if (Status != NO_ERROR) goto Next;
 
         /* Parse graphics modes only if linear framebuffer support */
-        VideoDebugPrint(0, "attr: %lx\n", VbeModeInfo->ModeAttributes);
+        VideoDebugPrint((0, "attr: %lx\n", VbeModeInfo->ModeAttributes));
         if (!(VbeModeInfo->ModeAttributes & (VBE_MODEATTR_VALID |
                                              VBE_MODEATTR_GRAPHICS))) goto Next;
         LinearAddressing = ((VbeVersion >= 0x200) &&
@@ -325,7 +325,7 @@
                             TRUE : FALSE;
 
         /* Check SVGA modes if 8bpp or higher */
-        VideoDebugPrint(0, "PhysBase: %lx\n", VbeModeInfo->PhysBasePtr);
+        VideoDebugPrint((0, "PhysBase: %lx\n", VbeModeInfo->PhysBasePtr));
         if ((VbeModeInfo->XResolution >= 640) &&
             (VbeModeInfo->YResolution >= 480) &&
             (VbeModeInfo->NumberOfPlanes >= 1) &&
@@ -339,7 +339,7 @@
             VgaMode->Frequency = 1;
             VgaMode->Mode = (*ThisMode << 16) | VBE_SET_VBE_MODE;
             VgaMode->Granularity = VbeModeInfo->WinGranularity << 10;
-            VideoDebugPrint(0, "Mode %lx (Granularity %d)\n", VgaMode->Mode,
VgaMode->Granularity);
+            VideoDebugPrint((0, "Mode %lx (Granularity %d)\n",
VgaMode->Mode, VgaMode->Granularity));
 
             /* Set flags */
             if (VbeModeInfo->ModeAttributes & VBE_MODEATTR_COLOR)
VgaMode->fbType |= VIDEO_MODE_COLOR;
@@ -349,12 +349,12 @@
             /* If no char data, say 80x25 */
             VgaMode->col = VbeModeInfo->XCharSize ? VbeModeInfo->XResolution /
VbeModeInfo->XCharSize : 80;
             VgaMode->row = VbeModeInfo->YCharSize ? VbeModeInfo->YResolution /
VbeModeInfo->YCharSize : 25;
-            VideoDebugPrint(0, "%d by %d rows\n", VgaMode->col,
VgaMode->row);
+            VideoDebugPrint((0, "%d by %d rows\n", VgaMode->col,
VgaMode->row));
 
             /* Check RGB555 (15bpp only) */
             VgaMode->bitsPerPlane = VbeModeInfo->BitsPerPixel /
VbeModeInfo->NumberOfPlanes;
             if ((VgaMode->bitsPerPlane == 16) &&
(VbeModeInfo->GreenMaskSize == 5)) VgaMode->bitsPerPlane = 15;
-            VideoDebugPrint(0, "BPP: %d\n", VgaMode->bitsPerPlane);
+            VideoDebugPrint((0, "BPP: %d\n", VgaMode->bitsPerPlane));
 
             /* Do linear or banked frame buffers */
             VgaMode->FrameBufferBase = 0;
@@ -364,18 +364,18 @@
                 ScreenStride = RaiseToPower2(VbeModeInfo->BytesPerScanLine);
                 //ASSERT(ScreenStride <= MAX_USHORT);
                 VgaMode->wbytes = (USHORT)ScreenStride;
-                VideoDebugPrint(0, "ScanLines: %lx Stride: %lx\n",
VbeModeInfo->BytesPerScanLine, VgaMode->wbytes);
+                VideoDebugPrint((0, "ScanLines: %lx Stride: %lx\n",
VbeModeInfo->BytesPerScanLine, VgaMode->wbytes));
 
                 /* Size of frame buffer is Height X ScanLine, align to bank/page size */
                 ScreenSize = VgaMode->hres * ScreenStride;
-                VideoDebugPrint(0, "Size: %lx\n", ScreenSize);
+                VideoDebugPrint((0, "Size: %lx\n", ScreenSize));
                 Size = (ScreenSize + ((64 * 1024) - 1)) & ((64 * 1024) - 1);
-                VideoDebugPrint(0, "Size: %lx\n", ScreenSize);
+                VideoDebugPrint((0, "Size: %lx\n", ScreenSize));
                 if (Size > TotalMemory) Size = (Size + ((4 * 1024) - 1)) & ((4 *
1024) - 1);
-                VideoDebugPrint(0, "Size: %lx\n", ScreenSize);
+                VideoDebugPrint((0, "Size: %lx\n", ScreenSize));
 
                 /* Banked VGA at 0xA0000 (64K) */
-                VideoDebugPrint(0, "Final size: %lx\n", Size);
+                VideoDebugPrint((0, "Final size: %lx\n", Size));
                 VgaMode->fbType |= VIDEO_MODE_BANKED;
                 VgaMode->sbytes = Size;
                 VgaMode->PhysSize = 64 * 1024;
@@ -387,20 +387,20 @@
             else
             {
                 /* VBE 3.00+ has specific field, read legacy field if not */
-                VideoDebugPrint(0, "LINEAR MODE!!!\n");
+                VideoDebugPrint((0, "LINEAR MODE!!!\n"));
                 ScreenStride = (VbeVersion >= 0x300) ?
VbeModeInfo->LinBytesPerScanLine : 0;
                 if (!ScreenStride) ScreenStride = VbeModeInfo->BytesPerScanLine;
                 //ASSERT(ScreenStride <= MAX_USHORT);
                 VgaMode->wbytes = (USHORT)ScreenStride;
-                VideoDebugPrint(0, "ScanLines: %lx Stride: %lx\n",
VbeModeInfo->BytesPerScanLine, VgaMode->wbytes);
+                VideoDebugPrint((0, "ScanLines: %lx Stride: %lx\n",
VbeModeInfo->BytesPerScanLine, VgaMode->wbytes));
 
                 /* Size of frame buffer is Height X ScanLine, align to page size */
                 ScreenSize = VgaMode->hres * LOWORD(VgaMode->wbytes);
-                VideoDebugPrint(0, "Size: %lx\n", ScreenSize);
+                VideoDebugPrint((0, "Size: %lx\n", ScreenSize));
                 Size = RaiseToPower2Ulong(ScreenSize);
-                VideoDebugPrint(0, "Size: %lx\n", ScreenSize);
+                VideoDebugPrint((0, "Size: %lx\n", ScreenSize));
                 if (Size > TotalMemory) Size = (Size + ((4 * 1024) - 1)) & ((4 *
1024) - 1);
-                VideoDebugPrint(0, "Size: %lx\n", ScreenSize);
+                VideoDebugPrint((0, "Size: %lx\n", ScreenSize));
 
                 /* Linear VGA must read settings from VBE */
                 VgaMode->fbType |= VIDEO_MODE_LINEAR;
@@ -431,7 +431,7 @@
     }
 
     /* Check if last mode was color to do test */
-    VideoDebugPrint(0, "mode scan complete. Total modes: %d\n", ModeCount);
+    VideoDebugPrint((0, "mode scan complete. Total modes: %d\n", ModeCount));
     if (--VgaMode->fbType & VIDEO_MODE_COLOR)
     {
         /* Try map physical buffer and free if worked */

Modified: trunk/reactos/win32ss/drivers/miniport/vga_new/vga.c
URL:
http://svn.reactos.org/svn/reactos/trunk/reactos/win32ss/drivers/miniport/v…
==============================================================================
--- trunk/reactos/win32ss/drivers/miniport/vga_new/vga.c	[iso-8859-1] (original)
+++ trunk/reactos/win32ss/drivers/miniport/vga_new/vga.c	[iso-8859-1] Thu Sep 12 21:22:42
2013
@@ -478,7 +478,7 @@
         return ERROR_INVALID_PARAMETER;
     }
 
-    VideoDebugPrint(0, "vga mapped at %x\n",
hwDeviceExtension->VideoMemoryAddress);
+    VideoDebugPrint((0, "vga mapped at %x\n",
hwDeviceExtension->VideoMemoryAddress));
 #endif
 // eVb: 1.7 [VDM] - Disable VDM for now
     ConfigInfo->VdmPhysicalVideoMemoryAddress.QuadPart = 0;
@@ -1508,7 +1508,7 @@
 --*/
 
 {
-    VideoDebugPrint(Error, "Detected internal VGA chip on embedded board,
todo\n");
+    VideoDebugPrint((Error, "Detected internal VGA chip on embedded board,
todo\n"));
     while (TRUE);
     return NO_ERROR;