Patchwork [SRU,Oneiric/Precise,Quantal] rt2x00: Add support for BUFFALO WLI-UC-GNM2 to rt2800usb.

login
register
mail settings
Submitter Herton Ronaldo Krzesinski
Date Aug. 6, 2012, 2:23 p.m.
Message ID <1344263026-5348-1-git-send-email-herton.krzesinski@canonical.com>
Download mbox | patch
Permalink /patch/175372/
State New
Headers show

Comments

Herton Ronaldo Krzesinski - Aug. 6, 2012, 2:23 p.m.
From: Jeongdo Son <sohn9086@gmail.com>

This is a RT3070 based device.

Signed-off-by: Jeongdo Son <sohn9086@gmail.com>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
BugLink: http://bugs.launchpad.net/bugs/871904
Signed-off-by: Herton Ronaldo Krzesinski <herton.krzesinski@canonical.com>
---
 drivers/net/wireless/rt2x00/rt2800usb.c |    1 +
 1 file changed, 1 insertion(+)

Note: original commit was not Cc'd to stable, I'll try to submit this to
stable mailing list as well. Marked for Oneiric and later as was
originally reported and tested against 3.0
Herton Ronaldo Krzesinski - Aug. 6, 2012, 2:26 p.m.
On Mon, Aug 06, 2012 at 11:23:46AM -0300, Herton Ronaldo Krzesinski wrote:
> From: Jeongdo Son <sohn9086@gmail.com>
> 
> This is a RT3070 based device.
> 
> Signed-off-by: Jeongdo Son <sohn9086@gmail.com>
> Signed-off-by: John W. Linville <linville@tuxdriver.com>

(chery picked from commit a769f9577232afe2c754606a83aad85127e7052a upstream)

> BugLink: http://bugs.launchpad.net/bugs/871904
> Signed-off-by: Herton Ronaldo Krzesinski <herton.krzesinski@canonical.com>
> ---
>  drivers/net/wireless/rt2x00/rt2800usb.c |    1 +
>  1 file changed, 1 insertion(+)
> 
> Note: original commit was not Cc'd to stable, I'll try to submit this to
> stable mailing list as well. Marked for Oneiric and later as was
> originally reported and tested against 3.0
> 
> diff --git a/drivers/net/wireless/rt2x00/rt2800usb.c b/drivers/net/wireless/rt2x00/rt2800usb.c
> index 20a5040..6cf3365 100644
> --- a/drivers/net/wireless/rt2x00/rt2800usb.c
> +++ b/drivers/net/wireless/rt2x00/rt2800usb.c
> @@ -971,6 +971,7 @@ static struct usb_device_id rt2800usb_device_table[] = {
>  	{ USB_DEVICE(0x0411, 0x015d) },
>  	{ USB_DEVICE(0x0411, 0x016f) },
>  	{ USB_DEVICE(0x0411, 0x01a2) },
> +	{ USB_DEVICE(0x0411, 0x01ee) },
>  	/* Corega */
>  	{ USB_DEVICE(0x07aa, 0x002f) },
>  	{ USB_DEVICE(0x07aa, 0x003c) },
> -- 
> 1.7.9.5
> 
> 
> -- 
> kernel-team mailing list
> kernel-team@lists.ubuntu.com
> https://lists.ubuntu.com/mailman/listinfo/kernel-team
>
Seth Forshee - Aug. 6, 2012, 2:47 p.m.
On Mon, Aug 06, 2012 at 11:23:46AM -0300, Herton Ronaldo Krzesinski wrote:
> From: Jeongdo Son <sohn9086@gmail.com>
> 
> This is a RT3070 based device.
> 
> Signed-off-by: Jeongdo Son <sohn9086@gmail.com>
> Signed-off-by: John W. Linville <linville@tuxdriver.com>
> BugLink: http://bugs.launchpad.net/bugs/871904
> Signed-off-by: Herton Ronaldo Krzesinski <herton.krzesinski@canonical.com>
> ---
>  drivers/net/wireless/rt2x00/rt2800usb.c |    1 +
>  1 file changed, 1 insertion(+)
> 
> Note: original commit was not Cc'd to stable, I'll try to submit this to
> stable mailing list as well. Marked for Oneiric and later as was
> originally reported and tested against 3.0
> 
> diff --git a/drivers/net/wireless/rt2x00/rt2800usb.c b/drivers/net/wireless/rt2x00/rt2800usb.c
> index 20a5040..6cf3365 100644
> --- a/drivers/net/wireless/rt2x00/rt2800usb.c
> +++ b/drivers/net/wireless/rt2x00/rt2800usb.c
> @@ -971,6 +971,7 @@ static struct usb_device_id rt2800usb_device_table[] = {
>  	{ USB_DEVICE(0x0411, 0x015d) },
>  	{ USB_DEVICE(0x0411, 0x016f) },
>  	{ USB_DEVICE(0x0411, 0x01a2) },
> +	{ USB_DEVICE(0x0411, 0x01ee) },
>  	/* Corega */
>  	{ USB_DEVICE(0x07aa, 0x002f) },
>  	{ USB_DEVICE(0x07aa, 0x003c) },
> -- 
> 1.7.9.5
> 
> 
> -- 
> kernel-team mailing list
> kernel-team@lists.ubuntu.com
> https://lists.ubuntu.com/mailman/listinfo/kernel-team
Tim Gardner - Aug. 6, 2012, 2:51 p.m.
On 08/06/2012 08:23 AM, Herton Ronaldo Krzesinski wrote:
> From: Jeongdo Son <sohn9086@gmail.com>
> 
> This is a RT3070 based device.
> 
> Signed-off-by: Jeongdo Son <sohn9086@gmail.com>
> Signed-off-by: John W. Linville <linville@tuxdriver.com>
> BugLink: http://bugs.launchpad.net/bugs/871904
> Signed-off-by: Herton Ronaldo Krzesinski <herton.krzesinski@canonical.com>
> ---
>  drivers/net/wireless/rt2x00/rt2800usb.c |    1 +
>  1 file changed, 1 insertion(+)
> 
> Note: original commit was not Cc'd to stable, I'll try to submit this to
> stable mailing list as well. Marked for Oneiric and later as was
> originally reported and tested against 3.0
> 
> diff --git a/drivers/net/wireless/rt2x00/rt2800usb.c b/drivers/net/wireless/rt2x00/rt2800usb.c
> index 20a5040..6cf3365 100644
> --- a/drivers/net/wireless/rt2x00/rt2800usb.c
> +++ b/drivers/net/wireless/rt2x00/rt2800usb.c
> @@ -971,6 +971,7 @@ static struct usb_device_id rt2800usb_device_table[] = {
>  	{ USB_DEVICE(0x0411, 0x015d) },
>  	{ USB_DEVICE(0x0411, 0x016f) },
>  	{ USB_DEVICE(0x0411, 0x01a2) },
> +	{ USB_DEVICE(0x0411, 0x01ee) },
>  	/* Corega */
>  	{ USB_DEVICE(0x07aa, 0x002f) },
>  	{ USB_DEVICE(0x07aa, 0x003c) },
> 

This driver has undergone a lot of change. Do we really want to enable
it for releases prior to Precise? Its also not clear from the bug report
which kernel version was tested.

rtg
Herton Ronaldo Krzesinski - Aug. 6, 2012, 3:15 p.m.
On Mon, Aug 06, 2012 at 08:51:11AM -0600, Tim Gardner wrote:
> On 08/06/2012 08:23 AM, Herton Ronaldo Krzesinski wrote:
> > From: Jeongdo Son <sohn9086@gmail.com>
> > 
> > This is a RT3070 based device.
> > 
> > Signed-off-by: Jeongdo Son <sohn9086@gmail.com>
> > Signed-off-by: John W. Linville <linville@tuxdriver.com>
> > BugLink: http://bugs.launchpad.net/bugs/871904
> > Signed-off-by: Herton Ronaldo Krzesinski <herton.krzesinski@canonical.com>
> > ---
> >  drivers/net/wireless/rt2x00/rt2800usb.c |    1 +
> >  1 file changed, 1 insertion(+)
> > 
> > Note: original commit was not Cc'd to stable, I'll try to submit this to
> > stable mailing list as well. Marked for Oneiric and later as was
> > originally reported and tested against 3.0
> > 
> > diff --git a/drivers/net/wireless/rt2x00/rt2800usb.c b/drivers/net/wireless/rt2x00/rt2800usb.c
> > index 20a5040..6cf3365 100644
> > --- a/drivers/net/wireless/rt2x00/rt2800usb.c
> > +++ b/drivers/net/wireless/rt2x00/rt2800usb.c
> > @@ -971,6 +971,7 @@ static struct usb_device_id rt2800usb_device_table[] = {
> >  	{ USB_DEVICE(0x0411, 0x015d) },
> >  	{ USB_DEVICE(0x0411, 0x016f) },
> >  	{ USB_DEVICE(0x0411, 0x01a2) },
> > +	{ USB_DEVICE(0x0411, 0x01ee) },
> >  	/* Corega */
> >  	{ USB_DEVICE(0x07aa, 0x002f) },
> >  	{ USB_DEVICE(0x07aa, 0x003c) },
> > 
> 
> This driver has undergone a lot of change. Do we really want to enable
> it for releases prior to Precise? Its also not clear from the bug report
> which kernel version was tested.

Reporter on comment #5 said it worked using new_id:
https://bugs.launchpad.net/ubuntu/+source/linux/+bug/871904/comments/5

And he reported against 3.0 kernel, so I assumed it works since Oneiric.
He doesn't confirm the kernel version on the comment, but it's unlikely
it changed the kernel from 3.0 to a newer release (look at the dates
since it was reported until comment 5).

> 
> rtg
> -- 
> Tim Gardner tim.gardner@canonical.com
>
Tim Gardner - Aug. 6, 2012, 3:44 p.m.

Patch

diff --git a/drivers/net/wireless/rt2x00/rt2800usb.c b/drivers/net/wireless/rt2x00/rt2800usb.c
index 20a5040..6cf3365 100644
--- a/drivers/net/wireless/rt2x00/rt2800usb.c
+++ b/drivers/net/wireless/rt2x00/rt2800usb.c
@@ -971,6 +971,7 @@  static struct usb_device_id rt2800usb_device_table[] = {
 	{ USB_DEVICE(0x0411, 0x015d) },
 	{ USB_DEVICE(0x0411, 0x016f) },
 	{ USB_DEVICE(0x0411, 0x01a2) },
+	{ USB_DEVICE(0x0411, 0x01ee) },
 	/* Corega */
 	{ USB_DEVICE(0x07aa, 0x002f) },
 	{ USB_DEVICE(0x07aa, 0x003c) },