Bug#584945: [PATCH 1/2] Staging: rtl8192s_usb: Fix device ID table

[Greg KH]

On Thu, Jun 24, 2010 at 01:42:02AM +0100, Ben Hutchings wrote:
> On Tue, 2010-06-22 at 15:34 -0700, Greg KH wrote:
> > On Fri, Jun 18, 2010 at 01:00:35AM +0100, Ben Hutchings wrote:
> > > The rtl8192s_usb driver was claiming a number of device IDs which are
> > > actually supported by the code in the rtl8192u_usb driver.  Remove
> > > them from rtl8192s_usb, and add more device IDs as listed in the
> > > vendor driver version 0003.0825.2009.
> > > 
> > > Signed-off-by: Ben Hutchings <ben at decadent.org.uk>
> > > ---
> > > I don't know whether the device IDs I'm adding actually work, but we may
> > > as well let people try them.
> > 
> > I just applied a patch from Florian that should have worked all of this
> > out.  Can you look at the linux-next tree tomorrow and verify that it is
> > all ok now?
> > 
> > If not, can you resend whichever ids from these two patches that you
> > feel are still needed to be changed?
> 
> The result in rtl8192su basically matches what I have, although Florian
> seems to have found a new version of the vendor driver with a few more
> IDs.  (Florian, please send me it so I can add it to the archive at
> <http://people.debian.org/~benh/rtl-wlan/>.)
> 
> I believe the second patch, for rtl8192u, is still applicable.
> 
> At the end of the table in rtl8192su, Florian added:
> 
> +       /* these are not in the official list */
> +       {USB_DEVICE(0x050d, 0x815F)}, /* Belkin F5D8053 v6 */
> +       {USB_DEVICE(0x0df6, 0x004b)}, /* WL-349 */
> 
> However the first of these is included in the vendor driver... and
> further up the table in our version!

Hm, ok, if you think something still needs to be changed, can you send
me a patch to do so?

thanks,

greg k-h