AcoSail/kernel_optimize_test
8
0
Fork 0
forked from luck/tmp_suning_uos_patched
Code Pull Requests Activity

libertas: reset devices upon disconnect rather than module unloading

Browse Source 
1) Do not reset libertas devices upon module unload.  We're unloading
the module, we're not killing off devices.
2) Instead, reset libertas devices inside if_usb_disconnect, as we're
killing off interfaces and so on.
3) Resetting via disconnect callback means we no longer need to keep
track of probed libertas devices; kill off that list (and its wonderful
lack of locking..), as well.

Drop a useless comment as well.

Signed-off-by: Andres Salomon <dilinger@debian.org>
Signed-off-by: Dan Williams <dcbw@redhat.com>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
This commit is contained in:
Andres Salomon 2007-11-20 17:43:55 -05:00 committed by David S. Miller
parent 4fb910fd3a
commit be13f1897f
2 changed files with 4 additions and 25 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

26
drivers/net/wireless/libertas/if_usb.c
View File

@ -5,7 +5,6 @@
#include <linux/moduleparam.h> #include <linux/moduleparam.h>
#include <linux/firmware.h> #include <linux/firmware.h>
#include <linux/netdevice.h> #include <linux/netdevice.h>
#include <linux/list.h>
#include <linux/usb.h> #include <linux/usb.h>
#define DRV_NAME "usb8xxx" #define DRV_NAME "usb8xxx"
@ -23,15 +22,6 @@ static const char usbdriver_name[] = "usb8xxx";
static char *lbs_fw_name = "usb8388.bin"; static char *lbs_fw_name = "usb8388.bin";
module_param_named(fw_name, lbs_fw_name, charp, 0644); module_param_named(fw_name, lbs_fw_name, charp, 0644);
/*
* We need to send a RESET command to all USB devices before
* we tear down the USB connection. Otherwise we would not
* be able to re-init device the device if the module gets
* loaded again. This is a list of all initialized USB devices,
* for the reset code see if_usb_reset_device()
*/
static LIST_HEAD(usb_devices);
static struct usb_device_id if_usb_table[] = { static struct usb_device_id if_usb_table[] = {
/* Enter the device signature inside */ /* Enter the device signature inside */
{ USB_DEVICE(0x1286, 0x2001) }, { USB_DEVICE(0x1286, 0x2001) },
@ -244,8 +234,6 @@ static int if_usb_probe(struct usb_interface *intf,
if (lbs_start_card(priv)) if (lbs_start_card(priv))
goto err_start_card; goto err_start_card;
list_add_tail(&cardp->list, &usb_devices);
usb_get_dev(udev); usb_get_dev(udev);
usb_set_intfdata(intf, cardp); usb_set_intfdata(intf, cardp);
@ -279,8 +267,6 @@ static void if_usb_disconnect(struct usb_interface *intf)
/* Update Surprise removed to TRUE */ /* Update Surprise removed to TRUE */
cardp->surprise_removed = 1; cardp->surprise_removed = 1;
list_del(&cardp->list);
if (priv) { if (priv) {
lbs_adapter *adapter = priv->adapter; lbs_adapter *adapter = priv->adapter;
@ -290,6 +276,10 @@ static void if_usb_disconnect(struct usb_interface *intf)
lbs_remove_card(priv); lbs_remove_card(priv);
} }
/* this is (apparently?) necessary for future usage of the device */
lbs_prepare_and_send_command(priv, CMD_802_11_RESET, CMD_ACT_HALT,
0, 0, NULL);
/* Unlink and free urb */ /* Unlink and free urb */
if_usb_free(cardp); if_usb_free(cardp);
@ -1034,16 +1024,8 @@ static int __init if_usb_init_module(void)
static void __exit if_usb_exit_module(void) static void __exit if_usb_exit_module(void)
{ {
struct usb_card_rec *cardp, *cardp_temp;
lbs_deb_enter(LBS_DEB_MAIN); lbs_deb_enter(LBS_DEB_MAIN);
list_for_each_entry_safe(cardp, cardp_temp, &usb_devices, list) {
lbs_prepare_and_send_command(cardp->priv, CMD_802_11_RESET,
CMD_ACT_HALT, 0, 0, NULL);
}
/* API unregisters the driver from USB subsystem */
usb_deregister(&if_usb_driver); usb_deregister(&if_usb_driver);
lbs_deb_leave(LBS_DEB_MAIN); lbs_deb_leave(LBS_DEB_MAIN);

3
drivers/net/wireless/libertas/if_usb.h
View File

@ -1,8 +1,6 @@
#ifndef _LBS_IF_USB_H #ifndef _LBS_IF_USB_H
#define _LBS_IF_USB_H #define _LBS_IF_USB_H
#include <linux/list.h>
/** /**
* This file contains definition for USB interface. * This file contains definition for USB interface.
*/ */
@ -44,7 +42,6 @@ struct read_cb_info {
/** USB card description structure*/ /** USB card description structure*/
struct usb_card_rec { struct usb_card_rec {
struct list_head list;
struct net_device *eth_dev; struct net_device *eth_dev;
struct usb_device *udev; struct usb_device *udev;
struct urb *rx_urb, *tx_urb; struct urb *rx_urb, *tx_urb;