2011-05-11 16:00:37

by maximilian attems

[permalink] [raw]
Subject: [[email protected]: Bug#596626: Please support Linksys WUSB600N v2 WiFi USB stick]

please have a look at http://bugs.debian.org/596626

thanks.

--
maks

----- Forwarded message from Geoff Simmons <[email protected]> -----

Date: Wed, 11 May 2011 20:51:49 +1000
From: Geoff Simmons <[email protected]>
To: maximilian attems <[email protected]>, [email protected]
Subject: Bug#596626: Please support Linksys WUSB600N v2 WiFi USB stick
X-Original-To: [email protected]
X-Original-To: [email protected]

On Wed, May 11, 2011 at 10:12:55AM +0000, maximilian attems wrote:
> On Tue, May 10, 2011 at 11:24:34PM +1000, Geoff Simmons wrote:
> > Recognition of this device (USB ID 1737:0079) by the rt2800usb driver
> > currently requires RT2800USB_UNKNOWN to be enabled.
>
> strange it is listed in the dev table
> { USB_DEVICE(0x1737, 0x0079), USB_DEVICE_DATA(&rt2800usb_ops) },
> and even since long it seems:
> git name-rev 5e312589
> 5e312589 tags/v2.6.33-rc3~44^2^2~15

It is listed, but only incorporated in rt2800usb_device_table[] if
RT2800USB_UNKNOWN is set:

#ifdef CONFIG_RT2800USB_UNKNOWN
/*
* Unclear what kind of devices these are (they aren't supported by the
* vendor linux driver).
*/
[...]
/* Linksys */
[...]
{ USB_DEVICE(0x1737, 0x0079), USB_DEVICE_DATA(&rt2800usb_ops) },

As mentioned by Jan, the device contains a RT3572 chipset. Its entry
could moved into the section for known RT35XX devices (i.e. "#ifdef
CONFIG_RT2800USB_RT35XX") as an alternative.

Geoff



--
To UNSUBSCRIBE, email to [email protected]
with a subject of "unsubscribe". Trouble? Contact [email protected]
Archive: http://lists.debian.org/[email protected]


----- End forwarded message -----


2011-05-13 07:17:06

by Ivo Van Doorn

[permalink] [raw]
Subject: Re: Bug#596626: [[email protected]: Bug#596626: Please support Linksys WUSB600N v2 WiFi USB stick]

Hi,

>> > As mentioned by Jan, the device contains a RT3572
>> > chipset. ?Its entry
>> > could moved into the section for known RT35XX devices (i.e.
>> > "#ifdef
>> > CONFIG_RT2800USB_RT35XX") as an alternative.
>> >
>> > Geoff
>>
>> This has already been done: http://git.kernel.org/?p=linux/kernel/git/ivd/rt2x00.git;a=commitdiff;h=ce2919c9fffe2aa52f9c3e327176d03764dbf9b5
>
> That's all very well, but that isn't going to get into a stable release
> for another 3 months! ?Device ID updates that don't require new
> supporting code should be sent upstream straight away (and cc'd to
> [email protected]).

The patch has been send upstream, it has been in wireless-next-2.6.git
since April 19...
http://git.kernel.org/?p=linux/kernel/git/linville/wireless-next-2.6.git;a=commit;h=ce2919c9fffe2aa52f9c3e327176d03764dbf9b5
The patch depends on the RT53xx support patch which was also intented
for 2.6.40.

> I've cherry-picked this and the other two updates in rt2800usb that
> aren't in Linus's tree, but that doesn't help the users of other
> distributions that would benefit from them.

True, but the normal flow for new features and hardware support is that they
should be properly merged during the normal merge window. The addition
of support of RT53xx is not something that should go to [email protected]...

Patches from rt2x00.git are quite quickly send from rt2x00.git to upstream,
and patches which are not send upstream directly have a reason for not being
send at that time (Usually it means that it requires some extra testing).

Ivo

2011-05-11 16:25:10

by Walter Goldens

[permalink] [raw]
Subject: Re: [[email protected]: Bug#596626: Please support Linksys WUSB600N v2 WiFi USB stick]

> As mentioned by Jan, the device contains a RT3572
> chipset.? Its entry
> could moved into the section for known RT35XX devices (i.e.
> "#ifdef
> CONFIG_RT2800USB_RT35XX") as an alternative.
>
> Geoff

This has already been done: http://git.kernel.org/?p=linux/kernel/git/ivd/rt2x00.git;a=commitdiff;h=ce2919c9fffe2aa52f9c3e327176d03764dbf9b5

2011-05-13 13:01:03

by Ben Hutchings

[permalink] [raw]
Subject: Re: Bug#596626: [[email protected]: Bug#596626: Please support Linksys WUSB600N v2 WiFi USB stick]

On Fri, 2011-05-13 at 09:17 +0200, Ivo Van Doorn wrote:
> Hi,
>
> >> > As mentioned by Jan, the device contains a RT3572
> >> > chipset. Its entry
> >> > could moved into the section for known RT35XX devices (i.e.
> >> > "#ifdef
> >> > CONFIG_RT2800USB_RT35XX") as an alternative.
> >> >
> >> > Geoff
> >>
> >> This has already been done: http://git.kernel.org/?p=linux/kernel/git/ivd/rt2x00.git;a=commitdiff;h=ce2919c9fffe2aa52f9c3e327176d03764dbf9b5
> >
> > That's all very well, but that isn't going to get into a stable release
> > for another 3 months! Device ID updates that don't require new
> > supporting code should be sent upstream straight away (and cc'd to
> > [email protected]).
>
> The patch has been send upstream, it has been in wireless-next-2.6.git
> since April 19...
> http://git.kernel.org/?p=linux/kernel/git/linville/wireless-next-2.6.git;a=commit;h=ce2919c9fffe2aa52f9c3e327176d03764dbf9b5
> The patch depends on the RT53xx support patch which was also intented
> for 2.6.40.

What is it, RT35XX or RT53XX?!

> > I've cherry-picked this and the other two updates in rt2800usb that
> > aren't in Linus's tree, but that doesn't help the users of other
> > distributions that would benefit from them.
>
> True, but the normal flow for new features and hardware support is that they
> should be properly merged during the normal merge window. The addition
> of support of RT53xx is not something that should go to [email protected]...

I understand that.

Ben.

> Patches from rt2x00.git are quite quickly send from rt2x00.git to upstream,
> and patches which are not send upstream directly have a reason for not being
> send at that time (Usually it means that it requires some extra testing).
>
> Ivo
>

--
Ben Hutchings
Once a job is fouled up, anything done to improve it makes it worse.


Attachments:
signature.asc (828.00 B)
This is a digitally signed message part

2011-05-13 13:05:47

by Ivo Van Doorn

[permalink] [raw]
Subject: Re: Bug#596626: [[email protected]: Bug#596626: Please support Linksys WUSB600N v2 WiFi USB stick]

Hi,

>> >> > As mentioned by Jan, the device contains a RT3572
>> >> > chipset. ?Its entry
>> >> > could moved into the section for known RT35XX devices (i.e.
>> >> > "#ifdef
>> >> > CONFIG_RT2800USB_RT35XX") as an alternative.
>> >> >
>> >> > Geoff
>> >>
>> >> This has already been done: http://git.kernel.org/?p=linux/kernel/git/ivd/rt2x00.git;a=commitdiff;h=ce2919c9fffe2aa52f9c3e327176d03764dbf9b5
>> >
>> > That's all very well, but that isn't going to get into a stable release
>> > for another 3 months! ?Device ID updates that don't require new
>> > supporting code should be sent upstream straight away (and cc'd to
>> > [email protected]).
>>
>> The patch has been send upstream, it has been in wireless-next-2.6.git
>> since April 19...
>> http://git.kernel.org/?p=linux/kernel/git/linville/wireless-next-2.6.git;a=commit;h=ce2919c9fffe2aa52f9c3e327176d03764dbf9b5
>> The patch depends on the RT53xx support patch which was also intented
>> for 2.6.40.
>
> What is it, RT35XX or RT53XX?!

Oh sorry, RT35xx in this case, but basically the rest of the reply
still applies.
Both chipsets series have the same experimental state.

Ivo

2011-05-13 04:58:45

by Ben Hutchings

[permalink] [raw]
Subject: Re: Bug#596626: [[email protected]: Bug#596626: Please support Linksys WUSB600N v2 WiFi USB stick]

On Wed, 2011-05-11 at 09:25 -0700, Walter Goldens wrote:
> > As mentioned by Jan, the device contains a RT3572
> > chipset. Its entry
> > could moved into the section for known RT35XX devices (i.e.
> > "#ifdef
> > CONFIG_RT2800USB_RT35XX") as an alternative.
> >
> > Geoff
>
> This has already been done: http://git.kernel.org/?p=linux/kernel/git/ivd/rt2x00.git;a=commitdiff;h=ce2919c9fffe2aa52f9c3e327176d03764dbf9b5

That's all very well, but that isn't going to get into a stable release
for another 3 months! Device ID updates that don't require new
supporting code should be sent upstream straight away (and cc'd to
[email protected]).

I've cherry-picked this and the other two updates in rt2800usb that
aren't in Linus's tree, but that doesn't help the users of other
distributions that would benefit from them.

Ben.

--
Ben Hutchings
Once a job is fouled up, anything done to improve it makes it worse.


Attachments:
signature.asc (828.00 B)
This is a digitally signed message part

2011-05-13 09:12:40

by Gertjan van Wingerde

[permalink] [raw]
Subject: Re: Bug#596626: [[email protected]: Bug#596626: Please support Linksys WUSB600N v2 WiFi USB stick]

On Fri, May 13, 2011 at 6:58 AM, Ben Hutchings <[email protected]> wrote:
> On Wed, 2011-05-11 at 09:25 -0700, Walter Goldens wrote:
>> > As mentioned by Jan, the device contains a RT3572
>> > chipset. ?Its entry
>> > could moved into the section for known RT35XX devices (i.e.
>> > "#ifdef
>> > CONFIG_RT2800USB_RT35XX") as an alternative.
>> >
>> > Geoff
>>
>> This has already been done: http://git.kernel.org/?p=linux/kernel/git/ivd/rt2x00.git;a=commitdiff;h=ce2919c9fffe2aa52f9c3e327176d03764dbf9b5
>
> That's all very well, but that isn't going to get into a stable release
> for another 3 months! ?Device ID updates that don't require new
> supporting code should be sent upstream straight away (and cc'd to
> [email protected]).
>
> I've cherry-picked this and the other two updates in rt2800usb that
> aren't in Linus's tree, but that doesn't help the users of other
> distributions that would benefit from them.

Well, as this moving of the USB ID to a different (now named) set of
USB IDs in the code is not going to help the users,
as rt2x00 currently does not have support for RT35xx devices yet. Some
patches have been submitted for their support,
but these haven't been accepted (and thus applied) yet.

There's a reason why the Kconfig for these devices is stating that
support for these devices is non-functional!!!

That's why it didn't make sense to send this patch upstream
immediately and sending it to stable made even less sense.

---
Gertjan