Re: [PATCH] Add Realtek 8187B support

[Larry Finger <Larry.Finger@lwfinger.net>]

Herton Ronaldo Krzesinski wrote:
> 
> Can you open your dongle and see if you really have a RTL8187B? (as you
> said you had to force the ID). May be it wasn't really a 8187b device but
> another variant. I did many tests this week with a 8187b that continues to
> work ok.

I don't want to open it as it should still be under warranty. I'm
pretty sure it is an 8187b. When I tried it with the 8187 driver, it
gets an rf calibration error. By using your driver with the patch
below, it loaded and worked, at least for a while. It probably was a
natural death, but I felt I should issue a warning, jest in case.

I was going to hold the following patch until I had more to 
contribute; however, I don't know if I will be able to participate. 
Please apply the patch, as it lets devices like mine, where the 
iProduct string identifies is as an RTL8187B device, but the idProduct 
value is 0x8187, not 0x8189.

Thanks,

Larry

Index: wireless-testing/drivers/net/wireless/rtl8187_dev.c
===================================================================
--- wireless-testing.orig/drivers/net/wireless/rtl8187_dev.c
+++ wireless-testing/drivers/net/wireless/rtl8187_dev.c
@@ -946,7 +946,8 @@ static int __devinit rtl8187_probe(struc
  	}

  	priv = dev->priv;
-       priv->is_rtl8187b = (id->driver_info == DEVICE_RTL8187B);
+	priv->is_rtl8187b = (id->driver_info == DEVICE_RTL8187B) ||
+			    !memcmp(udev->product, "RTL8187B", 8);

  	SET_IEEE80211_DEV(dev, &intf->dev);
  	usb_set_intfdata(intf, dev);