[PATCH] bcm43xx: optimization of DMA bitfields

Convert the bitfields in the bcm43xx DMA code to properly
aligned u8 booleans. These flags are accessed in the DMA
hotpath, so it's a good idea to waste a few bytes of memory
for the sake of speed by not requiring masking (and probably
shifting) of the bitfields.

Signed-off-by: Michael Buesch <mb@bu3sch.de>
Signed-Off-By: Larry Finger <Larry.Finger@lwfinger.net>
Signed-off-by: John W. Linville <linville@tuxdriver.com>

diff --git a/drivers/net/wireless/bcm43xx/bcm43xx_dma.h b/drivers/net/wireless/bcm43xx/bcm43xx_dma.h
index 258a2f9bd7a63c67810e352f4f007fd9a456271e..e04bcaddd1d004d1b514a39bf047f11a963e1bbf 100644 (file)
--- a/drivers/net/wireless/bcm43xx/bcm43xx_dma.h
+++ b/drivers/net/wireless/bcm43xx/bcm43xx_dma.h
@@ -235,9 +235,12 @@ struct bcm43xx_dmaring {
        u16 mmio_base;
        /* DMA controller index number (0-5). */
        int index;
-       u8 tx:1,        /* TRUE, if this is a TX ring. */
-          dma64:1,     /* TRUE, if 64-bit DMA is enabled (FALSE if 32bit). */
-          suspended:1; /* TRUE, if transfers are suspended on this ring. */
+       /* Boolean. Is this a TX ring? */
+       u8 tx;
+       /* Boolean. 64bit DMA if true, 32bit DMA otherwise. */
+       u8 dma64;
+       /* Boolean. Are transfers suspended on this ring? */
+       u8 suspended;
        struct bcm43xx_private *bcm;
 #ifdef CONFIG_BCM43XX_DEBUG
        /* Maximum number of used slots. */