diff options
author | Ulrich Kunitz <kune@deine-taler.de> | 2006-11-21 19:05:53 -0500 |
---|---|---|
committer | Jeff Garzik <jeff@garzik.org> | 2006-12-02 00:12:05 -0500 |
commit | 741fec53f268b691b869ffc70023319406e0cc97 (patch) | |
tree | 47ea2334fd44daaaa19b9a3d22d34466d0250069 /drivers/net/wireless/zd1211rw/zd_usb.c | |
parent | 84bc715c465f76584fc5127955fca0c61592e04b (diff) |
[PATCH] zd1211rw: cleanups
Bit-field constants in zd_chip.h are now defined using a shift expression.
The value 0x08 is now (1 << 3). The fix is intended to improve readability.
Remove misleading comment in zd_mac.c: The function already returns -EPERM
in managed mode (IW_MODE_INFRA).
Remove unused code in zd_mac.c: The unused code intended for debugging
rx_status values is no longer useful.
Added dump_stack() to ZD_ASSERT macro: Output of the stack helps to debug
assertions. Keep in mind that the ZD_ASSERT() macro only results in code,
if DEBUG is defined.
Improved comments for filter_rx()
zd_usb.c: Added driver name to module init and exit functions
Signed-off-by: Ulrich Kunitz <kune@deine-taler.de>
Signed-off-by: Daniel Drake <dsd@gentoo.org>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
Diffstat (limited to 'drivers/net/wireless/zd1211rw/zd_usb.c')
-rw-r--r-- | drivers/net/wireless/zd1211rw/zd_usb.c | 11 |
1 files changed, 6 insertions, 5 deletions
diff --git a/drivers/net/wireless/zd1211rw/zd_usb.c b/drivers/net/wireless/zd1211rw/zd_usb.c index 34524b73324f..c3449cfae7e8 100644 --- a/drivers/net/wireless/zd1211rw/zd_usb.c +++ b/drivers/net/wireless/zd1211rw/zd_usb.c | |||
@@ -1116,27 +1116,28 @@ static int __init usb_init(void) | |||
1116 | { | 1116 | { |
1117 | int r; | 1117 | int r; |
1118 | 1118 | ||
1119 | pr_debug("usb_init()\n"); | 1119 | pr_debug("%s usb_init()\n", driver.name); |
1120 | 1120 | ||
1121 | zd_workqueue = create_singlethread_workqueue(driver.name); | 1121 | zd_workqueue = create_singlethread_workqueue(driver.name); |
1122 | if (zd_workqueue == NULL) { | 1122 | if (zd_workqueue == NULL) { |
1123 | printk(KERN_ERR "%s: couldn't create workqueue\n", driver.name); | 1123 | printk(KERN_ERR "%s couldn't create workqueue\n", driver.name); |
1124 | return -ENOMEM; | 1124 | return -ENOMEM; |
1125 | } | 1125 | } |
1126 | 1126 | ||
1127 | r = usb_register(&driver); | 1127 | r = usb_register(&driver); |
1128 | if (r) { | 1128 | if (r) { |
1129 | printk(KERN_ERR "usb_register() failed. Error number %d\n", r); | 1129 | printk(KERN_ERR "%s usb_register() failed. Error number %d\n", |
1130 | driver.name, r); | ||
1130 | return r; | 1131 | return r; |
1131 | } | 1132 | } |
1132 | 1133 | ||
1133 | pr_debug("zd1211rw initialized\n"); | 1134 | pr_debug("%s initialized\n", driver.name); |
1134 | return 0; | 1135 | return 0; |
1135 | } | 1136 | } |
1136 | 1137 | ||
1137 | static void __exit usb_exit(void) | 1138 | static void __exit usb_exit(void) |
1138 | { | 1139 | { |
1139 | pr_debug("usb_exit()\n"); | 1140 | pr_debug("%s usb_exit()\n", driver.name); |
1140 | usb_deregister(&driver); | 1141 | usb_deregister(&driver); |
1141 | destroy_workqueue(zd_workqueue); | 1142 | destroy_workqueue(zd_workqueue); |
1142 | } | 1143 | } |