diff options
author | Michael Buesch <mb@bu3sch.de> | 2007-09-20 14:13:40 -0400 |
---|---|---|
committer | David S. Miller <davem@sunset.davemloft.net> | 2007-10-10 19:52:07 -0400 |
commit | 1a09404a2338163f181d170c7abdc2242b6c6f03 (patch) | |
tree | 090a31041c1e294a2586f12679a17a5329129272 /drivers/net/wireless/b43/debugfs.c | |
parent | fb8e4444cc8c7719d9947e21a93e2e84bb1b36eb (diff) |
[B43]: Fix sparse warnings.
The remaining warning in phy.c will be fixed later.
Signed-off-by: Michael Buesch <mb@bu3sch.de>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'drivers/net/wireless/b43/debugfs.c')
-rw-r--r-- | drivers/net/wireless/b43/debugfs.c | 31 |
1 files changed, 21 insertions, 10 deletions
diff --git a/drivers/net/wireless/b43/debugfs.c b/drivers/net/wireless/b43/debugfs.c index f82e3ef9bd73..3aafde9f08a8 100644 --- a/drivers/net/wireless/b43/debugfs.c +++ b/drivers/net/wireless/b43/debugfs.c | |||
@@ -39,7 +39,7 @@ | |||
39 | 39 | ||
40 | 40 | ||
41 | /* The root directory. */ | 41 | /* The root directory. */ |
42 | struct dentry *rootdir; | 42 | static struct dentry *rootdir; |
43 | 43 | ||
44 | struct b43_debugfs_fops { | 44 | struct b43_debugfs_fops { |
45 | ssize_t (*read)(struct b43_wldev *dev, char *buf, size_t bufsize); | 45 | ssize_t (*read)(struct b43_wldev *dev, char *buf, size_t bufsize); |
@@ -76,7 +76,8 @@ struct b43_dfs_file * fops_to_dfs_file(struct b43_wldev *dev, | |||
76 | 76 | ||
77 | 77 | ||
78 | /* wl->irq_lock is locked */ | 78 | /* wl->irq_lock is locked */ |
79 | ssize_t tsf_read_file(struct b43_wldev *dev, char *buf, size_t bufsize) | 79 | static ssize_t tsf_read_file(struct b43_wldev *dev, |
80 | char *buf, size_t bufsize) | ||
80 | { | 81 | { |
81 | ssize_t count = 0; | 82 | ssize_t count = 0; |
82 | u64 tsf; | 83 | u64 tsf; |
@@ -90,7 +91,8 @@ ssize_t tsf_read_file(struct b43_wldev *dev, char *buf, size_t bufsize) | |||
90 | } | 91 | } |
91 | 92 | ||
92 | /* wl->irq_lock is locked */ | 93 | /* wl->irq_lock is locked */ |
93 | int tsf_write_file(struct b43_wldev *dev, const char *buf, size_t count) | 94 | static int tsf_write_file(struct b43_wldev *dev, |
95 | const char *buf, size_t count) | ||
94 | { | 96 | { |
95 | u64 tsf; | 97 | u64 tsf; |
96 | 98 | ||
@@ -102,7 +104,8 @@ int tsf_write_file(struct b43_wldev *dev, const char *buf, size_t count) | |||
102 | } | 104 | } |
103 | 105 | ||
104 | /* wl->irq_lock is locked */ | 106 | /* wl->irq_lock is locked */ |
105 | ssize_t ucode_regs_read_file(struct b43_wldev *dev, char *buf, size_t bufsize) | 107 | static ssize_t ucode_regs_read_file(struct b43_wldev *dev, |
108 | char *buf, size_t bufsize) | ||
106 | { | 109 | { |
107 | ssize_t count = 0; | 110 | ssize_t count = 0; |
108 | int i; | 111 | int i; |
@@ -116,7 +119,8 @@ ssize_t ucode_regs_read_file(struct b43_wldev *dev, char *buf, size_t bufsize) | |||
116 | } | 119 | } |
117 | 120 | ||
118 | /* wl->irq_lock is locked */ | 121 | /* wl->irq_lock is locked */ |
119 | ssize_t shm_read_file(struct b43_wldev *dev, char *buf, size_t bufsize) | 122 | static ssize_t shm_read_file(struct b43_wldev *dev, |
123 | char *buf, size_t bufsize) | ||
120 | { | 124 | { |
121 | ssize_t count = 0; | 125 | ssize_t count = 0; |
122 | int i; | 126 | int i; |
@@ -135,7 +139,8 @@ ssize_t shm_read_file(struct b43_wldev *dev, char *buf, size_t bufsize) | |||
135 | return count; | 139 | return count; |
136 | } | 140 | } |
137 | 141 | ||
138 | ssize_t txstat_read_file(struct b43_wldev *dev, char *buf, size_t bufsize) | 142 | static ssize_t txstat_read_file(struct b43_wldev *dev, |
143 | char *buf, size_t bufsize) | ||
139 | { | 144 | { |
140 | struct b43_txstatus_log *log = &dev->dfsentry->txstatlog; | 145 | struct b43_txstatus_log *log = &dev->dfsentry->txstatlog; |
141 | ssize_t count = 0; | 146 | ssize_t count = 0; |
@@ -182,7 +187,8 @@ out_unlock: | |||
182 | return count; | 187 | return count; |
183 | } | 188 | } |
184 | 189 | ||
185 | ssize_t txpower_g_read_file(struct b43_wldev *dev, char *buf, size_t bufsize) | 190 | static ssize_t txpower_g_read_file(struct b43_wldev *dev, |
191 | char *buf, size_t bufsize) | ||
186 | { | 192 | { |
187 | ssize_t count = 0; | 193 | ssize_t count = 0; |
188 | 194 | ||
@@ -214,7 +220,8 @@ out: | |||
214 | return count; | 220 | return count; |
215 | } | 221 | } |
216 | 222 | ||
217 | int txpower_g_write_file(struct b43_wldev *dev, const char *buf, size_t count) | 223 | static int txpower_g_write_file(struct b43_wldev *dev, |
224 | const char *buf, size_t count) | ||
218 | { | 225 | { |
219 | unsigned long flags; | 226 | unsigned long flags; |
220 | unsigned long phy_flags; | 227 | unsigned long phy_flags; |
@@ -262,7 +269,8 @@ out_unlock: | |||
262 | } | 269 | } |
263 | 270 | ||
264 | /* wl->irq_lock is locked */ | 271 | /* wl->irq_lock is locked */ |
265 | int restart_write_file(struct b43_wldev *dev, const char *buf, size_t count) | 272 | static int restart_write_file(struct b43_wldev *dev, |
273 | const char *buf, size_t count) | ||
266 | { | 274 | { |
267 | int err = 0; | 275 | int err = 0; |
268 | 276 | ||
@@ -294,7 +302,8 @@ static ssize_t append_lo_table(ssize_t count, char *buf, const size_t bufsize, | |||
294 | return count; | 302 | return count; |
295 | } | 303 | } |
296 | 304 | ||
297 | ssize_t loctls_read_file(struct b43_wldev *dev, char *buf, size_t bufsize) | 305 | static ssize_t loctls_read_file(struct b43_wldev *dev, |
306 | char *buf, size_t bufsize) | ||
298 | { | 307 | { |
299 | ssize_t count = 0; | 308 | ssize_t count = 0; |
300 | struct b43_txpower_lo_control *lo; | 309 | struct b43_txpower_lo_control *lo; |
@@ -383,6 +392,8 @@ static ssize_t b43_debugfs_read(struct file *file, char __user *userbuf, | |||
383 | err = -ENOMEM; | 392 | err = -ENOMEM; |
384 | goto out_unlock; | 393 | goto out_unlock; |
385 | } | 394 | } |
395 | /* Sparse warns about the following memset, because it has a big | ||
396 | * size value. That warning is bogus, so I will ignore it. --mb */ | ||
386 | memset(buf, 0, bufsize); | 397 | memset(buf, 0, bufsize); |
387 | if (dfops->take_irqlock) { | 398 | if (dfops->take_irqlock) { |
388 | spin_lock_irq(&dev->wl->irq_lock); | 399 | spin_lock_irq(&dev->wl->irq_lock); |