1 files changed, 244 insertions, 267 deletions

diff --git a/drivers/char/drm/mga_dma.c b/drivers/char/drm/mga_dma.c
index c8e1b6c83636..70dc7f64b7b9 100644
--- a/drivers/char/drm/mga_dma.c
+++ b/drivers/char/drm/mga_dma.c
@@ -28,7 +28,7 @@
 /**
  * \file mga_dma.c
  * DMA support for MGA G200 / G400.
- * 
+ *
  * \author Rickard E. (Rik) Faith <faith@valinux.com>
  * \author Jeff Hartmann <jhartmann@valinux.com>
  * \author Keith Whitwell <keith@tungstengraphics.com>
@@ -44,40 +44,40 @@
 #define MGA_DEFAULT_USEC_TIMEOUT        10000
 #define MGA_FREELIST_DEBUG              0
 
-static int mga_do_cleanup_dma( drm_device_t *dev );
+static int mga_do_cleanup_dma(drm_device_t * dev);
 
 /* ================================================================
  * Engine control
  */
 
-int mga_do_wait_for_idle( drm_mga_private_t *dev_priv )
+int mga_do_wait_for_idle(drm_mga_private_t * dev_priv)
 {
         u32 status = 0;
         int i;
-        DRM_DEBUG( "\n" );
+        DRM_DEBUG("\n");
 
-        for ( i = 0 ; i < dev_priv->usec_timeout ; i++ ) {
-                status = MGA_READ( MGA_STATUS ) & MGA_ENGINE_IDLE_MASK;
-                if ( status == MGA_ENDPRDMASTS ) {
-                        MGA_WRITE8( MGA_CRTC_INDEX, 0 );
+        for (i = 0; i < dev_priv->usec_timeout; i++) {
+                status = MGA_READ(MGA_STATUS) & MGA_ENGINE_IDLE_MASK;
+                if (status == MGA_ENDPRDMASTS) {
+                        MGA_WRITE8(MGA_CRTC_INDEX, 0);
                         return 0;
                 }
-                DRM_UDELAY( 1 );
+                DRM_UDELAY(1);
         }
 
 #if MGA_DMA_DEBUG
-        DRM_ERROR( "failed!\n" );
-        DRM_INFO( "   status=0x%08x\n", status );
+        DRM_ERROR("failed!\n");
+        DRM_INFO("   status=0x%08x\n", status);
 #endif
         return DRM_ERR(EBUSY);
 }
 
-static int mga_do_dma_reset( drm_mga_private_t *dev_priv )
+static int mga_do_dma_reset(drm_mga_private_t * dev_priv)
 {
         drm_mga_sarea_t *sarea_priv = dev_priv->sarea_priv;
         drm_mga_primary_buffer_t *primary = &dev_priv->prim;
 
-        DRM_DEBUG( "\n" );
+        DRM_DEBUG("\n");
 
         /* The primary DMA stream should look like new right about now.
          */
@@ -100,24 +100,25 @@ static int mga_do_dma_reset( drm_mga_private_t *dev_priv )
  * Primary DMA stream
  */
 
-void mga_do_dma_flush( drm_mga_private_t *dev_priv )
+void mga_do_dma_flush(drm_mga_private_t * dev_priv)
 {
         drm_mga_primary_buffer_t *primary = &dev_priv->prim;
         u32 head, tail;
         u32 status = 0;
         int i;
-        DMA_LOCALS;
-        DRM_DEBUG( "\n" );
-
-        /* We need to wait so that we can do an safe flush */
-        for ( i = 0 ; i < dev_priv->usec_timeout ; i++ ) {
-                status = MGA_READ( MGA_STATUS ) & MGA_ENGINE_IDLE_MASK;
-                if ( status == MGA_ENDPRDMASTS ) break;
-                DRM_UDELAY( 1 );
+        DMA_LOCALS;
+        DRM_DEBUG("\n");
+
+        /* We need to wait so that we can do an safe flush */
+        for (i = 0; i < dev_priv->usec_timeout; i++) {
+                status = MGA_READ(MGA_STATUS) & MGA_ENGINE_IDLE_MASK;
+                if (status == MGA_ENDPRDMASTS)
+                        break;
+                DRM_UDELAY(1);
         }
 
-        if ( primary->tail == primary->last_flush ) {
-                DRM_DEBUG( "   bailing out...\n" );
+        if (primary->tail == primary->last_flush) {
+                DRM_DEBUG("   bailing out...\n");
                 return;
         }
 
@@ -127,48 +128,46 @@ void mga_do_dma_flush( drm_mga_private_t *dev_priv )
          * actually (partially?) reads the first of these commands.
          * See page 4-16 in the G400 manual, middle of the page or so.
          */
-        BEGIN_DMA( 1 );
+        BEGIN_DMA(1);
 
-        DMA_BLOCK( MGA_DMAPAD,  0x00000000,
-                   MGA_DMAPAD,  0x00000000,
-                   MGA_DMAPAD,  0x00000000,
-                   MGA_DMAPAD,  0x00000000 );
+        DMA_BLOCK(MGA_DMAPAD, 0x00000000,
+                  MGA_DMAPAD, 0x00000000,
+                  MGA_DMAPAD, 0x00000000, MGA_DMAPAD, 0x00000000);
 
         ADVANCE_DMA();
 
         primary->last_flush = primary->tail;
 
-        head = MGA_READ( MGA_PRIMADDRESS );
+        head = MGA_READ(MGA_PRIMADDRESS);
 
-        if ( head <= tail ) {
+        if (head <= tail) {
                 primary->space = primary->size - primary->tail;
         } else {
                 primary->space = head - tail;
         }
 
-        DRM_DEBUG( "   head = 0x%06lx\n", head - dev_priv->primary->offset );
-        DRM_DEBUG( "   tail = 0x%06lx\n", tail - dev_priv->primary->offset );
-        DRM_DEBUG( "  space = 0x%06x\n", primary->space );
+        DRM_DEBUG("   head = 0x%06lx\n", head - dev_priv->primary->offset);
+        DRM_DEBUG("   tail = 0x%06lx\n", tail - dev_priv->primary->offset);
+        DRM_DEBUG("  space = 0x%06x\n", primary->space);
 
         mga_flush_write_combine();
         MGA_WRITE(MGA_PRIMEND, tail | dev_priv->dma_access);
 
-        DRM_DEBUG( "done.\n" );
+        DRM_DEBUG("done.\n");
 }
 
-void mga_do_dma_wrap_start( drm_mga_private_t *dev_priv )
+void mga_do_dma_wrap_start(drm_mga_private_t * dev_priv)
 {
         drm_mga_primary_buffer_t *primary = &dev_priv->prim;
         u32 head, tail;
         DMA_LOCALS;
-        DRM_DEBUG( "\n" );
+        DRM_DEBUG("\n");
 
         BEGIN_DMA_WRAP();
 
-        DMA_BLOCK( MGA_DMAPAD,  0x00000000,
-                   MGA_DMAPAD,  0x00000000,
-                   MGA_DMAPAD,  0x00000000,
-                   MGA_DMAPAD,  0x00000000 );
+        DMA_BLOCK(MGA_DMAPAD, 0x00000000,
+                  MGA_DMAPAD, 0x00000000,
+                  MGA_DMAPAD, 0x00000000, MGA_DMAPAD, 0x00000000);
 
         ADVANCE_DMA();
 
@@ -178,45 +177,43 @@ void mga_do_dma_wrap_start( drm_mga_private_t *dev_priv )
         primary->last_flush = 0;
         primary->last_wrap++;
 
-        head = MGA_READ( MGA_PRIMADDRESS );
+        head = MGA_READ(MGA_PRIMADDRESS);
 
-        if ( head == dev_priv->primary->offset ) {
+        if (head == dev_priv->primary->offset) {
                 primary->space = primary->size;
         } else {
                 primary->space = head - dev_priv->primary->offset;
         }
 
-        DRM_DEBUG( "   head = 0x%06lx\n",
-                  head - dev_priv->primary->offset );
-        DRM_DEBUG( "   tail = 0x%06x\n", primary->tail );
-        DRM_DEBUG( "   wrap = %d\n", primary->last_wrap );
-        DRM_DEBUG( "  space = 0x%06x\n", primary->space );
+        DRM_DEBUG("   head = 0x%06lx\n", head - dev_priv->primary->offset);
+        DRM_DEBUG("   tail = 0x%06x\n", primary->tail);
+        DRM_DEBUG("   wrap = %d\n", primary->last_wrap);
+        DRM_DEBUG("  space = 0x%06x\n", primary->space);
 
         mga_flush_write_combine();
         MGA_WRITE(MGA_PRIMEND, tail | dev_priv->dma_access);
 
-        set_bit( 0, &primary->wrapped );
-        DRM_DEBUG( "done.\n" );
+        set_bit(0, &primary->wrapped);
+        DRM_DEBUG("done.\n");
 }
 
-void mga_do_dma_wrap_end( drm_mga_private_t *dev_priv )
+void mga_do_dma_wrap_end(drm_mga_private_t * dev_priv)
 {
         drm_mga_primary_buffer_t *primary = &dev_priv->prim;
         drm_mga_sarea_t *sarea_priv = dev_priv->sarea_priv;
         u32 head = dev_priv->primary->offset;
-        DRM_DEBUG( "\n" );
+        DRM_DEBUG("\n");
 
         sarea_priv->last_wrap++;
-        DRM_DEBUG( "   wrap = %d\n", sarea_priv->last_wrap );
+        DRM_DEBUG("   wrap = %d\n", sarea_priv->last_wrap);
 
         mga_flush_write_combine();
-        MGA_WRITE( MGA_PRIMADDRESS, head | MGA_DMA_GENERAL );
+        MGA_WRITE(MGA_PRIMADDRESS, head | MGA_DMA_GENERAL);
 
-        clear_bit( 0, &primary->wrapped );
-        DRM_DEBUG( "done.\n" );
+        clear_bit(0, &primary->wrapped);
+        DRM_DEBUG("done.\n");
 }
 
-
 /* ================================================================
  * Freelist management
  */
@@ -225,63 +222,61 @@ void mga_do_dma_wrap_end( drm_mga_private_t *dev_priv )
 #define MGA_BUFFER_FREE         0
 
 #if MGA_FREELIST_DEBUG
-static void mga_freelist_print( drm_device_t *dev )
+static void mga_freelist_print(drm_device_t * dev)
 {
         drm_mga_private_t *dev_priv = dev->dev_private;
         drm_mga_freelist_t *entry;
 
-        DRM_INFO( "\n" );
-        DRM_INFO( "current dispatch: last=0x%x done=0x%x\n",
-                  dev_priv->sarea_priv->last_dispatch,
-                  (unsigned int)(MGA_READ( MGA_PRIMADDRESS ) -
-                                 dev_priv->primary->offset) );
-        DRM_INFO( "current freelist:\n" );
-
-        for ( entry = dev_priv->head->next ; entry ; entry = entry->next ) {
-                DRM_INFO( "   %p   idx=%2d  age=0x%x 0x%06lx\n",
-                          entry, entry->buf->idx, entry->age.head,
-                          entry->age.head - dev_priv->primary->offset );
+        DRM_INFO("\n");
+        DRM_INFO("current dispatch: last=0x%x done=0x%x\n",
+                 dev_priv->sarea_priv->last_dispatch,
+                 (unsigned int)(MGA_READ(MGA_PRIMADDRESS) -
+                                dev_priv->primary->offset));
+        DRM_INFO("current freelist:\n");
+
+        for (entry = dev_priv->head->next; entry; entry = entry->next) {
+                DRM_INFO("   %p   idx=%2d  age=0x%x 0x%06lx\n",
+                         entry, entry->buf->idx, entry->age.head,
+                         entry->age.head - dev_priv->primary->offset);
         }
-        DRM_INFO( "\n" );
+        DRM_INFO("\n");
 }
 #endif
 
-static int mga_freelist_init( drm_device_t *dev, drm_mga_private_t *dev_priv )
+static int mga_freelist_init(drm_device_t * dev, drm_mga_private_t * dev_priv)
 {
         drm_device_dma_t *dma = dev->dma;
         drm_buf_t *buf;
         drm_mga_buf_priv_t *buf_priv;
         drm_mga_freelist_t *entry;
         int i;
-        DRM_DEBUG( "count=%d\n", dma->buf_count );
+        DRM_DEBUG("count=%d\n", dma->buf_count);
 
-        dev_priv->head = drm_alloc( sizeof(drm_mga_freelist_t),
-                                     DRM_MEM_DRIVER );
-        if ( dev_priv->head == NULL )
+        dev_priv->head = drm_alloc(sizeof(drm_mga_freelist_t), DRM_MEM_DRIVER);
+        if (dev_priv->head == NULL)
                 return DRM_ERR(ENOMEM);
 
-        memset( dev_priv->head, 0, sizeof(drm_mga_freelist_t) );
-        SET_AGE( &dev_priv->head->age, MGA_BUFFER_USED, 0 );
+        memset(dev_priv->head, 0, sizeof(drm_mga_freelist_t));
+        SET_AGE(&dev_priv->head->age, MGA_BUFFER_USED, 0);
 
-        for ( i = 0 ; i < dma->buf_count ; i++ ) {
+        for (i = 0; i < dma->buf_count; i++) {
                 buf = dma->buflist[i];
-                buf_priv = buf->dev_private;
+                buf_priv = buf->dev_private;
 
-                entry = drm_alloc( sizeof(drm_mga_freelist_t),
-                                    DRM_MEM_DRIVER );
-                if ( entry == NULL )
+                entry = drm_alloc(sizeof(drm_mga_freelist_t), DRM_MEM_DRIVER);
+                if (entry == NULL)
                         return DRM_ERR(ENOMEM);
 
-                memset( entry, 0, sizeof(drm_mga_freelist_t) );
+                memset(entry, 0, sizeof(drm_mga_freelist_t));
 
                 entry->next = dev_priv->head->next;
                 entry->prev = dev_priv->head;
-                SET_AGE( &entry->age, MGA_BUFFER_FREE, 0 );
+                SET_AGE(&entry->age, MGA_BUFFER_FREE, 0);
                 entry->buf = buf;
 
-                if ( dev_priv->head->next != NULL )
+                if (dev_priv->head->next != NULL)
                         dev_priv->head->next->prev = entry;
-                if ( entry->next == NULL )
+                if (entry->next == NULL)
                         dev_priv->tail = entry;
 
                 buf_priv->list_entry = entry;
@@ -294,17 +289,17 @@ static int mga_freelist_init( drm_device_t *dev, drm_mga_private_t *dev_priv )
         return 0;
 }
 
-static void mga_freelist_cleanup( drm_device_t *dev )
+static void mga_freelist_cleanup(drm_device_t * dev)
 {
         drm_mga_private_t *dev_priv = dev->dev_private;
         drm_mga_freelist_t *entry;
         drm_mga_freelist_t *next;
-        DRM_DEBUG( "\n" );
+        DRM_DEBUG("\n");
 
         entry = dev_priv->head;
-        while ( entry ) {
+        while (entry) {
                 next = entry->next;
-                drm_free( entry, sizeof(drm_mga_freelist_t), DRM_MEM_DRIVER );
+                drm_free(entry, sizeof(drm_mga_freelist_t), DRM_MEM_DRIVER);
                 entry = next;
         }
 
@@ -314,71 +309,69 @@ static void mga_freelist_cleanup( drm_device_t *dev )
 #if 0
 /* FIXME: Still needed?
  */
-static void mga_freelist_reset( drm_device_t *dev )
+static void mga_freelist_reset(drm_device_t * dev)
 {
         drm_device_dma_t *dma = dev->dma;
         drm_buf_t *buf;
         drm_mga_buf_priv_t *buf_priv;
         int i;
 
-        for ( i = 0 ; i < dma->buf_count ; i++ ) {
+        for (i = 0; i < dma->buf_count; i++) {
                 buf = dma->buflist[i];
-                buf_priv = buf->dev_private;
-                SET_AGE( &buf_priv->list_entry->age,
-                         MGA_BUFFER_FREE, 0 );
+                buf_priv = buf->dev_private;
+                SET_AGE(&buf_priv->list_entry->age, MGA_BUFFER_FREE, 0);
         }
 }
 #endif
 
-static drm_buf_t *mga_freelist_get( drm_device_t *dev )
+static drm_buf_t *mga_freelist_get(drm_device_t * dev)
 {
         drm_mga_private_t *dev_priv = dev->dev_private;
         drm_mga_freelist_t *next;
         drm_mga_freelist_t *prev;
         drm_mga_freelist_t *tail = dev_priv->tail;
         u32 head, wrap;
-        DRM_DEBUG( "\n" );
+        DRM_DEBUG("\n");
 
-        head = MGA_READ( MGA_PRIMADDRESS );
+        head = MGA_READ(MGA_PRIMADDRESS);
         wrap = dev_priv->sarea_priv->last_wrap;
 
-        DRM_DEBUG( "   tail=0x%06lx %d\n",
-                   tail->age.head ?
-                   tail->age.head - dev_priv->primary->offset : 0,
-                   tail->age.wrap );
-        DRM_DEBUG( "   head=0x%06lx %d\n",
-                   head - dev_priv->primary->offset, wrap );
+        DRM_DEBUG("   tail=0x%06lx %d\n",
+                  tail->age.head ?
+                  tail->age.head - dev_priv->primary->offset : 0,
+                  tail->age.wrap);
+        DRM_DEBUG("   head=0x%06lx %d\n",
+                  head - dev_priv->primary->offset, wrap);
 
-        if ( TEST_AGE( &tail->age, head, wrap ) ) {
+        if (TEST_AGE(&tail->age, head, wrap)) {
                 prev = dev_priv->tail->prev;
                 next = dev_priv->tail;
                 prev->next = NULL;
                 next->prev = next->next = NULL;
                 dev_priv->tail = prev;
-                SET_AGE( &next->age, MGA_BUFFER_USED, 0 );
+                SET_AGE(&next->age, MGA_BUFFER_USED, 0);
                 return next->buf;
         }
 
-        DRM_DEBUG( "returning NULL!\n" );
+        DRM_DEBUG("returning NULL!\n");
         return NULL;
 }
 
-int mga_freelist_put( drm_device_t *dev, drm_buf_t *buf )
+int mga_freelist_put(drm_device_t * dev, drm_buf_t * buf)
 {
         drm_mga_private_t *dev_priv = dev->dev_private;
         drm_mga_buf_priv_t *buf_priv = buf->dev_private;
         drm_mga_freelist_t *head, *entry, *prev;
 
-        DRM_DEBUG( "age=0x%06lx wrap=%d\n",
-                   buf_priv->list_entry->age.head -
-                   dev_priv->primary->offset,
-                   buf_priv->list_entry->age.wrap );
+        DRM_DEBUG("age=0x%06lx wrap=%d\n",
+                  buf_priv->list_entry->age.head -
+                  dev_priv->primary->offset, buf_priv->list_entry->age.wrap);
 
         entry = buf_priv->list_entry;
         head = dev_priv->head;
 
-        if ( buf_priv->list_entry->age.head == MGA_BUFFER_USED ) {
-                SET_AGE( &entry->age, MGA_BUFFER_FREE, 0 );
+        if (buf_priv->list_entry->age.head == MGA_BUFFER_USED) {
+                SET_AGE(&entry->age, MGA_BUFFER_FREE, 0);
                 prev = dev_priv->tail;
                 prev->next = entry;
                 entry->prev = prev;
@@ -394,15 +387,13 @@ int mga_freelist_put( drm_device_t *dev, drm_buf_t *buf )
         return 0;
 }
 
-
 /* ================================================================
  * DMA initialization, cleanup
  */
 
-
-int mga_driver_preinit(drm_device_t *dev, unsigned long flags)
+int mga_driver_preinit(drm_device_t * dev, unsigned long flags)
 {
-        drm_mga_private_t * dev_priv;
+        drm_mga_private_t *dev_priv;
 
         dev_priv = drm_alloc(sizeof(drm_mga_private_t), DRM_MEM_DRIVER);
         if (!dev_priv)
@@ -420,7 +411,7 @@ int mga_driver_preinit(drm_device_t *dev, unsigned long flags)
 #if __OS_HAS_AGP
 /**
  * Bootstrap the driver for AGP DMA.
- * 
+ *
  * \todo
  * Investigate whether there is any benifit to storing the WARP microcode in
  * AGP memory.  If not, the microcode may as well always be put in PCI
@@ -436,18 +427,18 @@ int mga_driver_preinit(drm_device_t *dev, unsigned long flags)
 static int mga_do_agp_dma_bootstrap(drm_device_t * dev,
                                     drm_mga_dma_bootstrap_t * dma_bs)
 {
-        drm_mga_private_t * const dev_priv = (drm_mga_private_t *) dev->dev_private;
+        drm_mga_private_t *const dev_priv =
+            (drm_mga_private_t *) dev->dev_private;
         unsigned int warp_size = mga_warp_microcode_size(dev_priv);
         int err;
-        unsigned  offset;
+        unsigned offset;
         const unsigned secondary_size = dma_bs->secondary_bin_count
-                * dma_bs->secondary_bin_size;
+            * dma_bs->secondary_bin_size;
         const unsigned agp_size = (dma_bs->agp_size << 20);
         drm_buf_desc_t req;
         drm_agp_mode_t mode;
         drm_agp_info_t info;
 
-        
         /* Acquire AGP. */
         err = drm_agp_acquire(dev);
         if (err) {
@@ -468,7 +459,6 @@ static int mga_do_agp_dma_bootstrap(drm_device_t * dev,
                 return err;
         }
 
-
         /* In addition to the usual AGP mode configuration, the G200 AGP cards
          * need to have the AGP mode "manually" set.
          */
@@ -476,24 +466,22 @@ static int mga_do_agp_dma_bootstrap(drm_device_t * dev,
         if (dev_priv->chipset == MGA_CARD_TYPE_G200) {
                 if (mode.mode & 0x02) {
                         MGA_WRITE(MGA_AGP_PLL, MGA_AGP2XPLL_ENABLE);
-                }
-                else {
+                } else {
                         MGA_WRITE(MGA_AGP_PLL, MGA_AGP2XPLL_DISABLE);
                 }
         }
 
-
         /* Allocate and bind AGP memory. */
         dev_priv->agp_pages = agp_size / PAGE_SIZE;
-        dev_priv->agp_mem = drm_alloc_agp( dev, dev_priv->agp_pages, 0 );
+        dev_priv->agp_mem = drm_alloc_agp(dev, dev_priv->agp_pages, 0);
         if (dev_priv->agp_mem == NULL) {
                 dev_priv->agp_pages = 0;
                 DRM_ERROR("Unable to allocate %uMB AGP memory\n",
                           dma_bs->agp_size);
                 return DRM_ERR(ENOMEM);
         }
-                
-        err = drm_bind_agp( dev_priv->agp_mem, 0 );
+
+        err = drm_bind_agp(dev_priv->agp_mem, 0);
         if (err) {
                 DRM_ERROR("Unable to bind AGP memory\n");
                 return err;
@@ -506,44 +494,44 @@ static int mga_do_agp_dma_bootstrap(drm_device_t * dev,
                 warp_size = PAGE_SIZE;
 
         offset = 0;
-        err = drm_addmap( dev, offset, warp_size,
-                          _DRM_AGP, _DRM_READ_ONLY, & dev_priv->warp );
+        err = drm_addmap(dev, offset, warp_size,
+                         _DRM_AGP, _DRM_READ_ONLY, &dev_priv->warp);
         if (err) {
                 DRM_ERROR("Unable to map WARP microcode\n");
                 return err;
         }
 
         offset += warp_size;
-        err = drm_addmap( dev, offset, dma_bs->primary_size,
-                          _DRM_AGP, _DRM_READ_ONLY, & dev_priv->primary );
+        err = drm_addmap(dev, offset, dma_bs->primary_size,
+                         _DRM_AGP, _DRM_READ_ONLY, &dev_priv->primary);
         if (err) {
                 DRM_ERROR("Unable to map primary DMA region\n");
                 return err;
         }
 
         offset += dma_bs->primary_size;
-        err = drm_addmap( dev, offset, secondary_size,
-                          _DRM_AGP, 0, & dev->agp_buffer_map );
+        err = drm_addmap(dev, offset, secondary_size,
+                         _DRM_AGP, 0, &dev->agp_buffer_map);
         if (err) {
                 DRM_ERROR("Unable to map secondary DMA region\n");
                 return err;
         }
 
-        (void) memset( &req, 0, sizeof(req) );
+        (void)memset(&req, 0, sizeof(req));
         req.count = dma_bs->secondary_bin_count;
         req.size = dma_bs->secondary_bin_size;
         req.flags = _DRM_AGP_BUFFER;
         req.agp_start = offset;
 
-        err = drm_addbufs_agp( dev, & req );
+        err = drm_addbufs_agp(dev, &req);
         if (err) {
                 DRM_ERROR("Unable to add secondary DMA buffers\n");
                 return err;
         }
 
         offset += secondary_size;
-        err = drm_addmap( dev, offset, agp_size - offset,
-                          _DRM_AGP, 0, & dev_priv->agp_textures );
+        err = drm_addmap(dev, offset, agp_size - offset,
+                         _DRM_AGP, 0, &dev_priv->agp_textures);
         if (err) {
                 DRM_ERROR("Unable to map AGP texture region\n");
                 return err;
@@ -577,7 +565,7 @@ static int mga_do_agp_dma_bootstrap(drm_device_t * dev,
 
 /**
  * Bootstrap the driver for PCI DMA.
- * 
+ *
  * \todo
  * The algorithm for decreasing the size of the primary DMA buffer could be
  * better.  The size should be rounded up to the nearest page size, then
@@ -586,20 +574,20 @@ static int mga_do_agp_dma_bootstrap(drm_device_t * dev,
  * \todo
  * Determine whether the maximum address passed to drm_pci_alloc is correct.
  * The same goes for drm_addbufs_pci.
- * 
+ *
  * \sa mga_do_dma_bootstrap, mga_do_agp_dma_bootstrap
  */
 static int mga_do_pci_dma_bootstrap(drm_device_t * dev,
                                     drm_mga_dma_bootstrap_t * dma_bs)
 {
-        drm_mga_private_t * const dev_priv = (drm_mga_private_t *) dev->dev_private;
+        drm_mga_private_t *const dev_priv =
+            (drm_mga_private_t *) dev->dev_private;
         unsigned int warp_size = mga_warp_microcode_size(dev_priv);
         unsigned int primary_size;
         unsigned int bin_count;
         int err;
         drm_buf_desc_t req;
 
-        
         if (dev->dma == NULL) {
                 DRM_ERROR("dev->dma is NULL\n");
                 return DRM_ERR(EFAULT);
@@ -624,9 +612,8 @@ static int mga_do_pci_dma_bootstrap(drm_device_t * dev,
          * alignment of the primary or secondary DMA buffers.
          */
 
-        for ( primary_size = dma_bs->primary_size
-              ; primary_size != 0
-              ; primary_size >>= 1 ) {
+        for (primary_size = dma_bs->primary_size; primary_size != 0;
+             primary_size >>= 1) {
                 /* The proper alignment for this mapping is 0x04 */
                 err = drm_addmap(dev, 0, primary_size, _DRM_CONSISTENT,
                                  _DRM_READ_ONLY, &dev_priv->primary);
@@ -641,24 +628,23 @@ static int mga_do_pci_dma_bootstrap(drm_device_t * dev,
 
         if (dev_priv->primary->size != dma_bs->primary_size) {
                 DRM_INFO("Primary DMA buffer size reduced from %u to %u.\n",
-                         dma_bs->primary_size, 
-                         (unsigned) dev_priv->primary->size);
+                         dma_bs->primary_size,
+                         (unsigned)dev_priv->primary->size);
                 dma_bs->primary_size = dev_priv->primary->size;
         }
 
-        for ( bin_count = dma_bs->secondary_bin_count
-              ; bin_count > 0 
-              ; bin_count-- ) {
-                (void) memset( &req, 0, sizeof(req) );
+        for (bin_count = dma_bs->secondary_bin_count; bin_count > 0;
+             bin_count--) {
+                (void)memset(&req, 0, sizeof(req));
                 req.count = bin_count;
                 req.size = dma_bs->secondary_bin_size;
 
-                err = drm_addbufs_pci( dev, & req );
+                err = drm_addbufs_pci(dev, &req);
                 if (!err) {
                         break;
                 }
         }
-        
+
         if (bin_count == 0) {
                 DRM_ERROR("Unable to add secondary DMA buffers\n");
                 return err;
@@ -680,38 +666,34 @@ static int mga_do_pci_dma_bootstrap(drm_device_t * dev,
         return 0;
 }
 
-
 static int mga_do_dma_bootstrap(drm_device_t * dev,
                                 drm_mga_dma_bootstrap_t * dma_bs)
 {
         const int is_agp = (dma_bs->agp_mode != 0) && drm_device_is_agp(dev);
         int err;
-        drm_mga_private_t * const dev_priv =
-                (drm_mga_private_t *) dev->dev_private;
-
+        drm_mga_private_t *const dev_priv =
+            (drm_mga_private_t *) dev->dev_private;
 
         dev_priv->used_new_dma_init = 1;
 
         /* The first steps are the same for both PCI and AGP based DMA.  Map
          * the cards MMIO registers and map a status page.
          */
-        err = drm_addmap( dev, dev_priv->mmio_base, dev_priv->mmio_size,
-                          _DRM_REGISTERS, _DRM_READ_ONLY, & dev_priv->mmio );
+        err = drm_addmap(dev, dev_priv->mmio_base, dev_priv->mmio_size,
+                         _DRM_REGISTERS, _DRM_READ_ONLY, &dev_priv->mmio);
         if (err) {
                 DRM_ERROR("Unable to map MMIO region\n");
                 return err;
         }
 
-
-        err = drm_addmap( dev, 0, SAREA_MAX, _DRM_SHM,
-                          _DRM_READ_ONLY | _DRM_LOCKED | _DRM_KERNEL,
-                          & dev_priv->status );
+        err = drm_addmap(dev, 0, SAREA_MAX, _DRM_SHM,
+                         _DRM_READ_ONLY | _DRM_LOCKED | _DRM_KERNEL,
+                         &dev_priv->status);
         if (err) {
                 DRM_ERROR("Unable to map status region\n");
                 return err;
         }
 
-
         /* The DMA initialization procedure is slightly different for PCI and
          * AGP cards.  AGP cards just allocate a large block of AGP memory and
          * carve off portions of it for internal uses.  The remaining memory
@@ -720,7 +702,7 @@ static int mga_do_dma_bootstrap(drm_device_t * dev,
         if (is_agp) {
                 err = mga_do_agp_dma_bootstrap(dev, dma_bs);
         }
-        
+
         /* If we attempted to initialize the card for AGP DMA but failed,
          * clean-up any mess that may have been created.
          */
@@ -729,7 +711,6 @@ static int mga_do_dma_bootstrap(drm_device_t * dev,
                 mga_do_cleanup_dma(dev);
         }
 
-
         /* Not only do we want to try and initialized PCI cards for PCI DMA,
          * but we also try to initialized AGP cards that could not be
          * initialized for AGP DMA.  This covers the case where we have an AGP
@@ -742,7 +723,6 @@ static int mga_do_dma_bootstrap(drm_device_t * dev,
                 err = mga_do_pci_dma_bootstrap(dev, dma_bs);
         }
 
-
         return err;
 }
 
@@ -752,45 +732,42 @@ int mga_dma_bootstrap(DRM_IOCTL_ARGS)
         drm_mga_dma_bootstrap_t bootstrap;
         int err;
 
-
         DRM_COPY_FROM_USER_IOCTL(bootstrap,
                                  (drm_mga_dma_bootstrap_t __user *) data,
                                  sizeof(bootstrap));
 
-        err = mga_do_dma_bootstrap(dev, & bootstrap);
-        if (! err) {
+        err = mga_do_dma_bootstrap(dev, &bootstrap);
+        if (!err) {
                 static const int modes[] = { 0, 1, 2, 2, 4, 4, 4, 4 };
-                const drm_mga_private_t * const dev_priv = 
-                        (drm_mga_private_t *) dev->dev_private;
+                const drm_mga_private_t *const dev_priv =
+                    (drm_mga_private_t *) dev->dev_private;
 
                 if (dev_priv->agp_textures != NULL) {
-                        bootstrap.texture_handle = dev_priv->agp_textures->offset;
+                        bootstrap.texture_handle =
+                            dev_priv->agp_textures->offset;
                         bootstrap.texture_size = dev_priv->agp_textures->size;
-                }
-                else {
+                } else {
                         bootstrap.texture_handle = 0;
                         bootstrap.texture_size = 0;
                 }
 
-                bootstrap.agp_mode = modes[ bootstrap.agp_mode & 0x07 ];
-                if (DRM_COPY_TO_USER( (void __user *) data, & bootstrap,
+                bootstrap.agp_mode = modes[bootstrap.agp_mode & 0x07];
+                if (DRM_COPY_TO_USER((void __user *)data, &bootstrap,
                                      sizeof(bootstrap))) {
                         err = DRM_ERR(EFAULT);
                 }
-        }
-        else {
+        } else {
                 mga_do_cleanup_dma(dev);
         }
 
         return err;
 }
 
-static int mga_do_init_dma( drm_device_t *dev, drm_mga_init_t *init )
+static int mga_do_init_dma(drm_device_t * dev, drm_mga_init_t * init)
 {
         drm_mga_private_t *dev_priv;
         int ret;
-        DRM_DEBUG( "\n" );
-
+        DRM_DEBUG("\n");
 
         dev_priv = dev->dev_private;
 
@@ -799,17 +776,17 @@ static int mga_do_init_dma( drm_device_t *dev, drm_mga_init_t *init )
         } else {
                 dev_priv->clear_cmd = MGA_DWGCTL_CLEAR | MGA_ATYPE_RSTR;
         }
-        dev_priv->maccess       = init->maccess;
+        dev_priv->maccess = init->maccess;
 
-        dev_priv->fb_cpp        = init->fb_cpp;
-        dev_priv->front_offset  = init->front_offset;
-        dev_priv->front_pitch   = init->front_pitch;
-        dev_priv->back_offset   = init->back_offset;
-        dev_priv->back_pitch    = init->back_pitch;
+        dev_priv->fb_cpp = init->fb_cpp;
+        dev_priv->front_offset = init->front_offset;
+        dev_priv->front_pitch = init->front_pitch;
+        dev_priv->back_offset = init->back_offset;
+        dev_priv->back_pitch = init->back_pitch;
 
-        dev_priv->depth_cpp     = init->depth_cpp;
-        dev_priv->depth_offset  = init->depth_offset;
-        dev_priv->depth_pitch   = init->depth_pitch;
+        dev_priv->depth_cpp = init->depth_cpp;
+        dev_priv->depth_offset = init->depth_offset;
+        dev_priv->depth_pitch = init->depth_pitch;
 
         /* FIXME: Need to support AGP textures...
          */
@@ -823,7 +800,7 @@ static int mga_do_init_dma( drm_device_t *dev, drm_mga_init_t *init )
                 return DRM_ERR(EINVAL);
         }
 
-        if (! dev_priv->used_new_dma_init) {
+        if (!dev_priv->used_new_dma_init) {
 
                 dev_priv->dma_access = MGA_PAGPXFER;
                 dev_priv->wagp_enable = MGA_WAGP_ENABLE;
@@ -849,7 +826,8 @@ static int mga_do_init_dma( drm_device_t *dev, drm_mga_init_t *init )
                         return DRM_ERR(EINVAL);
                 }
                 dev->agp_buffer_token = init->buffers_offset;
-                dev->agp_buffer_map = drm_core_findmap(dev, init->buffers_offset);
+                dev->agp_buffer_map =
+                    drm_core_findmap(dev, init->buffers_offset);
                 if (!dev->agp_buffer_map) {
                         DRM_ERROR("failed to find dma buffer region!\n");
                         return DRM_ERR(EINVAL);
@@ -861,8 +839,8 @@ static int mga_do_init_dma( drm_device_t *dev, drm_mga_init_t *init )
         }
 
         dev_priv->sarea_priv =
-                (drm_mga_sarea_t *)((u8 *)dev_priv->sarea->handle +
-                                    init->sarea_priv_offset);
+            (drm_mga_sarea_t *) ((u8 *) dev_priv->sarea->handle +
+                                 init->sarea_priv_offset);
 
         if (!dev_priv->warp->handle ||
             !dev_priv->primary->handle ||
@@ -885,23 +863,20 @@ static int mga_do_init_dma( drm_device_t *dev, drm_mga_init_t *init )
                 return ret;
         }
 
-        dev_priv->prim.status = (u32 *)dev_priv->status->handle;
+        dev_priv->prim.status = (u32 *) dev_priv->status->handle;
 
-        mga_do_wait_for_idle( dev_priv );
+        mga_do_wait_for_idle(dev_priv);
 
         /* Init the primary DMA registers.
          */
-        MGA_WRITE( MGA_PRIMADDRESS,
-                   dev_priv->primary->offset | MGA_DMA_GENERAL );
+        MGA_WRITE(MGA_PRIMADDRESS, dev_priv->primary->offset | MGA_DMA_GENERAL);
 #if 0
-        MGA_WRITE( MGA_PRIMPTR,
-                   virt_to_bus((void *)dev_priv->prim.status) |
-                   MGA_PRIMPTREN0 |     /* Soft trap, SECEND, SETUPEND */
-                   MGA_PRIMPTREN1 );    /* DWGSYNC */
+        MGA_WRITE(MGA_PRIMPTR, virt_to_bus((void *)dev_priv->prim.status) | MGA_PRIMPTREN0 |    /* Soft trap, SECEND, SETUPEND */
+                  MGA_PRIMPTREN1);      /* DWGSYNC */
 #endif
 
-        dev_priv->prim.start = (u8 *)dev_priv->primary->handle;
-        dev_priv->prim.end = ((u8 *)dev_priv->primary->handle
+        dev_priv->prim.start = (u8 *) dev_priv->primary->handle;
+        dev_priv->prim.end = ((u8 *) dev_priv->primary->handle
                               + dev_priv->primary->size);
         dev_priv->prim.size = dev_priv->primary->size;
 
@@ -929,7 +904,7 @@ static int mga_do_init_dma( drm_device_t *dev, drm_mga_init_t *init )
         return 0;
 }
 
-static int mga_do_cleanup_dma( drm_device_t *dev )
+static int mga_do_cleanup_dma(drm_device_t * dev)
 {
         int err = 0;
         DRM_DEBUG("\n");
@@ -938,16 +913,17 @@ static int mga_do_cleanup_dma( drm_device_t *dev )
          * may not have been called from userspace and after dev_private
          * is freed, it's too late.
          */
-        if ( dev->irq_enabled ) drm_irq_uninstall(dev);
+        if (dev->irq_enabled)
+                drm_irq_uninstall(dev);
 
-        if ( dev->dev_private ) {
+        if (dev->dev_private) {
                 drm_mga_private_t *dev_priv = dev->dev_private;
 
-                if ((dev_priv->warp != NULL) 
+                if ((dev_priv->warp != NULL)
                     && (dev_priv->warp->type != _DRM_CONSISTENT))
                         drm_core_ioremapfree(dev_priv->warp, dev);
 
-                if ((dev_priv->primary != NULL) 
+                if ((dev_priv->primary != NULL)
                     && (dev_priv->primary->type != _DRM_CONSISTENT))
                         drm_core_ioremapfree(dev_priv->primary, dev);
 
@@ -960,7 +936,8 @@ static int mga_do_cleanup_dma( drm_device_t *dev )
                                 dev_priv->agp_textures = NULL;
                                 drm_unbind_agp(dev_priv->agp_mem);
 
-                                drm_free_agp(dev_priv->agp_mem, dev_priv->agp_pages);
+                                drm_free_agp(dev_priv->agp_mem,
+                                             dev_priv->agp_pages);
                                 dev_priv->agp_pages = 0;
                                 dev_priv->agp_mem = NULL;
                         }
@@ -982,7 +959,8 @@ static int mga_do_cleanup_dma( drm_device_t *dev )
 
                 memset(&dev_priv->prim, 0, sizeof(dev_priv->prim));
                 dev_priv->warp_pipe = 0;
-                memset(dev_priv->warp_pipe_phys, 0, sizeof(dev_priv->warp_pipe_phys));
+                memset(dev_priv->warp_pipe_phys, 0,
+                       sizeof(dev_priv->warp_pipe_phys));
 
                 if (dev_priv->head != NULL) {
                         mga_freelist_cleanup(dev);
@@ -992,103 +970,102 @@ static int mga_do_cleanup_dma( drm_device_t *dev )
         return err;
 }
 
-int mga_dma_init( DRM_IOCTL_ARGS )
+int mga_dma_init(DRM_IOCTL_ARGS)
 {
         DRM_DEVICE;
         drm_mga_init_t init;
         int err;
 
-        LOCK_TEST_WITH_RETURN( dev, filp );
+        LOCK_TEST_WITH_RETURN(dev, filp);
 
         DRM_COPY_FROM_USER_IOCTL(init, (drm_mga_init_t __user *) data,
                                  sizeof(init));
 
-        switch ( init.func ) {
+        switch (init.func) {
         case MGA_INIT_DMA:
                 err = mga_do_init_dma(dev, &init);
                 if (err) {
-                        (void) mga_do_cleanup_dma(dev);
+                        (void)mga_do_cleanup_dma(dev);
                 }
                 return err;
         case MGA_CLEANUP_DMA:
-                return mga_do_cleanup_dma( dev );
+                return mga_do_cleanup_dma(dev);
         }
 
         return DRM_ERR(EINVAL);
 }
 
-
 /* ================================================================
  * Primary DMA stream management
  */
 
-int mga_dma_flush( DRM_IOCTL_ARGS )
+int mga_dma_flush(DRM_IOCTL_ARGS)
 {
         DRM_DEVICE;
-        drm_mga_private_t *dev_priv = (drm_mga_private_t *)dev->dev_private;
+        drm_mga_private_t *dev_priv = (drm_mga_private_t *) dev->dev_private;
         drm_lock_t lock;
 
-        LOCK_TEST_WITH_RETURN( dev, filp );
+        LOCK_TEST_WITH_RETURN(dev, filp);
 
-        DRM_COPY_FROM_USER_IOCTL( lock, (drm_lock_t __user *)data, sizeof(lock) );
+        DRM_COPY_FROM_USER_IOCTL(lock, (drm_lock_t __user *) data,
+                                 sizeof(lock));
 
-        DRM_DEBUG( "%s%s%s\n",
-                   (lock.flags & _DRM_LOCK_FLUSH) ?     "flush, " : "",
-                   (lock.flags & _DRM_LOCK_FLUSH_ALL) ? "flush all, " : "",
-                   (lock.flags & _DRM_LOCK_QUIESCENT) ? "idle, " : "" );
+        DRM_DEBUG("%s%s%s\n",
+                  (lock.flags & _DRM_LOCK_FLUSH) ? "flush, " : "",
+                  (lock.flags & _DRM_LOCK_FLUSH_ALL) ? "flush all, " : "",
+                  (lock.flags & _DRM_LOCK_QUIESCENT) ? "idle, " : "");
 
-        WRAP_WAIT_WITH_RETURN( dev_priv );
+        WRAP_WAIT_WITH_RETURN(dev_priv);
 
-        if ( lock.flags & (_DRM_LOCK_FLUSH | _DRM_LOCK_FLUSH_ALL) ) {
-                mga_do_dma_flush( dev_priv );
+        if (lock.flags & (_DRM_LOCK_FLUSH | _DRM_LOCK_FLUSH_ALL)) {
+                mga_do_dma_flush(dev_priv);
         }
 
-        if ( lock.flags & _DRM_LOCK_QUIESCENT ) {
+        if (lock.flags & _DRM_LOCK_QUIESCENT) {
 #if MGA_DMA_DEBUG
-                int ret = mga_do_wait_for_idle( dev_priv );
-                if ( ret < 0 )
-                        DRM_INFO( "%s: -EBUSY\n", __FUNCTION__ );
+                int ret = mga_do_wait_for_idle(dev_priv);
+                if (ret < 0)
+                        DRM_INFO("%s: -EBUSY\n", __FUNCTION__);
                 return ret;
 #else
-                return mga_do_wait_for_idle( dev_priv );
+                return mga_do_wait_for_idle(dev_priv);
 #endif
         } else {
                 return 0;
         }
 }
 
-int mga_dma_reset( DRM_IOCTL_ARGS )
+int mga_dma_reset(DRM_IOCTL_ARGS)
 {
         DRM_DEVICE;
-        drm_mga_private_t *dev_priv = (drm_mga_private_t *)dev->dev_private;
+        drm_mga_private_t *dev_priv = (drm_mga_private_t *) dev->dev_private;
 
-        LOCK_TEST_WITH_RETURN( dev, filp );
+        LOCK_TEST_WITH_RETURN(dev, filp);
 
-        return mga_do_dma_reset( dev_priv );
+        return mga_do_dma_reset(dev_priv);
 }
 
-
 /* ================================================================
  * DMA buffer management
  */
 
-static int mga_dma_get_buffers( DRMFILE filp,
-                                drm_device_t *dev, drm_dma_t *d )
+static int mga_dma_get_buffers(DRMFILE filp, drm_device_t * dev, drm_dma_t * d)
 {
         drm_buf_t *buf;
         int i;
 
-        for ( i = d->granted_count ; i < d->request_count ; i++ ) {
-                buf = mga_freelist_get( dev );
-                if ( !buf ) return DRM_ERR(EAGAIN);
+        for (i = d->granted_count; i < d->request_count; i++) {
+                buf = mga_freelist_get(dev);
+                if (!buf)
+                        return DRM_ERR(EAGAIN);
 
                 buf->filp = filp;
 
-                if ( DRM_COPY_TO_USER( &d->request_indices[i],
-                                   &buf->idx, sizeof(buf->idx) ) )
+                if (DRM_COPY_TO_USER(&d->request_indices[i],
+                                     &buf->idx, sizeof(buf->idx)))
                         return DRM_ERR(EFAULT);
-                if ( DRM_COPY_TO_USER( &d->request_sizes[i],
-                                   &buf->total, sizeof(buf->total) ) )
+                if (DRM_COPY_TO_USER(&d->request_sizes[i],
+                                     &buf->total, sizeof(buf->total)))
                         return DRM_ERR(EFAULT);
 
                 d->granted_count++;
@@ -1096,44 +1073,44 @@ static int mga_dma_get_buffers( DRMFILE filp,
         return 0;
 }
 
-int mga_dma_buffers( DRM_IOCTL_ARGS )
+int mga_dma_buffers(DRM_IOCTL_ARGS)
 {
         DRM_DEVICE;
         drm_device_dma_t *dma = dev->dma;
-        drm_mga_private_t *dev_priv = (drm_mga_private_t *)dev->dev_private;
+        drm_mga_private_t *dev_priv = (drm_mga_private_t *) dev->dev_private;
         drm_dma_t __user *argp = (void __user *)data;
         drm_dma_t d;
         int ret = 0;
 
-        LOCK_TEST_WITH_RETURN( dev, filp );
+        LOCK_TEST_WITH_RETURN(dev, filp);
 
-        DRM_COPY_FROM_USER_IOCTL( d, argp, sizeof(d) );
+        DRM_COPY_FROM_USER_IOCTL(d, argp, sizeof(d));
 
         /* Please don't send us buffers.
          */
-        if ( d.send_count != 0 ) {
-                DRM_ERROR( "Process %d trying to send %d buffers via drmDMA\n",
-                           DRM_CURRENTPID, d.send_count );
+        if (d.send_count != 0) {
+                DRM_ERROR("Process %d trying to send %d buffers via drmDMA\n",
+                          DRM_CURRENTPID, d.send_count);
                 return DRM_ERR(EINVAL);
         }
 
         /* We'll send you buffers.
          */
-        if ( d.request_count < 0 || d.request_count > dma->buf_count ) {
-                DRM_ERROR( "Process %d trying to get %d buffers (of %d max)\n",
-                           DRM_CURRENTPID, d.request_count, dma->buf_count );
+        if (d.request_count < 0 || d.request_count > dma->buf_count) {
+                DRM_ERROR("Process %d trying to get %d buffers (of %d max)\n",
+                          DRM_CURRENTPID, d.request_count, dma->buf_count);
                 return DRM_ERR(EINVAL);
         }
 
-        WRAP_TEST_WITH_RETURN( dev_priv );
+        WRAP_TEST_WITH_RETURN(dev_priv);
 
         d.granted_count = 0;
 
-        if ( d.request_count ) {
-                ret = mga_dma_get_buffers( filp, dev, &d );
+        if (d.request_count) {
+                ret = mga_dma_get_buffers(filp, dev, &d);
         }
 
-        DRM_COPY_TO_USER_IOCTL( argp, d, sizeof(d) );
+        DRM_COPY_TO_USER_IOCTL(argp, d, sizeof(d));
 
         return ret;
 }
@@ -1154,11 +1131,11 @@ int mga_driver_postcleanup(drm_device_t * dev)
  */
 void mga_driver_pretakedown(drm_device_t * dev)
 {
-        mga_do_cleanup_dma( dev );
+        mga_do_cleanup_dma(dev);
 }
 
-int mga_driver_dma_quiescent(drm_device_t *dev)
+int mga_driver_dma_quiescent(drm_device_t * dev)
 {
         drm_mga_private_t *dev_priv = dev->dev_private;
-        return mga_do_wait_for_idle( dev_priv );
+        return mga_do_wait_for_idle(dev_priv);
 }