netdev
[Top] [All Lists]

mv643xx(20/20): Fix promiscuous mode handling

To: Netdev <netdev@xxxxxxxxxxx>, Jeff Garzik <jgarzik@xxxxxxxxx>
Subject: mv643xx(20/20): Fix promiscuous mode handling
From: "Dale Farnsworth" <dale@xxxxxxxxxxxxxx>
Date: Mon, 28 Mar 2005 17:01:00 -0700
Cc: Ralf Baechle <ralf@xxxxxxxxxxxxxx>, Manish Lachwani <mlachwani@xxxxxxxxxx>, Brian Waite <brian@xxxxxxxxxxxxxx>, "Steven J. Hill" <sjhill@xxxxxxxxxxxxxxxxxx>, Benjamin Herrenschmidt <benh@xxxxxxxxxxxxxxxxxxx>, James Chapman <jchapman@xxxxxxxxxxx>
In-reply-to: <20050328233807.GA28423@xyzzy>
References: <20050328233807.GA28423@xyzzy>
Sender: netdev-bounce@xxxxxxxxxxx
User-agent: Mutt/1.5.6+20040907i
mv643xx_eth_get_config_reg() was reading the wrong register.
mv643xx_eth_set_config_reg() was or'ing instead of setting the
register.  These functions are trivial and both are called only from
mv643xx_eth_set_rx_mode() when changing the promiscuous mode.  Remove both
functions and do the operations directly in mv643xx_eth_set_rx_mode().

Also, maintain promiscuous mode setting across port resets.

Signed-off-by: Dale Farnsworth <dale@xxxxxxxxxxxxxx>

Index: linux-2.5-enet/drivers/net/mv643xx_eth.h
===================================================================
--- linux-2.5-enet.orig/drivers/net/mv643xx_eth.h
+++ linux-2.5-enet/drivers/net/mv643xx_eth.h
@@ -474,6 +474,7 @@
        int port_num;                   /* User Ethernet port number    */
        u8 port_mac_addr[6];            /* User defined port MAC address.*/
        u32 port_serial_control;        /* Port serial control value */
+       u32 port_config;                /* Port config register value */
        struct ethtool_cmd ethtool_cmd; /* ethtool_cmd used at open */
        u32 port_tx_queue_command;      /* Port active Tx queues summary*/
        u32 port_rx_queue_command;      /* Port active Rx queues summary*/
Index: linux-2.5-enet/drivers/net/mv643xx_eth.c
===================================================================
--- linux-2.5-enet.orig/drivers/net/mv643xx_eth.c
+++ linux-2.5-enet/drivers/net/mv643xx_eth.c
@@ -390,7 +390,7 @@
 
        /* Assign port configuration and command. */
        mv643xx_eth_write(MV643XX_ETH_PORT_CONFIG_REG(port_num),
-                         MV643XX_ETH_PORT_CONFIG_DEFAULT_VALUE);
+                         mp->port_config);
 
        mv643xx_eth_write(MV643XX_ETH_PORT_CONFIG_EXTEND_REG(port_num),
                          MV643XX_ETH_PORT_CONFIG_EXTEND_DEFAULT_VALUE);
@@ -683,38 +683,6 @@
        return 0;
 }
 
-/*
- * This function sets specified bits in the given ethernet
- * configuration register.
- */
-static void mv643xx_eth_set_config_reg(struct net_device *dev,
-                                      unsigned int value)
-{
-       struct mv643xx_private *mp = netdev_priv(dev);
-       unsigned int eth_port_num = mp->port_num;
-       unsigned int eth_config_reg;
-
-       eth_config_reg =
-               mv643xx_eth_read(MV643XX_ETH_PORT_CONFIG_REG(eth_port_num));
-       eth_config_reg |= value;
-       mv643xx_eth_write(MV643XX_ETH_PORT_CONFIG_REG(eth_port_num),
-                         eth_config_reg);
-}
-
-/*
- * This function returns the configuration register value of the given
- * ethernet port.
- */
-static unsigned int mv643xx_eth_get_config_reg(struct net_device *dev)
-{
-       struct mv643xx_private *mp = netdev_priv(dev);
-       unsigned int eth_config_reg;
-
-       eth_config_reg =
-           mv643xx_eth_read(MV643XX_ETH_PORT_CONFIG_EXTEND_REG(mp->port_num));
-       return eth_config_reg;
-}
-
 /*****************************************************************************
  * Device init and shutdown
  *****************************************************************************/
@@ -2027,14 +1995,14 @@
 
 static void mv643xx_eth_set_rx_mode(struct net_device *dev)
 {
-       u32 config_reg;
+       struct mv643xx_private *mp = netdev_priv(dev);
 
-       config_reg = mv643xx_eth_get_config_reg(dev);
        if (dev->flags & IFF_PROMISC)
-               config_reg |= (u32) MV643XX_ETH_UNICAST_PROMISCUOUS_MODE;
+               mp->port_config |= MV643XX_ETH_UNICAST_PROMISCUOUS_MODE;
        else
-               config_reg &= ~(u32) MV643XX_ETH_UNICAST_PROMISCUOUS_MODE;
-       mv643xx_eth_set_config_reg(dev, config_reg);
+               mp->port_config &= ~MV643XX_ETH_UNICAST_PROMISCUOUS_MODE;
+       mv643xx_eth_write(MV643XX_ETH_PORT_CONFIG_REG(mp->port_num),
+                         mp->port_config);
 }
 
 /*
@@ -2562,6 +2530,7 @@
        mp->port_serial_control |= MV643XX_ETH_DISABLE_AUTO_NEG_FOR_FLOW_CTRL |
                                   MV643XX_ETH_SERIAL_PORT_CONTROL_RESERVED   |
                                   MV643XX_ETH_DO_NOT_FORCE_LINK_FAIL;
+       mp->port_config = MV643XX_ETH_PORT_CONFIG_DEFAULT_VALUE;
        mp->rx_ring_size = MV643XX_ETH_PORT_DEFAULT_RECEIVE_QUEUE_SIZE;
        mp->tx_ring_size = MV643XX_ETH_PORT_DEFAULT_TRANSMIT_QUEUE_SIZE;
 

<Prev in Thread] Current Thread [Next in Thread>