* [PATCH] hostap: proc: Remove unused variable
@ 2014-09-16 23:16 Mark Brown
  2014-09-30 16:56 ` John W. Linville
  0 siblings, 1 reply; 3+ messages in thread
From: Mark Brown @ 2014-09-16 23:16 UTC (permalink / raw)
  To: Jouni Malinen, John W. Linville
  Cc: linux-wireless, netdev, linaro-kernel, Mark Brown
Since "hostap: proc: substitute loops by %*phN" (62c5afb8e388) the
variable i has been unused in prism2_bss_list_proc_show() so remove it.
Signed-off-by: Mark Brown <broonie@kernel.org>
---
 drivers/net/wireless/hostap/hostap_proc.c | 1 -
 1 file changed, 1 deletion(-)
diff --git a/drivers/net/wireless/hostap/hostap_proc.c b/drivers/net/wireless/hostap/hostap_proc.c
index 3a23a6d..dd84557 100644
--- a/drivers/net/wireless/hostap/hostap_proc.c
+++ b/drivers/net/wireless/hostap/hostap_proc.c
@@ -168,7 +168,6 @@ static int prism2_bss_list_proc_show(struct seq_file *m, void *v)
 	local_info_t *local = m->private;
 	struct list_head *ptr = v;
 	struct hostap_bss_info *bss;
-	int i;
 
 	if (ptr == &local->bss_list) {
 		seq_printf(m, "#BSSID\tlast_update\tcount\tcapab_info\tSSID(txt)\t"
-- 
2.1.0
^ permalink raw reply related	[flat|nested] 3+ messages in thread- * Re: [PATCH] hostap: proc: Remove unused variable
  2014-09-16 23:16 [PATCH] hostap: proc: Remove unused variable Mark Brown
@ 2014-09-30 16:56 ` John W. Linville
       [not found]   ` <20140930165626.GD11919-2XuSBdqkA4R54TAoqtyWWQ@public.gmane.org>
  0 siblings, 1 reply; 3+ messages in thread
From: John W. Linville @ 2014-09-30 16:56 UTC (permalink / raw)
  To: Mark Brown; +Cc: Jouni Malinen, linux-wireless, netdev, linaro-kernel
On Tue, Sep 16, 2014 at 04:16:55PM -0700, Mark Brown wrote:
> Since "hostap: proc: substitute loops by %*phN" (62c5afb8e388) the
> variable i has been unused in prism2_bss_list_proc_show() so remove it.
> 
> Signed-off-by: Mark Brown <broonie@kernel.org>
> ---
>  drivers/net/wireless/hostap/hostap_proc.c | 1 -
>  1 file changed, 1 deletion(-)
> 
> diff --git a/drivers/net/wireless/hostap/hostap_proc.c b/drivers/net/wireless/hostap/hostap_proc.c
> index 3a23a6d..dd84557 100644
> --- a/drivers/net/wireless/hostap/hostap_proc.c
> +++ b/drivers/net/wireless/hostap/hostap_proc.c
> @@ -168,7 +168,6 @@ static int prism2_bss_list_proc_show(struct seq_file *m, void *v)
>  	local_info_t *local = m->private;
>  	struct list_head *ptr = v;
>  	struct hostap_bss_info *bss;
> -	int i;
>  
>  	if (ptr == &local->bss_list) {
>  		seq_printf(m, "#BSSID\tlast_update\tcount\tcapab_info\tSSID(txt)\t"
> -- 
> 2.1.0
What tree are you using?  In wireless-next, I still see a loop in
that function using i as an index variable.
John
-- 
John W. Linville		Someday the world will need a hero, and you
linville@tuxdriver.com			might be all we have.  Be ready.
^ permalink raw reply	[flat|nested] 3+ messages in thread
end of thread, other threads:[~2014-09-30 22:56 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2014-09-16 23:16 [PATCH] hostap: proc: Remove unused variable Mark Brown
2014-09-30 16:56 ` John W. Linville
     [not found]   ` <20140930165626.GD11919-2XuSBdqkA4R54TAoqtyWWQ@public.gmane.org>
2014-09-30 22:56     ` Mark Brown
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).