igb: change pba size determination from if to switch statement
authorAlexander Duyck <alexander.h.duyck@intel.com>
Fri, 6 Feb 2009 23:21:31 +0000 (23:21 +0000)
committerDavid S. Miller <davem@davemloft.net>
Sat, 7 Feb 2009 10:43:14 +0000 (02:43 -0800)
As additional hardware is added to the igb driver it is easier to support
the expansion via switch statements instead of using nested ifs.  For
this reason I am changing this to a switch statement.

Signed-off-by: Alexander Duyck <alexander.h.duyck@intel.com>
Signed-off-by: Jeff Kirsher <jeffrey.t.kirsher@intel.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/igb/igb_main.c

index accab3f7357e4e4ae1c55100c3ac756db3cb03b1..d27e502097eb503351185429ab41adaf83784570 100644 (file)
@@ -888,11 +888,14 @@ void igb_reset(struct igb_adapter *adapter)
        /* Repartition Pba for greater than 9k mtu
         * To take effect CTRL.RST is required.
         */
-       if (mac->type != e1000_82576) {
-       pba = E1000_PBA_34K;
-       }
-       else {
+       switch (mac->type) {
+       case e1000_82576:
                pba = E1000_PBA_64K;
+               break;
+       case e1000_82575:
+       default:
+               pba = E1000_PBA_34K;
+               break;
        }
 
        if ((adapter->max_frame_size > ETH_FRAME_LEN + ETH_FCS_LEN) &&