sfc: Don't use enums as a bitmask.
authorDavid S. Miller <davem@davemloft.net>
Tue, 17 May 2011 21:53:22 +0000 (17:53 -0400)
committerDavid S. Miller <davem@davemloft.net>
Tue, 17 May 2011 21:53:22 +0000 (17:53 -0400)
This fixes:

drivers/net/sfc/mcdi_mac.c: In function ‘efx_mcdi_set_mac’:
drivers/net/sfc/mcdi_mac.c:36:2: warning: case value ‘3’ not in enumerated type ‘enum efx_fc_type’

Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/sfc/efx.c
drivers/net/sfc/efx.h
drivers/net/sfc/ethtool.c
drivers/net/sfc/mdio_10g.c
drivers/net/sfc/mdio_10g.h
drivers/net/sfc/net_driver.h

index 05502b359b9e0083dd37fe3d60b346fccb925fce..c914729f95546469c42d3d193bfa8ffb3bd235b7 100644 (file)
@@ -833,7 +833,7 @@ void efx_link_set_advertising(struct efx_nic *efx, u32 advertising)
        }
 }
 
-void efx_link_set_wanted_fc(struct efx_nic *efx, enum efx_fc_type wanted_fc)
+void efx_link_set_wanted_fc(struct efx_nic *efx, u8 wanted_fc)
 {
        efx->wanted_fc = wanted_fc;
        if (efx->link_advertising) {
index 3d83a1f74fef4557315a7d7ae9c29f6fe61f86ac..b0d1209ea18d091abee41ed339b4fe03b2b02c5c 100644 (file)
@@ -142,6 +142,6 @@ static inline void efx_schedule_channel(struct efx_channel *channel)
 
 extern void efx_link_status_changed(struct efx_nic *efx);
 extern void efx_link_set_advertising(struct efx_nic *efx, u32);
-extern void efx_link_set_wanted_fc(struct efx_nic *efx, enum efx_fc_type);
+extern void efx_link_set_wanted_fc(struct efx_nic *efx, u8);
 
 #endif /* EFX_EFX_H */
index 348437a8117eb9596a452b3534c0ba8fc9f10cb0..d229027dc36380c8585b598a2ec58980d621bab9 100644 (file)
@@ -698,7 +698,7 @@ static int efx_ethtool_set_pauseparam(struct net_device *net_dev,
                                      struct ethtool_pauseparam *pause)
 {
        struct efx_nic *efx = netdev_priv(net_dev);
-       enum efx_fc_type wanted_fc, old_fc;
+       u8 wanted_fc, old_fc;
        u32 old_adv;
        bool reset;
        int rc = 0;
index 71159145b4bf7cd2c33605a161472cef28514507..7ab385c8136ddfc25a6b261c97908458e2faa188 100644 (file)
@@ -284,7 +284,7 @@ void efx_mdio_an_reconfigure(struct efx_nic *efx)
        efx_mdio_write(efx, MDIO_MMD_AN, MDIO_CTRL1, reg);
 }
 
-enum efx_fc_type efx_mdio_get_pause(struct efx_nic *efx)
+u8 efx_mdio_get_pause(struct efx_nic *efx)
 {
        BUILD_BUG_ON(EFX_FC_AUTO & (EFX_FC_RX | EFX_FC_TX));
 
index df0703940c837094b57ac266529520445ec1f151..a97dbbd2de99880e6c59c35ea8e10195729841fc 100644 (file)
@@ -92,7 +92,7 @@ extern void efx_mdio_an_reconfigure(struct efx_nic *efx);
 /* Get pause parameters from AN if available (otherwise return
  * requested pause parameters)
  */
-enum efx_fc_type efx_mdio_get_pause(struct efx_nic *efx);
+u8 efx_mdio_get_pause(struct efx_nic *efx);
 
 /* Wait for specified MMDs to exit reset within a timeout */
 extern int efx_mdio_wait_reset_mmds(struct efx_nic *efx,
index ce9697bac1b93d18cf8cd9ee8fb866028413511f..e8d5f03a89fe3722207ce3fb1aa884f623b1e45a 100644 (file)
@@ -449,11 +449,9 @@ enum nic_state {
 struct efx_nic;
 
 /* Pseudo bit-mask flow control field */
-enum efx_fc_type {
-       EFX_FC_RX = FLOW_CTRL_RX,
-       EFX_FC_TX = FLOW_CTRL_TX,
-       EFX_FC_AUTO = 4,
-};
+#define EFX_FC_RX      FLOW_CTRL_RX
+#define EFX_FC_TX      FLOW_CTRL_TX
+#define EFX_FC_AUTO    4
 
 /**
  * struct efx_link_state - Current state of the link
@@ -465,7 +463,7 @@ enum efx_fc_type {
 struct efx_link_state {
        bool up;
        bool fd;
-       enum efx_fc_type fc;
+       u8 fc;
        unsigned int speed;
 };
 
@@ -784,7 +782,7 @@ struct efx_nic {
 
        bool promiscuous;
        union efx_multicast_hash multicast_hash;
-       enum efx_fc_type wanted_fc;
+       u8 wanted_fc;
 
        atomic_t rx_reset;
        enum efx_loopback_mode loopback_mode;