Whitespace Fix
- Remove trailing whitespaces
- Improve readability

Signed-off-by: Jeff Kirsher <[EMAIL PROTECTED]>
Signed-off-by: John Ronciak <[EMAIL PROTECTED]>
Signed-off-by: Jesse Brandeburg <[EMAIL PROTECTED]>


diff -up linux-2.6/drivers/net/e1000/e1000_ethtool.c
linux-2.6.new/drivers/net/e1000/e1000_ethtool.c
--- linux-2.6/drivers/net/e1000/e1000_ethtool.c 2005-11-14
16:20:34.000000000 -0800
+++ linux-2.6.new/drivers/net/e1000/e1000_ethtool.c     2005-11-04
01:23:40.000000000 -0800
@@ -195,11 +195,11 @@ e1000_set_settings(struct net_device *ne

        if(ecmd->autoneg == AUTONEG_ENABLE) {
                hw->autoneg = 1;
-               if(hw->media_type == e1000_media_type_fiber)
+               if (hw->media_type == e1000_media_type_fiber)
                        hw->autoneg_advertised = ADVERTISED_1000baseT_Full |
                                     ADVERTISED_FIBRE |
                                     ADVERTISED_Autoneg;
-               else
+               else
                        hw->autoneg_advertised = ADVERTISED_10baseT_Half |
                                                  ADVERTISED_10baseT_Full |
                                                  ADVERTISED_100baseT_Half |
@@ -231,9 +231,9 @@ e1000_get_pauseparam(struct net_device *
        struct e1000_adapter *adapter = netdev_priv(netdev);
        struct e1000_hw *hw = &adapter->hw;

-       pause->autoneg =
+       pause->autoneg =
                (adapter->fc_autoneg ? AUTONEG_ENABLE : AUTONEG_DISABLE);
-       
+
        if(hw->fc == e1000_fc_rx_pause)
                pause->rx_pause = 1;
        else if(hw->fc == e1000_fc_tx_pause)
@@ -250,7 +250,7 @@ e1000_set_pauseparam(struct net_device *
 {
        struct e1000_adapter *adapter = netdev_priv(netdev);
        struct e1000_hw *hw = &adapter->hw;
-       
+
        adapter->fc_autoneg = pause->autoneg;

        if(pause->rx_pause && pause->tx_pause)
@@ -274,7 +274,7 @@ e1000_set_pauseparam(struct net_device *
        else
                return ((hw->media_type == e1000_media_type_fiber) ?
                        e1000_setup_link(hw) : e1000_force_mac_fc(hw));
-       
+
        return 0;
 }

@@ -298,7 +298,7 @@ e1000_set_rx_csum(struct net_device *net
                e1000_reset(adapter);
        return 0;
 }
-       
+
 static uint32_t
 e1000_get_tx_csum(struct net_device *netdev)
 {
@@ -329,8 +334,8 @@ static int
 e1000_set_tso(struct net_device *netdev, uint32_t data)
 {
        struct e1000_adapter *adapter = netdev_priv(netdev);
-       if((adapter->hw.mac_type < e1000_82544) ||
-           (adapter->hw.mac_type == e1000_82547))
+       if ((adapter->hw.mac_type < e1000_82544) ||
+           (adapter->hw.mac_type == e1000_82547))
                return data ? -EINVAL : 0;

        if (data)
@@ -338,7 +334,7 @@
        else
                netdev->features &= ~NETIF_F_TSO;
        return 0;
-}
+}
 #endif /* NETIF_F_TSO */

 static uint32_t
@@ -355,7 +355,7 @@ e1000_set_msglevel(struct net_device *ne
        adapter->msg_enable = data;
 }

-static int
+static int
 e1000_get_regs_len(struct net_device *netdev)
 {
 #define E1000_REGS_LEN 32
@@ -557,9 +551,9 @@ e1000_set_eeprom(struct net_device *netd
        ret_val = e1000_write_eeprom(hw, first_word,
                                     last_word - first_word + 1, eeprom_buff);

-       /* Update the checksum over the first part of the EEPROM if needed
+       /* Update the checksum over the first part of the EEPROM if needed
         * and flush shadow RAM for 82573 conrollers */
-       if((ret_val == 0) && ((first_word <= EEPROM_CHECKSUM_REG) ||
+       if ((ret_val == 0) && ((first_word <= EEPROM_CHECKSUM_REG) ||
                                (hw->mac_type == e1000_82573)))
                e1000_update_eeprom_checksum(hw);

@@ -623,7 +623,7 @@ e1000_get_ringparam(struct net_device *n
        ring->rx_jumbo_pending = 0;
 }

-static int
+static int
 e1000_set_ringparam(struct net_device *netdev,
                     struct ethtool_ringparam *ring)
 {
@@ -666,12 +666,12 @@ e1000_set_ringparam(struct net_device *n
        rxdr->count = max(ring->rx_pending,(uint32_t)E1000_MIN_RXD);
        rxdr->count = min(rxdr->count,(uint32_t)(mac_type < e1000_82544 ?
                E1000_MAX_RXD : E1000_MAX_82544_RXD));
-       E1000_ROUNDUP(rxdr->count, REQ_RX_DESCRIPTOR_MULTIPLE);
+       E1000_ROUNDUP(rxdr->count, REQ_RX_DESCRIPTOR_MULTIPLE);

        txdr->count = max(ring->tx_pending,(uint32_t)E1000_MIN_TXD);
        txdr->count = min(txdr->count,(uint32_t)(mac_type < e1000_82544 ?
                E1000_MAX_TXD : E1000_MAX_82544_TXD));
-       E1000_ROUNDUP(txdr->count, REQ_TX_DESCRIPTOR_MULTIPLE);
+       E1000_ROUNDUP(txdr->count, REQ_TX_DESCRIPTOR_MULTIPLE);

        for (i = 0; i < adapter->num_queues; i++) {
                txdr[i].count = txdr->count;
@@ -736,7 +736,7 @@ err_setup_rx:
        uint32_t value;                                                        \
        E1000_WRITE_REG(&adapter->hw, R, W & M);                               \
        value = E1000_READ_REG(&adapter->hw, R);                               \
-       if((W & M) != (value & M)) {                                          \
+       if ((W & M) != (value & M)) {                                          \
                DPRINTK(DRV, ERR, "set/check reg %04X test failed: got 0x%08X "\
                        "expected 0x%08X\n", E1000_##R, (value & M), (W & M)); \
                *data = (adapter->hw.mac_type < e1000_82543) ?                 \
@@ -907,7 +907,7 @@ e1000_intr_test(struct e1000_adapter *ad
                        E1000_WRITE_REG(&adapter->hw, IMC, mask);
                        E1000_WRITE_REG(&adapter->hw, ICS, mask);
                        msec_delay(10);
-
+
                        if(adapter->test_icr & mask) {
                                *data = 3;
                                break;
@@ -1011,7 +1011,7 @@ e1000_setup_desc_rings(struct e1000_adap
        /* Setup Tx descriptor ring and Tx buffers */

        if(!txdr->count)
-               txdr->count = E1000_DEFAULT_TXD;
+               txdr->count = E1000_DEFAULT_TXD;

        size = txdr->count * sizeof(struct e1000_buffer);
        if(!(txdr->buffer_info = kmalloc(size, GFP_KERNEL))) {
@@ -1067,7 +1067,7 @@ e1000_setup_desc_rings(struct e1000_adap
        /* Setup Rx descriptor ring and Rx buffers */

        if(!rxdr->count)
-               rxdr->count = E1000_DEFAULT_RXD;
+               rxdr->count = E1000_DEFAULT_RXD;

        size = rxdr->count * sizeof(struct e1000_buffer);
        if(!(rxdr->buffer_info = kmalloc(size, GFP_KERNEL))) {
@@ -1433,7 +1433,7 @@ e1000_run_loopback_test(struct e1000_ada

        E1000_WRITE_REG(&adapter->hw, RDT, rxdr->count - 1);

-       /* Calculate the loop count based on the largest descriptor ring
+       /* Calculate the loop count based on the largest descriptor ring
         * The idea is to wrap the largest ring a number of times using 64
         * send/receive pairs during each loop
         */
@@ -1446,9 +1446,9 @@ e1000_run_loopback_test(struct e1000_ada
        k = l = 0;
        for(j = 0; j <= lc; j++) { /* loop count loop */
                for(i = 0; i < 64; i++) { /* send the packets */
-                       e1000_create_lbtest_frame(txdr->buffer_info[i].skb,
+                       e1000_create_lbtest_frame(txdr->buffer_info[i].skb,
                                        1024);
-                       pci_dma_sync_single_for_device(pdev,
+                       pci_dma_sync_single_for_device(pdev,
                                        txdr->buffer_info[k].dma,
                                        txdr->buffer_info[k].length,
                                        PCI_DMA_TODEVICE);
@@ -1459,25 +1459,25 @@ e1000_run_loopback_test(struct e1000_ada
                time = jiffies; /* set the start time for the receive */
                good_cnt = 0;
                do { /* receive the sent packets */
-                       pci_dma_sync_single_for_cpu(pdev,
+                       pci_dma_sync_single_for_cpu(pdev,
                                        rxdr->buffer_info[l].dma,
                                        rxdr->buffer_info[l].length,
                                        PCI_DMA_FROMDEVICE);
-       
+
                        ret_val = e1000_check_lbtest_frame(
                                        rxdr->buffer_info[l].skb,
                                        1024);
                        if(!ret_val)
                                good_cnt++;
                        if(unlikely(++l == rxdr->count)) l = 0;
-                       /* time + 20 msecs (200 msecs on 2.4) is more than
-                        * enough time to complete the receives, if it's
+                       /* time + 20 msecs (200 msecs on 2.4) is more than
+                        * enough time to complete the receives, if it's
                         * exceeded, break and error off
                         */
-               } while (good_cnt < 64 && jiffies < (time + 20));
+               } while(good_cnt < 64 && jiffies < (time + 20));
                if(good_cnt != 64) {
                        ret_val = 13; /* ret_val is the same as mis-compare */
-                       break;
+                       break;
                }
                if(jiffies >= (time + 2)) {
                        ret_val = 14; /* error code for time out error */
@@ -1540,7 +1540,7 @@ e1000_link_test(struct e1000_adapter *ad
        return *data;
 }

-static int
+static int
 e1000_diag_test_count(struct net_device *netdev)
 {
        return E1000_TEST_LEN;
@@ -1745,7 +1689,7 @@ e1000_phys_id(struct net_device *netdev,
        }
        else {
                E1000_WRITE_REG(&adapter->hw, LEDCTL, 
(E1000_LEDCTL_LED2_BLINK_RATE |
-                       E1000_LEDCTL_LED1_BLINK | E1000_LEDCTL_LED2_BLINK |
+                       E1000_LEDCTL_LED1_BLINK | E1000_LEDCTL_LED2_BLINK |
                        (E1000_LEDCTL_MODE_LED_ON << 
E1000_LEDCTL_LED2_MODE_SHIFT) |
                        (E1000_LEDCTL_MODE_LINK_ACTIVITY << 
E1000_LEDCTL_LED1_MODE_SHIFT) |
                        (E1000_LEDCTL_MODE_LED_OFF << 
E1000_LEDCTL_LED0_MODE_SHIFT)));
@@ -1770,14 +1716,14 @@ e1000_nway_reset(struct net_device *netd
        return 0;
 }

-static int
+static int
 e1000_get_stats_count(struct net_device *netdev)
 {
        return E1000_STATS_LEN;
 }

-static void
-e1000_get_ethtool_stats(struct net_device *netdev,
+static void
+e1000_get_ethtool_stats(struct net_device *netdev,
                struct ethtool_stats *stats, uint64_t *data)
 {
        struct e1000_adapter *adapter = netdev_priv(netdev);
@@ -1791,7 +1716,7 @@
        }
 }

-static void
+static void
 e1000_get_strings(struct net_device *netdev, uint32_t stringset, uint8_t *data)
 {
        int i;
@@ -1798,7 +1716,7 @@

        switch(stringset) {
        case ETH_SS_TEST:
-               memcpy(data, *e1000_gstrings_test,
+               memcpy(data, *e1000_gstrings_test,
                        E1000_TEST_LEN*ETH_GSTRING_LEN);
                break;
        case ETH_SS_STATS:
@@ -1832,7 +1832,7 @@ static struct ethtool_ops e1000_ethtool_
        .set_pauseparam         = e1000_set_pauseparam,
        .get_rx_csum            = e1000_get_rx_csum,
        .set_rx_csum            = e1000_set_rx_csum,
-       .get_tx_csum            = e1000_get_tx_csum,
+       .get_tx_csum            = e1000_get_tx_csum,
        .set_tx_csum            = e1000_set_tx_csum,
        .get_sg                 = ethtool_op_get_sg,
        .set_sg                 = ethtool_op_set_sg,
diff -up linux-2.6/drivers/net/e1000/e1000_hw.c
linux-2.6.new/drivers/net/e1000/e1000_hw.c
--- linux-2.6/drivers/net/e1000/e1000_hw.c      2005-11-14 16:20:34.000000000 
-0800
+++ linux-2.6.new/drivers/net/e1000/e1000_hw.c  2005-11-04
01:23:40.000000000 -0800
@@ -6815,7 +6815,8 @@ int32_t
 e1000_check_phy_reset_block(struct e1000_hw *hw)
 {
     uint32_t manc = 0;
-    if(hw->mac_type > e1000_82547_rev_2)
+
+    if (hw->mac_type > e1000_82547_rev_2)
         manc = E1000_READ_REG(hw, MANC);
     return (manc & E1000_MANC_BLK_PHY_RST_ON_IDE) ?
            E1000_BLK_PHY_RESET : E1000_SUCCESS;
diff -up linux-2.6/drivers/net/e1000/e1000_hw.h
linux-2.6.new/drivers/net/e1000/e1000_hw.h
--- linux-2.6/drivers/net/e1000/e1000_hw.h      2005-11-14 16:20:34.000000000 
-0800
+++ linux-2.6.new/drivers/net/e1000/e1000_hw.h  2005-11-04
01:23:40.000000000 -0800
@@ -898,14 +898,14 @@ struct e1000_ffvt_entry {
 #define E1000_TXDCTL   0x03828  /* TX Descriptor Control - RW */
 #define E1000_TADV     0x0382C  /* TX Interrupt Absolute Delay Val - RW */
 #define E1000_TSPMT    0x03830  /* TCP Segmentation PAD & Min Threshold - RW */
-#define E1000_TARC0    0x03840 /* TX Arbitration Count (0) */
-#define E1000_TDBAL1   0x03900 /* TX Desc Base Address Low (1) - RW */
-#define E1000_TDBAH1   0x03904 /* TX Desc Base Address High (1) - RW */
-#define E1000_TDLEN1   0x03908 /* TX Desc Length (1) - RW */
-#define E1000_TDH1     0x03910 /* TX Desc Head (1) - RW */
-#define E1000_TDT1     0x03918 /* TX Desc Tail (1) - RW */
-#define E1000_TXDCTL1  0x03928 /* TX Descriptor Control (1) - RW */
-#define E1000_TARC1    0x03940 /* TX Arbitration Count (1) */
+#define E1000_TARC0    0x03840  /* TX Arbitration Count (0) */
+#define E1000_TDBAL1   0x03900  /* TX Desc Base Address Low (1) - RW */
+#define E1000_TDBAH1   0x03904  /* TX Desc Base Address High (1) - RW */
+#define E1000_TDLEN1   0x03908  /* TX Desc Length (1) - RW */
+#define E1000_TDH1     0x03910  /* TX Desc Head (1) - RW */
+#define E1000_TDT1     0x03918  /* TX Desc Tail (1) - RW */
+#define E1000_TXDCTL1  0x03928  /* TX Descriptor Control (1) - RW */
+#define E1000_TARC1    0x03940  /* TX Arbitration Count (1) */
 #define E1000_CRCERRS  0x04000  /* CRC Error Count - R/clr */
 #define E1000_ALGNERRC 0x04004  /* Alignment Error Count - R/clr */
 #define E1000_SYMERRS  0x04008  /* Symbol Error Count - R/clr */
diff -up linux-2.6/drivers/net/e1000/e1000_main.c
linux-2.6.new/drivers/net/e1000/e1000_main.c
--- linux-2.6/drivers/net/e1000/e1000_main.c    2005-11-14 16:20:34.000000000 
-0800
+++ linux-2.6.new/drivers/net/e1000/e1000_main.c        2005-11-04
01:23:40.000000000 -0800
@@ -389,15 +389,15 @@ e1000_update_mng_vlan(struct e1000_adapt
                                adapter->mng_vlan_id = vid;
                        } else
                                adapter->mng_vlan_id = E1000_MNG_VLAN_NONE;
-                               
+
                        if((old_vid != (uint16_t)E1000_MNG_VLAN_NONE) &&
-                                       (vid != old_vid) &&
+                                       (vid != old_vid) &&
                                        !adapter->vlgrp->vlan_devices[old_vid])
                                e1000_vlan_rx_kill_vid(netdev, old_vid);
                }
        }
 }
-       
+
 int
 e1000_up(struct e1000_adapter *adapter)
 {
@@ -720,9 +641,9 @@ e1000_probe(struct pci_dev *pdev,

        adapter->en_mng_pt = e1000_enable_mng_pass_thru(&adapter->hw);

-       /* before reading the EEPROM, reset the controller to
+       /* before reading the EEPROM, reset the controller to
         * put the device in a known good starting state */
-       
+
        e1000_reset_hw(&adapter->hw);

        /* make sure the EEPROM is good */
@@ -1176,10 +1070,10 @@ e1000_open(struct net_device *netdev)
        if ((err = e1000_setup_all_rx_resources(adapter)))
                goto err_setup_rx;

-       if((err = e1000_up(adapter)))
+       if ((err = e1000_up(adapter)))
                goto err_up;
        adapter->mng_vlan_id = E1000_MNG_VLAN_NONE;
-       if((adapter->hw.mng_cookie.status &
+       if ((adapter->hw.mng_cookie.status &
                          E1000_MNG_DHCP_COOKIE_STATUS_VLAN_SUPPORT)) {
                e1000_update_mng_vlan(adapter);
        }
@@ -1284,7 +1158,7 @@ e1000_setup_tx_resources(struct e1000_ad
        size = sizeof(struct e1000_buffer) * txdr->count;

        txdr->buffer_info = vmalloc_node(size, pcibus_to_node(pdev->bus));
-       if(!txdr->buffer_info) {
+       if (!txdr->buffer_info) {
                DPRINTK(PROBE, ERR,
                "Unable to allocate memory for the transmit descriptor ring\n");
                return -ENOMEM;
@@ -1297,7 +1297,7 @@ e1000_setup_tx_resources(struct e1000_ad
        E1000_ROUNDUP(txdr->size, 4096);

        txdr->desc = pci_alloc_consistent(pdev, txdr->size, &txdr->dma);
-       if(!txdr->desc) {
+       if (!txdr->desc) {
 setup_tx_desc_die:
                vfree(txdr->buffer_info);
                DPRINTK(PROBE, ERR,
@@ -1313,8 +1313,8 @@ setup_tx_desc_die:
                                     "at %p\n", txdr->size, txdr->desc);
                /* Try again, without freeing the previous */
                txdr->desc = pci_alloc_consistent(pdev, txdr->size, &txdr->dma);
-               if(!txdr->desc) {
                /* Failed allocation, critical failure */
+               if (!txdr->desc) {
                        pci_free_consistent(pdev, txdr->size, olddesc, olddma);
                        goto setup_tx_desc_die;
                }
@@ -1509,7 +1509,7 @@ e1000_setup_rx_resources(struct e1000_ad

        size = sizeof(struct e1000_ps_page) * rxdr->count;
        rxdr->ps_page = kmalloc(size, GFP_KERNEL);
-       if(!rxdr->ps_page) {
+       if (!rxdr->ps_page) {
                vfree(rxdr->buffer_info);
                DPRINTK(PROBE, ERR,
                "Unable to allocate memory for the receive descriptor ring\n");
@@ -1519,7 +1519,7 @@ e1000_setup_rx_resources(struct e1000_ad

        size = sizeof(struct e1000_ps_page_dma) * rxdr->count;
        rxdr->ps_page_dma = kmalloc(size, GFP_KERNEL);
-       if(!rxdr->ps_page_dma) {
+       if (!rxdr->ps_page_dma) {
                vfree(rxdr->buffer_info);
                kfree(rxdr->ps_page);
                DPRINTK(PROBE, ERR,
@@ -1528,7 +1528,7 @@ e1000_setup_rx_resources(struct e1000_ad
        }
        memset(rxdr->ps_page_dma, 0, size);

-       if(adapter->hw.mac_type <= e1000_82547_rev_2)
+       if (adapter->hw.mac_type <= e1000_82547_rev_2)
                desc_len = sizeof(struct e1000_rx_desc);
        else
                desc_len = sizeof(union e1000_rx_desc_packet_split);
@@ -1691,7 +1691,7 @@ e1000_setup_rctl(struct e1000_adapter *a
                E1000_WRITE_REG(&adapter->hw, RFCTL, rfctl);

                rctl |= E1000_RCTL_DTYP_PS | E1000_RCTL_SECRC;
-               
+
                psrctl |= adapter->rx_ps_bsize0 >>
                        E1000_PSRCTL_BSIZE0_SHIFT;

@@ -1842,7 +1842,7 @@ e1000_configure_rx(struct e1000_adapter

                        /* Enable 82571 IPv4 payload checksum for UDP fragments
                         * Must be used in conjunction with packet-split. */
-                       if ((hw->mac_type >= e1000_82571) &&
+                       if ((hw->mac_type >= e1000_82571) &&
                           (adapter->rx_ps_pages)) {
                                rxcsum |= E1000_RXCSUM_IPPCSE;
                        }
@@ -1905,7 +1791,7 @@ static inline void
 e1000_unmap_and_free_tx_resource(struct e1000_adapter *adapter,
                        struct e1000_buffer *buffer_info)
 {
-       if(buffer_info->dma) {
+       if (buffer_info->dma) {
                pci_unmap_page(adapter->pdev,
                                buffer_info->dma,
                                buffer_info->length,
@@ -1932,7 +1820,7 @@ e1000_clean_tx_ring(struct e1000_adapter

        /* Free all the Tx ring sk_buffs */

-       for(i = 0; i < tx_ring->count; i++) {
+       for (i = 0; i < tx_ring->count; i++) {
                buffer_info = &tx_ring->buffer_info[i];
                e1000_unmap_and_free_tx_resource(adapter, buffer_info);
        }
@@ -2172,13 +2172,13 @@ e1000_set_mac(struct net_device *netdev,
                /* activate the work around */
                adapter->hw.laa_is_present = 1;

-               /* Hold a copy of the LAA in RAR[14] This is done so that
-                * between the time RAR[0] gets clobbered  and the time it
-                * gets fixed (in e1000_watchdog), the actual LAA is in one
+               /* Hold a copy of the LAA in RAR[14] This is done so that
+                * between the time RAR[0] gets clobbered  and the time it
+                * gets fixed (in e1000_watchdog), the actual LAA is in one
                 * of the RARs and no incoming packets directed to this port
-                * are dropped. Eventaully the LAA will be in RAR[0] and
+                * are dropped. Eventaully the LAA will be in RAR[0] and
                 * RAR[14] */
-               e1000_rar_set(&adapter->hw, adapter->hw.mac_addr,
+               e1000_rar_set(&adapter->hw, adapter->hw.mac_addr,
                                        E1000_RAR_ENTRIES - 1);
        }

@@ -2216,9 +2216,9 @@ e1000_set_multi(struct net_device *netde

        rctl = E1000_READ_REG(hw, RCTL);

-       if(netdev->flags & IFF_PROMISC) {
+       if (netdev->flags & IFF_PROMISC) {
                rctl |= (E1000_RCTL_UPE | E1000_RCTL_MPE);
-       } else if(netdev->flags & IFF_ALLMULTI) {
+       } else if (netdev->flags & IFF_ALLMULTI) {
                rctl |= E1000_RCTL_MPE;
                rctl &= ~E1000_RCTL_UPE;
        } else {
@@ -2229,7 +2229,7 @@ e1000_set_multi(struct net_device *netde

        /* 82542 2.0 needs to be in reset to write receive address registers */

-       if(hw->mac_type == e1000_82542_rev2_0)
+       if (hw->mac_type == e1000_82542_rev2_0)
                e1000_enter_82542_rst(adapter);

        /* load the first 14 multicast address into the exact filters 1-14
@@ -2239,7 +2239,7 @@ e1000_set_multi(struct net_device *netde
         */
        mc_ptr = netdev->mc_list;

-       for(i = 1; i < rar_entries; i++) {
+       for (i = 1; i < rar_entries; i++) {
                if (mc_ptr) {
                        e1000_rar_set(hw, mc_ptr->dmi_addr, i);
                        mc_ptr = mc_ptr->next;
@@ -2251,17 +2251,17 @@ e1000_set_multi(struct net_device *netde

        /* clear the old settings from the multicast hash table */

-       for(i = 0; i < E1000_NUM_MTA_REGISTERS; i++)
+       for (i = 0; i < E1000_NUM_MTA_REGISTERS; i++)
                E1000_WRITE_REG_ARRAY(hw, MTA, i, 0);

        /* load any remaining addresses into the hash table */

-       for(; mc_ptr; mc_ptr = mc_ptr->next) {
+       for (; mc_ptr; mc_ptr = mc_ptr->next) {
                hash_value = e1000_hash_mc_addr(hw, mc_ptr->dmi_addr);
                e1000_mta_set(hw, hash_value);
        }

-       if(hw->mac_type == e1000_82542_rev2_0)
+       if (hw->mac_type == e1000_82542_rev2_0)
                e1000_leave_82542_rst(adapter);
 }

@@ -2340,18 +2340,18 @@ e1000_watchdog_task(struct e1000_adapter
        e1000_check_for_link(&adapter->hw);
        if (adapter->hw.mac_type == e1000_82573) {
                e1000_enable_tx_pkt_filtering(&adapter->hw);
-               if(adapter->mng_vlan_id != adapter->hw.mng_cookie.vlan_id)
+               if (adapter->mng_vlan_id != adapter->hw.mng_cookie.vlan_id)
                        e1000_update_mng_vlan(adapter);
-       }       
+       }

-       if((adapter->hw.media_type == e1000_media_type_internal_serdes) &&
+       if ((adapter->hw.media_type == e1000_media_type_internal_serdes) &&
           !(E1000_READ_REG(&adapter->hw, TXCW) & E1000_TXCW_ANE))
                link = !adapter->hw.serdes_link_down;
        else
                link = E1000_READ_REG(&adapter->hw, STATUS) & E1000_STATUS_LU;

-       if(link) {
-               if(!netif_carrier_ok(netdev)) {
+       if (link) {
+               if (!netif_carrier_ok(netdev)) {
                        e1000_get_speed_and_duplex(&adapter->hw,
                                                   &adapter->link_speed,
                                                   &adapter->link_duplex);
@@ -2382,7 +2234,7 @@ e1000_watchdog_task(struct e1000_adapter
                        adapter->smartspeed = 0;
                }
        } else {
-               if(netif_carrier_ok(netdev)) {
+               if (netif_carrier_ok(netdev)) {
                        adapter->link_speed = 0;
                        adapter->link_duplex = 0;
                        DPRINTK(LINK, INFO, "NIC Link is Down\n");
@@ -2419,12 +2419,12 @@ e1000_watchdog_task(struct e1000_adapter
        }

        /* Dynamic mode for Interrupt Throttle Rate (ITR) */
-       if(adapter->hw.mac_type >= e1000_82540 && adapter->itr == 1) {
+       if (adapter->hw.mac_type >= e1000_82540 && adapter->itr == 1) {
                /* Symmetric Tx/Rx gets a reduced ITR=2000; Total
                 * asymmetrical Tx or Rx gets ITR=8000; everyone
                 * else is between 2000-8000. */
                uint32_t goc = (adapter->gotcl + adapter->gorcl) / 10000;
-               uint32_t dif = (adapter->gotcl > adapter->gorcl ?
+               uint32_t dif = (adapter->gotcl > adapter->gorcl ?
                        adapter->gotcl - adapter->gorcl :
                        adapter->gorcl - adapter->gotcl) / 10000;
                uint32_t itr = goc > 0 ? (dif * 6000 / goc + 2000) : 8000;
@@ -2437,7 +2437,7 @@ e1000_watchdog_task(struct e1000_adapter
        /* Force detection of hung controller every watchdog period */
        adapter->detect_tx_hung = TRUE;

-       /* With 82571 controllers, LAA may be overwritten due to controller
+       /* With 82571 controllers, LAA may be overwritten due to controller
         * reset from the other port. Set the appropriate LAA in RAR[0] */
        if (adapter->hw.mac_type == e1000_82571 && adapter->hw.laa_is_present)
                e1000_rar_set(&adapter->hw, adapter->hw.mac_addr, 0);
@@ -2466,7 +2322,7 @@ e1000_tso(struct e1000_adapter *adapter,
        int err;
        struct e1000_buffer *buffer_info;

-       if(skb_shinfo(skb)->tso_size) {
+       if (skb_shinfo(skb)->tso_size) {
                if (skb_header_cloned(skb)) {
                        err = pskb_expand_head(skb, 0, 0, GFP_ATOMIC);
                        if (err)
@@ -2475,7 +2475,7 @@ e1000_tso(struct e1000_adapter *adapter,

                hdr_len = ((skb->h.raw - skb->data) + (skb->h.th->doff << 2));
                mss = skb_shinfo(skb)->tso_size;
-               if(skb->protocol == ntohs(ETH_P_IP)) {
+               if (skb->protocol == ntohs(ETH_P_IP)) {
                        skb->nh.iph->tot_len = 0;
                        skb->nh.iph->check = 0;
                        skb->h.th->check =
@@ -2487,7 +2487,7 @@ e1000_tso(struct e1000_adapter *adapter,
                        cmd_length = E1000_TXD_CMD_IP;
                        ipcse = skb->h.raw - skb->data - 1;
 #ifdef NETIF_F_TSO_IPV6
-               } else if(skb->protocol == ntohs(ETH_P_IPV6)) {
+               } else if (skb->protocol == ntohs(ETH_P_IPV6)) {
                        skb->nh.ipv6h->payload_len = 0;
                        skb->h.th->check =
                                ~csum_ipv6_magic(&skb->nh.ipv6h->saddr,
@@ -2542,7 +2394,7 @@ e1000_tx_csum(struct e1000_adapter *adap
        uint8_t css;
        struct e1000_buffer *buffer_info;

-       if(likely(skb->ip_summed == CHECKSUM_HW)) {
+       if (likely(skb->ip_summed == CHECKSUM_HW)) {
                css = skb->h.raw - skb->data;

                i = tx_ring->next_to_use;
@@ -2609,7 +2609,7 @@ e1000_tx_map(struct e1000_adapter *adapt
                if(unlikely((adapter->hw.bus_type == e1000_bus_type_pcix) &&
                                (size > 2015) && count == 0))
                        size = 2015;
-
+
                /* Workaround for potential 82544 hang in PCI-X.  Avoid
                 * terminating buffers within evenly-aligned dwords. */
                if(unlikely(adapter->pcix_82544 &&
@@ -2783,11 +2620,11 @@ e1000_transfer_dhcp_info(struct e1000_ad
        if ((skb->len > MINIMUM_DHCP_PACKET_SIZE) && (!skb->protocol)) {
                struct ethhdr *eth = (struct ethhdr *) skb->data;
                if((htons(ETH_P_IP) == eth->h_proto)) {
-                       const struct iphdr *ip =
+                       const struct iphdr *ip =
                                (struct iphdr *)((uint8_t *)skb->data+14);
                        if(IPPROTO_UDP == ip->protocol) {
-                               struct udphdr *udp =
-                                       (struct udphdr *)((uint8_t *)ip +
+                               struct udphdr *udp =
+                                       (struct udphdr *)((uint8_t *)ip +
                                                (ip->ihl << 2));
                                if(ntohs(udp->dest) == 67) {
                                        offset = (uint8_t *)udp + 8 - skb->data;
@@ -2794,7 +2620,7 @@
                                        length = skb->len - offset;

                                        return e1000_mng_write_dhcp_info(hw,
-                                                       (uint8_t *)udp + 8,
+                                                       (uint8_t *)udp + 8,
                                                        length);
                                }
                        }
@@ -2817,7 +2817,7 @@ e1000_xmit_frame(struct sk_buff *skb, st
        unsigned int nr_frags = 0;
        unsigned int mss = 0;
        int count = 0;
-       int tso;
+       int tso;
        unsigned int f;
        len -= skb->data_len;

@@ -2885,7 +2704,7 @@ e1000_xmit_frame(struct sk_buff *skb, st
        if(adapter->pcix_82544)
                count++;

-       /* work-around for errata 10 and it applies to all controllers
+       /* work-around for errata 10 and it applies to all controllers
         * in PCI-X mode, so add one more descriptor to the count
         */
        if(unlikely((adapter->hw.bus_type == e1000_bus_type_pcix) &&
@@ -2900,7 +2723,8 @@ e1000_xmit_frame(struct sk_buff *skb, st
                count += nr_frags;


-       if(adapter->hw.tx_pkt_filtering && (adapter->hw.mac_type == 
e1000_82573) )
+       if (adapter->hw.tx_pkt_filtering &&
+           (adapter->hw.mac_type == e1000_82573))
                e1000_transfer_dhcp_info(adapter, skb);

        local_irq_save(flags);
@@ -2933,7 +2775,7 @@ e1000_xmit_frame(struct sk_buff *skb, st
        }

        first = tx_ring->next_to_use;
-       
+
        tso = e1000_tso(adapter, tx_ring, skb);
        if (tso < 0) {
                dev_kfree_skb_any(skb);
@@ -3023,15 +3024,15 @@ e1000_change_mtu(struct net_device *netd
        int max_frame = new_mtu + ENET_HEADER_SIZE + ETHERNET_FCS_SIZE;

        if((max_frame < MINIMUM_ETHERNET_FRAME_SIZE) ||
-               (max_frame > MAX_JUMBO_FRAME_SIZE)) {
-                       DPRINTK(PROBE, ERR, "Invalid MTU setting\n");
-                       return -EINVAL;
+          (max_frame > MAX_JUMBO_FRAME_SIZE)) {
+               DPRINTK(PROBE, ERR, "Invalid MTU setting\n");
+               return -EINVAL;
        }

 #define MAX_STD_JUMBO_FRAME_SIZE 9234
        /* might want this to be bigger enum check... */
        /* 82571 controllers limit jumbo frame size to 10500 bytes */
-       if ((adapter->hw.mac_type == e1000_82571 ||
+       if ((adapter->hw.mac_type == e1000_82571 ||
             adapter->hw.mac_type == e1000_82572) &&
            max_frame > MAX_STD_JUMBO_FRAME_SIZE) {
                DPRINTK(PROBE, ERR, "MTU > 9216 bytes not supported "
@@ -3303,7 +3137,6 @@ e1000_intr(int irq, void *data, struct p

 #else /* if !CONFIG_E1000_NAPI */
        /* Writing IMC and IMS is needed for 82547.
-       if(hw->mac_type == e1000_82547 || hw->mac_type == e1000_82547_rev_2){
         * Due to Hub Link bus being occupied, an interrupt
         * de-assertion message is not able to be sent.
         * When an interrupt assertion message is generated later,
@@ -3313,16 +3137,17 @@
         * in dead lock. Writing IMC forces 82547 into
         * de-assertion state.
         */
+       if (hw->mac_type == e1000_82547 || hw->mac_type == e1000_82547_rev_2) {
                atomic_inc(&adapter->irq_sem);
                E1000_WRITE_REG(hw, IMC, ~0);
        }

-       for(i = 0; i < E1000_MAX_INTR; i++)
+       for (i = 0; i < E1000_MAX_INTR; i++)
                if(unlikely(!adapter->clean_rx(adapter, adapter->rx_ring) &
                   !e1000_clean_tx_irq(adapter, adapter->tx_ring)))
                        break;

-       if(hw->mac_type == e1000_82547 || hw->mac_type == e1000_82547_rev_2)
+       if (hw->mac_type == e1000_82547 || hw->mac_type == e1000_82547_rev_2)
                e1000_irq_enable(adapter);

 #endif /* CONFIG_E1000_NAPI */
@@ -3362,7 +3363,7 @@ e1000_clean(struct net_device *poll_dev,

        *budget -= work_done;
        poll_dev->quota -= work_done;
-       
+
        /* If no Tx and not enough Rx work done, exit the polling mode */
        if((!tx_cleaned && (work_done == 0)) ||
           !netif_running(adapter->netdev)) {
@@ -3396,7 +3252,7 @@ e1000_clean_tx_irq(struct e1000_adapter
        eop_desc = E1000_TX_DESC(*tx_ring, eop);

        while (eop_desc->upper.data & cpu_to_le32(E1000_TXD_STAT_DD)) {
-               for(cleaned = FALSE; !cleaned; ) {
+               for (cleaned = FALSE; !cleaned; ) {
                        tx_desc = E1000_TX_DESC(*tx_ring, i);
                        buffer_info = &tx_ring->buffer_info[i];
                        cleaned = (i == eop);
@@ -3404,7 +3252,7 @@
                        e1000_unmap_and_free_tx_resource(adapter, buffer_info);
                        memset(tx_desc, 0, sizeof(struct e1000_tx_desc));

-                       if(unlikely(++i == tx_ring->count)) i = 0;
+                       if (unlikely(++i == tx_ring->count)) i = 0;
                }

                tx_ring->pkt++;
@@ -3417,7 +3418,7 @@ e1000_clean_tx_irq(struct e1000_adapter

        spin_lock(&tx_ring->tx_lock);

-       if(unlikely(cleaned && netif_queue_stopped(netdev) &&
+       if (unlikely(cleaned && netif_queue_stopped(netdev) &&
                    netif_carrier_ok(netdev)))
                netif_wake_queue(netdev);

@@ -3427,10 +3316,10 @@ e1000_clean_tx_irq(struct e1000_adapter
                /* Detect a transmit hang in hardware, this serializes the
                 * check with the clearing of time_stamp and movement of i */
                adapter->detect_tx_hung = FALSE;
-               if (tx_ring->buffer_info[i].dma &&
-                   time_after(jiffies, tx_ring->buffer_info[i].time_stamp + HZ)
+               if (tx_ring->buffer_info[eop].dma &&
+                   time_after(jiffies, tx_ring->buffer_info[eop].time_stamp+HZ)
                    && !(E1000_READ_REG(&adapter->hw, STATUS) &
-                       E1000_STATUS_TXOFF)) {
+                        E1000_STATUS_TXOFF)) {

                        /* detected Tx unit hang */
                        DPRINTK(DRV, ERR, "Detected Tx Unit Hang\n"
@@ -4006,7 +3786,7 @@ e1000_alloc_rx_buffers_ps(struct e1000_a
        while (cleaned_count--) {
                rx_desc = E1000_RX_DESC_PS(*rx_ring, i);

-               for(j = 0; j < PS_PAGE_BUFFERS; j++) {
+               for (j = 0; j < PS_PAGE_BUFFERS; j++) {
                        if (j < adapter->rx_ps_pages) {
                                if (likely(!ps_page->ps_page[j])) {
                                        ps_page->ps_page[j] =
@@ -4022,7 +4023,7 @@ e1000_alloc_rx_buffers_ps(struct e1000_a
                                                            PCI_DMA_FROMDEVICE);
                                }
                                /* Refresh the desc even if buffer_addrs didn't
-                                * change because each write-back erases
+                                * change because each write-back erases
                                 * this info.
                                 */
                                rx_desc->read.buffer_addr[j+1] =
@@ -4054,7 +3833,7 @@ e1000_alloc_rx_buffers_ps(struct e1000_a

                rx_desc->read.buffer_addr[0] = cpu_to_le64(buffer_info->dma);

-               if(unlikely(++i == rx_ring->count)) i = 0;
+               if (unlikely(++i == rx_ring->count)) i = 0;
                buffer_info = &rx_ring->buffer_info[i];
                ps_page = &rx_ring->ps_page[i];
                ps_page_dma = &rx_ring->ps_page_dma[i];
@@ -4222,7 +4223,7 @@ e1000_mii_ioctl(struct net_device *netde
                                                                    spddplx);
                                        if(retval) {
                                                spin_unlock_irqrestore(
-                                                       &adapter->stats_lock,
+                                                       &adapter->stats_lock,
                                                        flags);
                                                return retval;
                                        }
diff -up linux-2.6/drivers/net/e1000/e1000_param.c
linux-2.6.new/drivers/net/e1000/e1000_param.c
--- linux-2.6/drivers/net/e1000/e1000_param.c   2005-11-14
16:20:34.000000000 -0800
+++ linux-2.6.new/drivers/net/e1000/e1000_param.c       2005-11-04
01:23:41.000000000 -0800
@@ -315,7 +315,7 @@ e1000_check_options(struct e1000_adapter
                if (num_TxDescriptors > bd) {
                        tx_ring->count = TxDescriptors[bd];
                        e1000_validate_option(&tx_ring->count, &opt, adapter);
-                       E1000_ROUNDUP(tx_ring->count,
+                       E1000_ROUNDUP(tx_ring->count,
                                                REQ_TX_DESCRIPTOR_MULTIPLE);
                } else {
                        tx_ring->count = opt.def;
@@ -341,7 +362,7 @@ e1000_check_options(struct e1000_adapter
                if (num_RxDescriptors > bd) {
                        rx_ring->count = RxDescriptors[bd];
                        e1000_validate_option(&rx_ring->count, &opt, adapter);
-                       E1000_ROUNDUP(rx_ring->count,
+                       E1000_ROUNDUP(rx_ring->count,
                                                REQ_RX_DESCRIPTOR_MULTIPLE);
                } else {
                        rx_ring->count = opt.def;
@@ -403,7 +437,7 @@ e1000_check_options(struct e1000_adapter

                if (num_TxIntDelay > bd) {
                        adapter->tx_int_delay = TxIntDelay[bd];
-                       e1000_validate_option(&adapter->tx_int_delay, &opt,
+                       e1000_validate_option(&adapter->tx_int_delay, &opt,
                                                                adapter);
                } else {
                        adapter->tx_int_delay = opt.def;
@@ -421,7 +459,7 @@ e1000_check_options(struct e1000_adapter

                if (num_TxAbsIntDelay > bd) {
                        adapter->tx_abs_int_delay = TxAbsIntDelay[bd];
-                       e1000_validate_option(&adapter->tx_abs_int_delay, &opt,
+                       e1000_validate_option(&adapter->tx_abs_int_delay, &opt,
                                                                adapter);
                } else {
                        adapter->tx_abs_int_delay = opt.def;
@@ -439,7 +481,7 @@ e1000_check_options(struct e1000_adapter

                if (num_RxIntDelay > bd) {
                        adapter->rx_int_delay = RxIntDelay[bd];
-                       e1000_validate_option(&adapter->rx_int_delay, &opt,
+                       e1000_validate_option(&adapter->rx_int_delay, &opt,
                                                                adapter);
                } else {
                        adapter->rx_int_delay = opt.def;
@@ -457,7 +503,7 @@ e1000_check_options(struct e1000_adapter

                if (num_RxAbsIntDelay > bd) {
                        adapter->rx_abs_int_delay = RxAbsIntDelay[bd];
-                       e1000_validate_option(&adapter->rx_abs_int_delay, &opt,
+                       e1000_validate_option(&adapter->rx_abs_int_delay, &opt,
                                                                adapter);
                } else {
                        adapter->rx_abs_int_delay = opt.def;
@@ -477,15 +525,15 @@ e1000_check_options(struct e1000_adapter
                        adapter->itr = InterruptThrottleRate[bd];
                        switch(adapter->itr) {
                        case 0:
-                               DPRINTK(PROBE, INFO, "%s turned off\n",
+                               DPRINTK(PROBE, INFO, "%s turned off\n",
                                        opt.name);
                                break;
                        case 1:
-                               DPRINTK(PROBE, INFO, "%s set to dynamic mode\n",
+                               DPRINTK(PROBE, INFO, "%s set to dynamic mode\n",
                                        opt.name);
                                break;
                        default:
-                               e1000_validate_option(&adapter->itr, &opt,
+                               e1000_validate_option(&adapter->itr, &opt,
                                        adapter);
                                break;
                        }
-
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to