diff options
author | Johannes Berg <johannes@sipsolutions.net> | 2009-05-12 02:36:43 -0400 |
---|---|---|
committer | John W. Linville <linville@tuxdriver.com> | 2009-05-13 15:44:40 -0400 |
commit | f7eef3563cb3f05e7f0db992716c514af6f4d089 (patch) | |
tree | 97a4215b4d9ebe86aa2cca111d599849a15efc54 | |
parent | faa8fdc85347cc76d87b43ea718785661c54f656 (diff) |
wext: remove seq_start/stop sparse annotations
Even though they are true, they cause sparse to complain
because it doesn't see the __acquires(dev_base_lock) on
dev_seq_start() because it is only added to the function
in net/core/dev.c, not the header file. To keep track of
the nesting correctly we should probably annotate those
functions publically, but for now let's just remove the
annotation I added to wext.
Signed-off-by: Johannes Berg <johannes@sipsolutions.net>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
-rw-r--r-- | net/wireless/wext.c | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/net/wireless/wext.c b/net/wireless/wext.c index dc5b47e84a3d..d3bbef70cc7c 100644 --- a/net/wireless/wext.c +++ b/net/wireless/wext.c | |||
@@ -650,14 +650,12 @@ static int wireless_seq_show(struct seq_file *seq, void *v) | |||
650 | } | 650 | } |
651 | 651 | ||
652 | static void *wireless_dev_seq_start(struct seq_file *seq, loff_t *pos) | 652 | static void *wireless_dev_seq_start(struct seq_file *seq, loff_t *pos) |
653 | __acquires(dev_base_lock) | ||
654 | { | 653 | { |
655 | rtnl_lock(); | 654 | rtnl_lock(); |
656 | return dev_seq_start(seq, pos); | 655 | return dev_seq_start(seq, pos); |
657 | } | 656 | } |
658 | 657 | ||
659 | static void wireless_dev_seq_stop(struct seq_file *seq, void *v) | 658 | static void wireless_dev_seq_stop(struct seq_file *seq, void *v) |
660 | __releases(dev_base_lock) | ||
661 | { | 659 | { |
662 | dev_seq_stop(seq, v); | 660 | dev_seq_stop(seq, v); |
663 | rtnl_unlock(); | 661 | rtnl_unlock(); |