netxen: remove netxen_nic_isr.c
authorDhananjay Phadke <dhananjay@netxen.com>
Tue, 22 Jul 2008 02:44:05 +0000 (19:44 -0700)
committerJeff Garzik <jgarzik@redhat.com>
Tue, 22 Jul 2008 21:51:57 +0000 (17:51 -0400)
It had only couple of functions which are moved to main.c

Signed-off-by: Dhananjay Phadke <dhananjay@netxen.com>
Signed-off-by: Jeff Garzik <jgarzik@redhat.com>
drivers/net/netxen/Makefile
drivers/net/netxen/netxen_nic.h
drivers/net/netxen/netxen_nic_init.c
drivers/net/netxen/netxen_nic_isr.c [deleted file]
drivers/net/netxen/netxen_nic_main.c

index a07cdc6f7384fc5337cde587ba0879c77f374acf..c63a20790659ee35341ed0f1f0cc393766313d5b 100644 (file)
@@ -32,4 +32,4 @@
 obj-$(CONFIG_NETXEN_NIC) := netxen_nic.o
 
 netxen_nic-y := netxen_nic_hw.o netxen_nic_main.o netxen_nic_init.o \
-       netxen_nic_isr.o netxen_nic_ethtool.o netxen_nic_niu.o
+       netxen_nic_ethtool.o netxen_nic_niu.o
index b23a5a447f0cc6b37a5b1857d5ff8bc7e37a5bd9..87be0a6ef51a10c7647bd34e79c533355e5c4fd9 100644 (file)
@@ -810,8 +810,7 @@ struct netxen_hardware_context {
        u8 revision_id;
        u16 board_type;
        struct netxen_board_info boardcfg;
-       u32 xg_linkup;
-       u32 qg_linksup;
+       u32 linkup;
        /* Address of cmd ring in Phantom */
        struct cmd_desc_type0 *cmd_desc_head;
        dma_addr_t cmd_desc_phys_addr;
@@ -965,7 +964,6 @@ struct netxen_adapter {
        int msi_mode;
        int (*enable_phy_interrupts) (struct netxen_adapter *);
        int (*disable_phy_interrupts) (struct netxen_adapter *);
-       void (*handle_phy_intr) (struct netxen_adapter *);
        int (*macaddr_set) (struct netxen_adapter *, netxen_ethernet_macaddr_t);
        int (*set_mtu) (struct netxen_adapter *, int);
        int (*set_promisc) (struct netxen_adapter *, netxen_niu_prom_mode_t);
@@ -1049,8 +1047,6 @@ int netxen_niu_xgbe_enable_phy_interrupts(struct netxen_adapter *adapter);
 int netxen_niu_gbe_enable_phy_interrupts(struct netxen_adapter *adapter);
 int netxen_niu_xgbe_disable_phy_interrupts(struct netxen_adapter *adapter);
 int netxen_niu_gbe_disable_phy_interrupts(struct netxen_adapter *adapter);
-void netxen_nic_xgbe_handle_phy_intr(struct netxen_adapter *adapter);
-void netxen_nic_gbe_handle_phy_intr(struct netxen_adapter *adapter);
 int netxen_niu_gbe_phy_read(struct netxen_adapter *adapter, long reg,
                            __u32 * readval);
 int netxen_niu_gbe_phy_write(struct netxen_adapter *adapter,
index c6e2bc71d8f33e89bed81a7eedac22bf672b0dc0..7c3fbc4a5723f827976c4aa7edc9292d577758d6 100644 (file)
@@ -324,7 +324,6 @@ void netxen_initialize_adapter_ops(struct netxen_adapter *adapter)
                    netxen_niu_gbe_enable_phy_interrupts;
                adapter->disable_phy_interrupts =
                    netxen_niu_gbe_disable_phy_interrupts;
-               adapter->handle_phy_intr = netxen_nic_gbe_handle_phy_intr;
                adapter->macaddr_set = netxen_niu_macaddr_set;
                adapter->set_mtu = netxen_nic_set_mtu_gb;
                adapter->set_promisc = netxen_niu_set_promiscuous_mode;
@@ -339,7 +338,6 @@ void netxen_initialize_adapter_ops(struct netxen_adapter *adapter)
                    netxen_niu_xgbe_enable_phy_interrupts;
                adapter->disable_phy_interrupts =
                    netxen_niu_xgbe_disable_phy_interrupts;
-               adapter->handle_phy_intr = netxen_nic_xgbe_handle_phy_intr;
                adapter->macaddr_set = netxen_niu_xg_macaddr_set;
                adapter->set_mtu = netxen_nic_set_mtu_xgb;
                adapter->init_port = netxen_niu_xg_init_port;
@@ -1139,60 +1137,6 @@ int netxen_receive_peg_ready(struct netxen_adapter *adapter)
        return 0;
 }
 
-static int netxen_nic_check_temp(struct netxen_adapter *adapter)
-{
-       struct net_device *netdev = adapter->netdev;
-       uint32_t temp, temp_state, temp_val;
-       int rv = 0;
-
-       temp = adapter->pci_read_normalize(adapter, CRB_TEMP_STATE);
-
-       temp_state = nx_get_temp_state(temp);
-       temp_val = nx_get_temp_val(temp);
-
-       if (temp_state == NX_TEMP_PANIC) {
-               printk(KERN_ALERT
-                      "%s: Device temperature %d degrees C exceeds"
-                      " maximum allowed. Hardware has been shut down.\n",
-                      netxen_nic_driver_name, temp_val);
-
-               netif_carrier_off(netdev);
-               netif_stop_queue(netdev);
-               rv = 1;
-       } else if (temp_state == NX_TEMP_WARN) {
-               if (adapter->temp == NX_TEMP_NORMAL) {
-                       printk(KERN_ALERT
-                              "%s: Device temperature %d degrees C "
-                              "exceeds operating range."
-                              " Immediate action needed.\n",
-                              netxen_nic_driver_name, temp_val);
-               }
-       } else {
-               if (adapter->temp == NX_TEMP_WARN) {
-                       printk(KERN_INFO
-                              "%s: Device temperature is now %d degrees C"
-                              " in normal range.\n", netxen_nic_driver_name,
-                              temp_val);
-               }
-       }
-       adapter->temp = temp_state;
-       return rv;
-}
-
-void netxen_watchdog_task(struct work_struct *work)
-{
-       struct netxen_adapter *adapter =
-               container_of(work, struct netxen_adapter, watchdog_task);
-
-       if ((adapter->portnum  == 0) && netxen_nic_check_temp(adapter))
-               return;
-
-       if (adapter->handle_phy_intr)
-               adapter->handle_phy_intr(adapter);
-
-       mod_timer(&adapter->watchdog_timer, jiffies + 2 * HZ);
-}
-
 /*
  * netxen_process_rcv() send the received packet to the protocol stack.
  * and if the number of receives exceeds RX_BUFFERS_REFILL, then we
diff --git a/drivers/net/netxen/netxen_nic_isr.c b/drivers/net/netxen/netxen_nic_isr.c
deleted file mode 100644 (file)
index 104c287..0000000
+++ /dev/null
@@ -1,220 +0,0 @@
-/*
- * Copyright (C) 2003 - 2006 NetXen, Inc.
- * All rights reserved.
- *
- * This program is free software; you can redistribute it and/or
- * modify it under the terms of the GNU General Public License
- * as published by the Free Software Foundation; either version 2
- * of the License, or (at your option) any later version.
- *
- * This program is distributed in the hope that it will be useful, but
- * WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
- * GNU General Public License for more details.
- *
- * You should have received a copy of the GNU General Public License
- * along with this program; if not, write to the Free Software
- * Foundation, Inc., 59 Temple Place - Suite 330, Boston,
- * MA  02111-1307, USA.
- *
- * The full GNU General Public License is included in this distribution
- * in the file called LICENSE.
- *
- * Contact Information:
- *    info@netxen.com
- * NetXen,
- * 3965 Freedom Circle, Fourth floor,
- * Santa Clara, CA 95054
- */
-
-#include <linux/netdevice.h>
-#include <linux/delay.h>
-
-#include "netxen_nic.h"
-#include "netxen_nic_hw.h"
-#include "netxen_nic_phan_reg.h"
-
-/*
- * netxen_nic_get_stats - Get System Network Statistics
- * @netdev: network interface device structure
- */
-struct net_device_stats *netxen_nic_get_stats(struct net_device *netdev)
-{
-       struct netxen_adapter *adapter = netdev_priv(netdev);
-       struct net_device_stats *stats = &adapter->net_stats;
-
-       memset(stats, 0, sizeof(*stats));
-
-       /* total packets received   */
-       stats->rx_packets = adapter->stats.no_rcv;
-       /* total packets transmitted    */
-       stats->tx_packets = adapter->stats.xmitedframes +
-               adapter->stats.xmitfinished;
-       /* total bytes received     */
-       stats->rx_bytes = adapter->stats.rxbytes;
-       /* total bytes transmitted  */
-       stats->tx_bytes = adapter->stats.txbytes;
-       /* bad packets received     */
-       stats->rx_errors = adapter->stats.rcvdbadskb;
-       /* packet transmit problems */
-       stats->tx_errors = adapter->stats.nocmddescriptor;
-       /* no space in linux buffers    */
-       stats->rx_dropped = adapter->stats.rxdropped;
-       /* no space available in linux  */
-       stats->tx_dropped = adapter->stats.txdropped;
-
-       return stats;
-}
-
-static void netxen_indicate_link_status(struct netxen_adapter *adapter,
-                                       u32 link)
-{
-       struct net_device *netdev = adapter->netdev;
-
-       if (link)
-               netif_carrier_on(netdev);
-       else
-               netif_carrier_off(netdev);
-}
-
-#if 0
-void netxen_handle_port_int(struct netxen_adapter *adapter, u32 enable)
-{
-       __u32 int_src;
-
-       /*  This should clear the interrupt source */
-       if (adapter->phy_read)
-               adapter->phy_read(adapter,
-                                 NETXEN_NIU_GB_MII_MGMT_ADDR_INT_STATUS,
-                                 &int_src);
-       if (int_src == 0) {
-               DPRINTK(INFO, "No phy interrupts for port #%d\n", portno);
-               return;
-       }
-       if (adapter->disable_phy_interrupts)
-               adapter->disable_phy_interrupts(adapter);
-
-       if (netxen_get_phy_int_jabber(int_src))
-               DPRINTK(INFO, "Jabber interrupt \n");
-
-       if (netxen_get_phy_int_polarity_changed(int_src))
-               DPRINTK(INFO, "POLARITY CHANGED int \n");
-
-       if (netxen_get_phy_int_energy_detect(int_src))
-               DPRINTK(INFO, "ENERGY DETECT INT \n");
-
-       if (netxen_get_phy_int_downshift(int_src))
-               DPRINTK(INFO, "DOWNSHIFT INT \n");
-       /* write it down later.. */
-       if ((netxen_get_phy_int_speed_changed(int_src))
-           || (netxen_get_phy_int_link_status_changed(int_src))) {
-               __u32 status;
-
-               DPRINTK(INFO, "SPEED CHANGED OR LINK STATUS CHANGED \n");
-
-               if (adapter->phy_read
-                   && adapter->phy_read(adapter,
-                                        NETXEN_NIU_GB_MII_MGMT_ADDR_PHY_STATUS,
-                                        &status) == 0) {
-                       if (netxen_get_phy_int_link_status_changed(int_src)) {
-                               if (netxen_get_phy_link(status)) {
-                                       printk(KERN_INFO "%s: %s Link UP\n",
-                                              netxen_nic_driver_name,
-                                              adapter->netdev->name);
-
-                               } else {
-                                       printk(KERN_INFO "%s: %s Link DOWN\n",
-                                              netxen_nic_driver_name,
-                                              adapter->netdev->name);
-                               }
-                               netxen_indicate_link_status(adapter,
-                                                           netxen_get_phy_link
-                                                           (status));
-                       }
-               }
-       }
-       if (adapter->enable_phy_interrupts)
-               adapter->enable_phy_interrupts(adapter);
-}
-#endif  /*  0  */
-
-static void netxen_nic_isr_other(struct netxen_adapter *adapter)
-{
-       int portno = adapter->portnum;
-       u32 val, linkup, qg_linksup;
-
-       /* verify the offset */
-       val = adapter->pci_read_normalize(adapter, CRB_XG_STATE);
-       val = val >> adapter->physical_port;
-       if (val == adapter->ahw.qg_linksup)
-               return;
-
-       qg_linksup = adapter->ahw.qg_linksup;
-       adapter->ahw.qg_linksup = val;
-       DPRINTK(INFO, "link update 0x%08x\n", val);
-
-       linkup = val & 1;
-
-       if (linkup != (qg_linksup & 1)) {
-               printk(KERN_INFO "%s: %s PORT %d link %s\n",
-                      adapter->netdev->name,
-                      netxen_nic_driver_name, portno,
-                      ((linkup == 0) ? "down" : "up"));
-               netxen_indicate_link_status(adapter, linkup);
-               if (linkup)
-                       netxen_nic_set_link_parameters(adapter);
-
-       }
-}
-
-void netxen_nic_gbe_handle_phy_intr(struct netxen_adapter *adapter)
-{
-       netxen_nic_isr_other(adapter);
-}
-
-#if 0
-int netxen_nic_link_ok(struct netxen_adapter *adapter)
-{
-       switch (adapter->ahw.board_type) {
-       case NETXEN_NIC_GBE:
-               return ((adapter->ahw.qg_linksup) & 1);
-
-       case NETXEN_NIC_XGBE:
-               return ((adapter->ahw.xg_linkup) & 1);
-
-       default:
-               printk(KERN_ERR"%s: Function: %s, Unknown board type\n",
-                       netxen_nic_driver_name, __FUNCTION__);
-               break;
-       }
-
-       return 0;
-}
-#endif  /*  0  */
-
-void netxen_nic_xgbe_handle_phy_intr(struct netxen_adapter *adapter)
-{
-       struct net_device *netdev = adapter->netdev;
-       u32 val;
-
-       /* WINDOW = 1 */
-       val = adapter->pci_read_normalize(adapter, CRB_XG_STATE);
-       val >>= (adapter->physical_port * 8);
-       val &= 0xff;
-
-       if (adapter->ahw.xg_linkup == 1 && val != XG_LINK_UP) {
-               printk(KERN_INFO "%s: %s NIC Link is down\n",
-                      netxen_nic_driver_name, netdev->name);
-               adapter->ahw.xg_linkup = 0;
-               if (netif_running(netdev)) {
-                       netif_carrier_off(netdev);
-                       netif_stop_queue(netdev);
-               }
-       } else if (adapter->ahw.xg_linkup == 0 && val == XG_LINK_UP) {
-               printk(KERN_INFO "%s: %s NIC Link is up\n",
-                      netxen_nic_driver_name, netdev->name);
-               adapter->ahw.xg_linkup = 1;
-               netif_carrier_on(netdev);
-               netif_wake_queue(netdev);
-       }
-}
index 4d32f3cc2fbd8a5e17e827d6f64590f3d369210c..2d0963f4d194343aee6bbb7be34b061aedb7946a 100644 (file)
@@ -904,7 +904,7 @@ request_msi:
                goto err_out_disable_msi;
 
        init_timer(&adapter->watchdog_timer);
-       adapter->ahw.xg_linkup = 0;
+       adapter->ahw.linkup = 0;
        adapter->watchdog_timer.function = &netxen_watchdog;
        adapter->watchdog_timer.data = (unsigned long)adapter;
        INIT_WORK(&adapter->watchdog_task, netxen_watchdog_task);
@@ -1335,6 +1335,80 @@ static int netxen_nic_xmit_frame(struct sk_buff *skb, struct net_device *netdev)
        return NETDEV_TX_OK;
 }
 
+static int netxen_nic_check_temp(struct netxen_adapter *adapter)
+{
+       struct net_device *netdev = adapter->netdev;
+       uint32_t temp, temp_state, temp_val;
+       int rv = 0;
+
+       temp = adapter->pci_read_normalize(adapter, CRB_TEMP_STATE);
+
+       temp_state = nx_get_temp_state(temp);
+       temp_val = nx_get_temp_val(temp);
+
+       if (temp_state == NX_TEMP_PANIC) {
+               printk(KERN_ALERT
+                      "%s: Device temperature %d degrees C exceeds"
+                      " maximum allowed. Hardware has been shut down.\n",
+                      netxen_nic_driver_name, temp_val);
+
+               netif_carrier_off(netdev);
+               netif_stop_queue(netdev);
+               rv = 1;
+       } else if (temp_state == NX_TEMP_WARN) {
+               if (adapter->temp == NX_TEMP_NORMAL) {
+                       printk(KERN_ALERT
+                              "%s: Device temperature %d degrees C "
+                              "exceeds operating range."
+                              " Immediate action needed.\n",
+                              netxen_nic_driver_name, temp_val);
+               }
+       } else {
+               if (adapter->temp == NX_TEMP_WARN) {
+                       printk(KERN_INFO
+                              "%s: Device temperature is now %d degrees C"
+                              " in normal range.\n", netxen_nic_driver_name,
+                              temp_val);
+               }
+       }
+       adapter->temp = temp_state;
+       return rv;
+}
+
+static void netxen_nic_handle_phy_intr(struct netxen_adapter *adapter)
+{
+       struct net_device *netdev = adapter->netdev;
+       u32 val, port, linkup;
+
+       port = adapter->physical_port;
+
+       val = adapter->pci_read_normalize(adapter, CRB_XG_STATE);
+       if (adapter->ahw.board_type == NETXEN_NIC_GBE)
+               linkup = (val >> port) & 1;
+       else {
+               val = (val >> port*8) & 0xff;
+               linkup = (val == XG_LINK_UP);
+       }
+
+       if (adapter->ahw.linkup && !linkup) {
+               printk(KERN_INFO "%s: %s NIC Link is down\n",
+                      netxen_nic_driver_name, netdev->name);
+               adapter->ahw.linkup = 0;
+               if (netif_running(netdev)) {
+                       netif_carrier_off(netdev);
+                       netif_stop_queue(netdev);
+               }
+       } else if (!adapter->ahw.linkup && linkup) {
+               printk(KERN_INFO "%s: %s NIC Link is up\n",
+                      netxen_nic_driver_name, netdev->name);
+               adapter->ahw.linkup = 1;
+               if (netif_running(netdev)) {
+                       netif_carrier_on(netdev);
+                       netif_wake_queue(netdev);
+               }
+       }
+}
+
 static void netxen_watchdog(unsigned long v)
 {
        struct netxen_adapter *adapter = (struct netxen_adapter *)v;
@@ -1342,6 +1416,19 @@ static void netxen_watchdog(unsigned long v)
        SCHEDULE_WORK(&adapter->watchdog_task);
 }
 
+void netxen_watchdog_task(struct work_struct *work)
+{
+       struct netxen_adapter *adapter =
+               container_of(work, struct netxen_adapter, watchdog_task);
+
+       if ((adapter->portnum  == 0) && netxen_nic_check_temp(adapter))
+               return;
+
+       netxen_nic_handle_phy_intr(adapter);
+
+       mod_timer(&adapter->watchdog_timer, jiffies + 2 * HZ);
+}
+
 static void netxen_tx_timeout(struct net_device *netdev)
 {
        struct netxen_adapter *adapter = (struct netxen_adapter *)
@@ -1367,6 +1454,38 @@ static void netxen_tx_timeout_task(struct work_struct *work)
        netif_wake_queue(adapter->netdev);
 }
 
+/*
+ * netxen_nic_get_stats - Get System Network Statistics
+ * @netdev: network interface device structure
+ */
+struct net_device_stats *netxen_nic_get_stats(struct net_device *netdev)
+{
+       struct netxen_adapter *adapter = netdev_priv(netdev);
+       struct net_device_stats *stats = &adapter->net_stats;
+
+       memset(stats, 0, sizeof(*stats));
+
+       /* total packets received   */
+       stats->rx_packets = adapter->stats.no_rcv;
+       /* total packets transmitted    */
+       stats->tx_packets = adapter->stats.xmitedframes +
+               adapter->stats.xmitfinished;
+       /* total bytes received     */
+       stats->rx_bytes = adapter->stats.rxbytes;
+       /* total bytes transmitted  */
+       stats->tx_bytes = adapter->stats.txbytes;
+       /* bad packets received     */
+       stats->rx_errors = adapter->stats.rcvdbadskb;
+       /* packet transmit problems */
+       stats->tx_errors = adapter->stats.nocmddescriptor;
+       /* no space in linux buffers    */
+       stats->rx_dropped = adapter->stats.rxdropped;
+       /* no space available in linux  */
+       stats->tx_dropped = adapter->stats.txdropped;
+
+       return stats;
+}
+
 static inline void
 netxen_handle_int(struct netxen_adapter *adapter)
 {