Re: [PATCH] rt2x00: Add new USB ID to rt2500usb

[Ivo van Doorn <ivdoorn@gmail.com>]

On Sunday 10 February 2008, Michael Buesch wrote:
> On Sunday 10 February 2008 11:36:28 Ivo van Doorn wrote:
> > On Sunday 10 February 2008, Michael Buesch wrote:
> > > On Sunday 10 February 2008 11:21:57 Ivo van Doorn wrote:
> > > > 
> > > > Signed-off-by: Ivo van Doorn <IvDoorn@gmail.com>
> > > > ---
> > > > John, please send this patch to the 2.6.24/2.6.25 stable trees.
> > > > It is already present in the wireless-2.6 tree queued for 2.6.26.
> > > > 
> > > >  drivers/net/wireless/rt2x00/rt2500usb.c |    2 +-
> > > >  1 files changed, 1 insertions(+), 1 deletions(-)
> > > > 
> > > > diff --git a/drivers/net/wireless/rt2x00/rt2500usb.c b/drivers/net/wireless/rt2x00/rt2500usb.c
> > > > index cba7f57..c4a6ab4 100644
> > > > --- a/drivers/net/wireless/rt2x00/rt2500usb.c
> > > > +++ b/drivers/net/wireless/rt2x00/rt2500usb.c
> > > > @@ -1871,11 +1871,11 @@ static struct usb_device_id rt2500usb_device_table[] = {
> > > >  	/* Hercules */
> > > >  	{ USB_DEVICE(0x06f8, 0xe000), USB_DEVICE_DATA(&rt2500usb_ops) },
> > > >  	/* Melco */
> > > > +	{ USB_DEVICE(0x0411, 0x005e), USB_DEVICE_DATA(&rt2500usb_ops) },
> > > >  	{ USB_DEVICE(0x0411, 0x0066), USB_DEVICE_DATA(&rt2500usb_ops) },
> > > >  	{ USB_DEVICE(0x0411, 0x0067), USB_DEVICE_DATA(&rt2500usb_ops) },
> > > >  	{ USB_DEVICE(0x0411, 0x008b), USB_DEVICE_DATA(&rt2500usb_ops) },
> > > >  	{ USB_DEVICE(0x0411, 0x0097), USB_DEVICE_DATA(&rt2500usb_ops) },
> > > > -
> > > >  	/* MSI */
> > > >  	{ USB_DEVICE(0x0db0, 0x6861), USB_DEVICE_DATA(&rt2500usb_ops) },
> > > >  	{ USB_DEVICE(0x0db0, 0x6865), USB_DEVICE_DATA(&rt2500usb_ops) },
> > > 
> > > I'm unsure on how this is an actual _fix_ that we need in a stable kernel.
> > > We always handled adding IDs as features.
> > 
> > Ok, the user has to wait untill 2.6.26 then.
> 
> Oh, it seems that -stable does actually accept such patches:
> http://git.kernel.org/?p=linux/kernel/git/stable/linux-2.6.23.y.git;a=commitdiff;h=2cf220bb5b30a922aebdd5841a7975e02a70ce59
> http://git.kernel.org/?p=linux/kernel/git/stable/linux-2.6.23.y.git;a=commitdiff;h=955ab48db7fb9e3d74dc770cca9aa6b194e53025
> 
> I wasn't aware of this, because we always had the rule of ID additions
> being features, in the wireless team.
> So well. I think John should probably make an official decision. :)
> 
> I would suggest the following: Unless there's another driver in
> the tree that (partially) supports the device being added, adding the ID
> can not cause a regression. So it's OK to add it, even in a stable cycle.

The ID was previously unknown, and I have 2 users who reported success when the
ID was added to rt2500usb and the legacy rt2570 driver.

Ivo