[haiku-commits] r33815 - in haiku/trunk/src/add-ons: accelerants/intel_extreme kernel/busses/agp_gart kernel/drivers/graphics/intel_extreme

  • From: brecht@xxxxxxxxxxx
  • To: haiku-commits@xxxxxxxxxxxxx
  • Date: Wed, 28 Oct 2009 19:58:36 +0100 (CET)

Author: brecht
Date: 2009-10-28 19:58:36 +0100 (Wed, 28 Oct 2009)
New Revision: 33815
Changeset: http://dev.haiku-os.org/changeset/33815/haiku

Modified:
   haiku/trunk/src/add-ons/accelerants/intel_extreme/Jamfile
   haiku/trunk/src/add-ons/kernel/busses/agp_gart/Jamfile
   haiku/trunk/src/add-ons/kernel/busses/agp_gart/intel_gart.cpp
   haiku/trunk/src/add-ons/kernel/drivers/graphics/intel_extreme/Jamfile
   
haiku/trunk/src/add-ons/kernel/drivers/graphics/intel_extreme/intel_extreme.cpp
Log:
made code style more consistent with the rest of the driver


Modified: haiku/trunk/src/add-ons/accelerants/intel_extreme/Jamfile
===================================================================
--- haiku/trunk/src/add-ons/accelerants/intel_extreme/Jamfile   2009-10-28 
18:46:23 UTC (rev 33814)
+++ haiku/trunk/src/add-ons/accelerants/intel_extreme/Jamfile   2009-10-28 
18:58:36 UTC (rev 33815)
@@ -5,7 +5,6 @@
 UsePrivateHeaders graphics ;
 UsePrivateHeaders [ FDirName graphics intel_extreme ] ;
 UsePrivateHeaders [ FDirName graphics common ] ;
-UsePrivateHeaders shared ;
 
 Addon intel_extreme.accelerant :
        accelerant.cpp

Modified: haiku/trunk/src/add-ons/kernel/busses/agp_gart/Jamfile
===================================================================
--- haiku/trunk/src/add-ons/kernel/busses/agp_gart/Jamfile      2009-10-28 
18:46:23 UTC (rev 33814)
+++ haiku/trunk/src/add-ons/kernel/busses/agp_gart/Jamfile      2009-10-28 
18:58:36 UTC (rev 33815)
@@ -7,7 +7,6 @@
 UsePrivateHeaders [ FDirName graphics intel_extreme ] ;
 UsePrivateHeaders [ FDirName graphics common ] ;
 UsePrivateHeaders drivers graphics kernel ;
-UsePrivateHeaders shared ;
 
 KernelAddon <agp_gart>intel :
        intel_gart.cpp

Modified: haiku/trunk/src/add-ons/kernel/busses/agp_gart/intel_gart.cpp
===================================================================
--- haiku/trunk/src/add-ons/kernel/busses/agp_gart/intel_gart.cpp       
2009-10-28 18:46:23 UTC (rev 33814)
+++ haiku/trunk/src/add-ons/kernel/busses/agp_gart/intel_gart.cpp       
2009-10-28 18:58:36 UTC (rev 33815)
@@ -149,23 +149,21 @@
                                break;
                }
        } else if ((info.type & INTEL_TYPE_GROUP_MASK) == INTEL_TYPE_G4x) {
-               switch (memoryConfig & G4X_GGC_GGMS_MASK) {
-                       case G4X_GGMS_NONE:
+               switch (memoryConfig & G4X_GTT_MASK) {
+                       case G4X_GTT_NONE:
                                gttSize = 0;
                                break;
-                       case G4X_GGMS_NO_IVT_1M:
+                       case G4X_GTT_1M_NO_IVT:
                                gttSize = 1 << 20;
                                break;
-                       case G4X_GGMS_NO_IVT_2M:
+                       case G4X_GTT_2M_NO_IVT:
+                       case G4X_GTT_2M_IVT:
                                gttSize = 2 << 20;
                                break;
-                       case G4X_GGMS_IVT_2M:
-                               gttSize = 2 << 20;
-                               break;
-                       case G4X_GGMS_IVT_3M:
+                       case G4X_GTT_3M_IVT:
                                gttSize = 3 << 20;
                                break;
-                       case G4X_GGMS_IVT_4M:
+                       case G4X_GTT_4M_IVT:
                                gttSize = 4 << 20;
                                break;                  
                }
@@ -205,33 +203,6 @@
                                memorySize *= 8;
                                break;
                }
-       } else if (info.type == INTEL_TYPE_G4x) {
-               switch (memoryConfig & G4X_GGC_GMS_MASK) {
-                       case G4X_GMS_32MB:
-                               memorySize *= 32;
-                               break;
-                       case G4X_GMS_64MB:
-                               memorySize *= 64;
-                               break;
-                       case G4X_GMS_128MB:
-                               memorySize *= 128;
-                               break;
-                       case G4X_GMS_256MB:
-                               memorySize *= 256;
-                               break;
-                       case G4X_GMS_96MB:
-                               memorySize *= 96;
-                               break;
-                       case G4X_GMS_160MB:
-                               memorySize *= 160;
-                               break;
-                       case G4X_GMS_224MB:
-                               memorySize *= 224;
-                               break;
-                       case G4X_GMS_352MB:
-                               memorySize *= 352;
-                               break;
-               }
        } else if (info.type == INTEL_TYPE_85x
                || (info.type & INTEL_TYPE_9xx) == INTEL_TYPE_9xx) {
                switch (memoryConfig & STOLEN_MEMORY_MASK) {
@@ -259,6 +230,18 @@
                        case i855_STOLEN_MEMORY_256M:
                                memorySize *= 256;
                                break;
+                       case G4X_STOLEN_MEMORY_96MB:
+                               memorySize *= 96;
+                               break;
+                       case G4X_STOLEN_MEMORY_160MB:
+                               memorySize *= 160;
+                               break;
+                       case G4X_STOLEN_MEMORY_224MB:
+                               memorySize *= 224;
+                               break;
+                       case G4X_STOLEN_MEMORY_352MB:
+                               memorySize *= 352;
+                               break;
                }
        } else {
                // TODO: error out!
@@ -329,12 +312,13 @@
        if (get_memory_map(scratchAddress, B_PAGE_SIZE, &entry, 1) != B_OK)
                return B_ERROR;
 
-       if ((info.type & INTEL_TYPE_GROUP_MASK) == INTEL_TYPE_G4x)
-               info.gtt_physical_base = 
info.display.u.h0.base_registers[mmioIndex]
-                               + (2UL << 20);
-       else if ((info.type & INTEL_TYPE_FAMILY_MASK) == INTEL_TYPE_9xx)
-               info.gtt_physical_base = get_pci_config(info.display, 
i915_GTT_BASE, 4);
-       else {
+       if ((info.type & INTEL_TYPE_FAMILY_MASK) == INTEL_TYPE_9xx) {
+               if ((info.type & INTEL_TYPE_GROUP_MASK) == INTEL_TYPE_G4x) {
+                       info.gtt_physical_base = 
info.display.u.h0.base_registers[mmioIndex]
+                                       + (2UL << 20);
+               } else
+                       info.gtt_physical_base = get_pci_config(info.display, 
i915_GTT_BASE, 4);
+       } else {
                info.gtt_physical_base = read32(info.registers
                        + INTEL_PAGE_TABLE_CONTROL) & ~PAGE_TABLE_ENABLED;
                if (info.gtt_physical_base == 0) {

Modified: haiku/trunk/src/add-ons/kernel/drivers/graphics/intel_extreme/Jamfile
===================================================================
--- haiku/trunk/src/add-ons/kernel/drivers/graphics/intel_extreme/Jamfile       
2009-10-28 18:46:23 UTC (rev 33814)
+++ haiku/trunk/src/add-ons/kernel/drivers/graphics/intel_extreme/Jamfile       
2009-10-28 18:58:36 UTC (rev 33815)
@@ -5,7 +5,6 @@
 UsePrivateHeaders [ FDirName graphics intel_extreme ] ;
 UsePrivateHeaders [ FDirName graphics common ] ;
 UsePrivateHeaders graphics kernel ;
-UsePrivateHeaders shared ;
 
 KernelAddon intel_extreme :
        driver.cpp

Modified: 
haiku/trunk/src/add-ons/kernel/drivers/graphics/intel_extreme/intel_extreme.cpp
===================================================================
--- 
haiku/trunk/src/add-ons/kernel/drivers/graphics/intel_extreme/intel_extreme.cpp 
    2009-10-28 18:46:23 UTC (rev 33814)
+++ 
haiku/trunk/src/add-ons/kernel/drivers/graphics/intel_extreme/intel_extreme.cpp 
    2009-10-28 18:58:36 UTC (rev 33815)
@@ -247,13 +247,13 @@
        } else if (info.device_type.InGroup(INTEL_TYPE_G4x)) {
                dprintf("G4x clock gating\n");
                write32(info.registers + 0x6204, 0);
-               write32(info.registers + 0x6208, BIT(9) | BIT(7) | BIT(6));
+               write32(info.registers + 0x6208, (1L << 9) | (1L << 7) | (1L << 
6));
                write32(info.registers + 0x6210, 0);
 
-               uint32 dspclk_gate_val = BIT(28) | BIT(3) | BIT(2);
+               uint32 dspclk_gate_val = (1L << 28) | (1L << 3) | (1L << 2);
                if ((info.device_type.type & INTEL_TYPE_MOBILE) == 
INTEL_TYPE_MOBILE) {
                        dprintf("G4x mobile clock gating\n");
-                   dspclk_gate_val |= BIT(18);
+                   dspclk_gate_val |= 1L << 18;
                }
                write32(info.registers + 0x6200, dspclk_gate_val)       ;       
 


Other related posts: