2008-01-13 18:54:50

by Stefano Brivio

[permalink] [raw]
Subject: [PATCH] rt2x00: remove unused variable

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 <[email protected]>
---
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


2008-01-13 22:13:03

by Michael Büsch

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

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 <[email protected]>
> 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 <[email protected]>
> Signed-off-by: John W. Linville <[email protected]>

I cannot find this commit in wireless-2.6

--
Greetings Michael.

2008-01-13 19:15:04

by Ivo Van Doorn

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

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

2008-01-13 19:33:01

by Michael Büsch

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

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.

2008-01-15 18:24:16

by John W. Linville

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

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
[email protected]

2008-01-13 21:48:56

by Ivo Van Doorn

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

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 <[email protected]>
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 <[email protected]>
Signed-off-by: John W. Linville <[email protected]>

2008-01-13 19:23:05

by Stefano Brivio

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

On Sun, 13 Jan 2008 20:14:53 +0100
Ivo van Doorn <[email protected]> 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

2008-01-13 22:30:48

by Ivo Van Doorn

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

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 <[email protected]>
> > 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 <[email protected]>
> > Signed-off-by: John W. Linville <[email protected]>
>
> 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