lk/linux-kernel-test
1
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

[PATCH] libertas: split wlan_add_card()

Browse Source 
Split wlan_add_card() into a part that just setups kernel parameters and
into the function libertas_activate_card(), which will implizitly use
hardware functions by the started thread.

This allows us later to do something like this:

priv = libertas_add_card();
priv->hw_command_to_host = if_usb_command_to_host;
priv->hw_xxxx = if_usb_xxxx;
priv->hw_yyyy = if_usb_yyyy;
wlan_activate_card()

and of course the CF driver can set it's own functions.

Signed-off-by: Holger Schurig <hs4233@mail.mn-solutions.de>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
This commit is contained in:
Holger Schurig
2007-05-25 12:04:31 -04:00
committed by John W. Linville
parent 3874d0fefd
commit 32a74b7c8f
3 changed files with 24 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files

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

@ -812,6 +812,21 @@ wlan_private *wlan_add_card(void *card)
spin_lock_init(&priv->adapter->driver_lock);
init_waitqueue_head(&priv->adapter->cmd_pending);
priv->adapter->nr_cmd_pending = 0;
goto done;
err_kzalloc:
free_netdev(dev);
done:
lbs_deb_leave_args(LBS_DEB_NET, "priv %p", priv);
return priv;
}
int libertas_activate_card(wlan_private *priv)
{
struct net_device *dev = priv->wlan_dev.netdev;
int ret = -1;
lbs_deb_enter(LBS_DEB_MAIN);
lbs_deb_thread("Starting kthread...\n");
priv->mainthread.priv = priv;
@ -847,8 +862,8 @@ wlan_private *wlan_add_card(void *card)
libertas_debugfs_init_one(priv, dev);
lbs_deb_leave_args(LBS_DEB_NET, "priv %p", priv);
return priv;
ret = 0;
goto done;
err_init_fw:
libertas_sbi_unregister_dev(priv);
@ -858,11 +873,10 @@ err_registerdev:
wake_up_interruptible(&priv->mainthread.waitq);
wlan_terminate_thread(&priv->mainthread);
kfree(priv->adapter);
err_kzalloc:
free_netdev(dev);
lbs_deb_leave_args(LBS_DEB_NET, "priv NULL");
return NULL;
done:
lbs_deb_leave_args(LBS_DEB_NET, "ret %d", ret);
return ret;
}
/**