[PATCH] rt2x00: remove unused variable

[PATCH] rt2x00: remove unused variable

[Stefano Brivio]

Remove an unused variable. Note that the pipe obtained there isn't used as
well.

Signed-off-by: Stefano Brivio <stefano.brivio@polimi.it>
---
Index: wireless-2.6/drivers/net/wireless/rt2x00/rt2x00usb.c
===================================================================
--- wireless-2.6.orig/drivers/net/wireless/rt2x00/rt2x00usb.c
+++ wireless-2.6/drivers/net/wireless/rt2x00/rt2x00usb.c
@@ -176,7 +176,6 @@ int rt2x00usb_write_tx_data(struct rt2x0
 	struct usb_device *usb_dev =
 	    interface_to_usbdev(rt2x00dev_usb(rt2x00dev));
 	struct data_entry *entry = rt2x00_get_data_entry(ring);
-	int pipe = usb_sndbulkpipe(usb_dev, 1);
 	struct skb_desc *desc;
 	u32 length;
 
 
--
Ciao
Stefano

[PATCH] rt2x00: remove unused variable

[Stefano Brivio]

Remove an unused variable that caused a warning while building. Please note
that the pipe obtained there isn't used anywhere else.

Signed-off-by: Stefano Brivio <stefano.brivio@polimi.it>
---
Index: wireless-2.6/drivers/net/wireless/rt2x00/rt2x00usb.c
===================================================================
--- wireless-2.6.orig/drivers/net/wireless/rt2x00/rt2x00usb.c
+++ wireless-2.6/drivers/net/wireless/rt2x00/rt2x00usb.c
@@ -176,7 +176,6 @@ int rt2x00usb_write_tx_data(struct rt2x0
 	struct usb_device *usb_dev =
 	    interface_to_usbdev(rt2x00dev_usb(rt2x00dev));
 	struct data_entry *entry = rt2x00_get_data_entry(ring);
-	int pipe = usb_sndbulkpipe(usb_dev, 1);
 	struct skb_desc *desc;
 	u32 length;
 


--
Ciao
Stefano

Re: [Rt2400-devel] [PATCH] rt2x00: remove unused variable

[Ivo van Doorn]

Hi,

> Remove an unused variable that caused a warning while building. Please note
> that the pipe obtained there isn't used anywhere else.

This patch is in wireless-2.6 already...

Ivo

Re: [Rt2400-devel] [PATCH] rt2x00: remove unused variable

[Stefano Brivio]

On Sun, 13 Jan 2008 20:14:53 +0100
Ivo van Doorn <ivdoorn@gmail.com> wrote:

> Hi,
> 
> > Remove an unused variable that caused a warning while building. Please note
> > that the pipe obtained there isn't used anywhere else.
> 
> This patch is in wireless-2.6 already...

Actually, I thought I already sent this patch, but couldn't find it
anywhere, neither in my outbox nor in git-log. Could you tell me what's the
related SHA commit? Thanks.


-- 
Ciao
Stefano

Re: [Rt2400-devel] [PATCH] rt2x00: remove unused variable

[Michael Buesch]

On Sunday 13 January 2008 20:14:53 Ivo van Doorn wrote:
> Hi,
> 
> > Remove an unused variable that caused a warning while building. Please note
> > that the pipe obtained there isn't used anywhere else.
> 
> This patch is in wireless-2.6 already...

No it is not in wireless-2.6.

-- 
Greetings Michael.

Re: [Rt2400-devel] [PATCH] rt2x00: remove unused variable

[Ivo van Doorn]

Hi.

> > > Remove an unused variable that caused a warning while building. Please note
> > > that the pipe obtained there isn't used anywhere else.
> > 
> > This patch is in wireless-2.6 already...
> 
> Actually, I thought I already sent this patch, but couldn't find it
> anywhere, neither in my outbox nor in git-log. Could you tell me what's the
> related SHA commit? Thanks.

This is in my rt2x00.git log. It is signed off by John and not by me,
so that means it didn't go through rt2x00.git to get into the wireless tree.
I have merged the patch into rt2x00.git during my last clone from wireless...

commit 01b0258008a73648245dcbc628ff948878c428e0
Author: Stefano Brivio <stefano.brivio@polimi.it>
Date:   Wed Dec 19 01:43:33 2007 +0100

    rt2x00: remove unused variable

    Remove an unused variable. Note that the pipe obtained there isn't used as
    well.

    Signed-off-by: Stefano Brivio <stefano.brivio@polimi.it>
    Signed-off-by: John W. Linville <linville@tuxdriver.com>

Re: [Rt2400-devel] [PATCH] rt2x00: remove unused variable

[Michael Buesch]

On Sunday 13 January 2008 22:48:47 Ivo van Doorn wrote:
> Hi.
> 
> > > > Remove an unused variable that caused a warning while building. Please note
> > > > that the pipe obtained there isn't used anywhere else.
> > > 
> > > This patch is in wireless-2.6 already...
> > 
> > Actually, I thought I already sent this patch, but couldn't find it
> > anywhere, neither in my outbox nor in git-log. Could you tell me what's the
> > related SHA commit? Thanks.
> 
> This is in my rt2x00.git log. It is signed off by John and not by me,
> so that means it didn't go through rt2x00.git to get into the wireless tree.
> I have merged the patch into rt2x00.git during my last clone from wireless...
> 
> commit 01b0258008a73648245dcbc628ff948878c428e0
> Author: Stefano Brivio <stefano.brivio@polimi.it>
> Date:   Wed Dec 19 01:43:33 2007 +0100
> 
>     rt2x00: remove unused variable
> 
>     Remove an unused variable. Note that the pipe obtained there isn't used as
>     well.
> 
>     Signed-off-by: Stefano Brivio <stefano.brivio@polimi.it>
>     Signed-off-by: John W. Linville <linville@tuxdriver.com>

I cannot find this commit in wireless-2.6

-- 
Greetings Michael.

Re: [Rt2400-devel] [PATCH] rt2x00: remove unused variable

[Ivo van Doorn]

On Sunday 13 January 2008, Michael Buesch wrote:
> On Sunday 13 January 2008 22:48:47 Ivo van Doorn wrote:
> > Hi.
> > 
> > > > > Remove an unused variable that caused a warning while building. Please note
> > > > > that the pipe obtained there isn't used anywhere else.
> > > > 
> > > > This patch is in wireless-2.6 already...
> > > 
> > > Actually, I thought I already sent this patch, but couldn't find it
> > > anywhere, neither in my outbox nor in git-log. Could you tell me what's the
> > > related SHA commit? Thanks.
> > 
> > This is in my rt2x00.git log. It is signed off by John and not by me,
> > so that means it didn't go through rt2x00.git to get into the wireless tree.
> > I have merged the patch into rt2x00.git during my last clone from wireless...
> > 
> > commit 01b0258008a73648245dcbc628ff948878c428e0
> > Author: Stefano Brivio <stefano.brivio@polimi.it>
> > Date:   Wed Dec 19 01:43:33 2007 +0100
> > 
> >     rt2x00: remove unused variable
> > 
> >     Remove an unused variable. Note that the pipe obtained there isn't used as
> >     well.
> > 
> >     Signed-off-by: Stefano Brivio <stefano.brivio@polimi.it>
> >     Signed-off-by: John W. Linville <linville@tuxdriver.com>
> 
> I cannot find this commit in wireless-2.6

Strange, git is pulling tricks of some kind. I have the patch in 3 branches of rt2x00.git,
master, wireless and upstream. This means it was pulled in from the everything branch
on wireless-2.6. So apparently during the rebasing in wireless the patch has disappeared
or something. :S

John, can you apply the patch to wireless? Otherwise I'll apply it to rt2x00.git later
this week when I rebase everything against the latest wireless git repository (which
would remove the old patch from my tree, so I can apply it without conflicts).

Ivo

Re: [Rt2400-devel] [PATCH] rt2x00: remove unused variable

[John W. Linville]

On Sun, Jan 13, 2008 at 11:30:39PM +0100, Ivo van Doorn wrote:

> Strange, git is pulling tricks of some kind. I have the patch in 3 branches of rt2x00.git,
> master, wireless and upstream. This means it was pulled in from the everything branch
> on wireless-2.6. So apparently during the rebasing in wireless the patch has disappeared
> or something. :S

Yes, that is likely what happened.  I'll take care of it.

John
-- 
John W. Linville
linville@tuxdriver.com