On Thursday, March 26, 2015 at 01:50:59 AM, Larry Finger wrote:
> On 03/25/2015 04:18 PM, Marek Vasut wrote:
> > Add new ID for ASUS N10 WiFi dongle.
> >
> > Signed-off-by: Marek Vasut <ma...@denx.de>
> > Cc: Larry Finger <larry.fin...@lwfinger.net>
> > Cc: John W. Linville <linvi...@tuxdriver.com>
> > Cc: Chaoming Li <chaoming...@realsil.com.cn>
> > ---
> >
> > drivers/net/wireless/rtlwifi/rtl8192cu/sw.c | 1 +
> > 1 file changed, 1 insertion(+)
> >
> > Bus 001 Device 004: ID 0b05:17ba ASUSTek Computer, Inc.
> >
> > Device Descriptor:
> > bLength 18
> > bDescriptorType 1
> > bcdUSB 2.00
[...]
> > diff --git a/drivers/net/wireless/rtlwifi/rtl8192cu/sw.c
> > b/drivers/net/wireless/rtlwifi/rtl8192cu/sw.c index 90a714c..d451d2b
> > 100644
> > --- a/drivers/net/wireless/rtlwifi/rtl8192cu/sw.c
> > +++ b/drivers/net/wireless/rtlwifi/rtl8192cu/sw.c
> > @@ -360,6 +360,7 @@ static struct usb_device_id rtl8192c_usb_ids[] = {
> >
> > {RTL_USB_DEVICE(0x04f2, 0xaffb, rtl92cu_hal_cfg)}, /*Xavi*/
> > {RTL_USB_DEVICE(0x04f2, 0xaffc, rtl92cu_hal_cfg)}, /*Xavi*/
> > {RTL_USB_DEVICE(0x2019, 0x1201, rtl92cu_hal_cfg)}, /*Planex-Vencer*/
> >
> > + {RTL_USB_DEVICE(0x0b05, 0x17ba, rtl92cu_hal_cfg)}, /*ASUS-Edimax*/
> >
> > /****** 8192CU ********/
> > {RTL_USB_DEVICE(0x050d, 0x1004, rtl92cu_hal_cfg)}, /*Belcom-SurfN300*/
Hi!
> The full dump from lsusb is a bit much for a commit message. All you need
> is the one line from lsusb with no switches set.
That's why I added it below the diffstat, so that it doesn't polute the commit
message.
> According to the Vendor driver, which the comment indicates as the source
> of part of your patch,
In fact, the comment came from that same source file, from 0b05:17ab line.
The vendor is the same though, so I kept the comment.
> this particular device belongs in the "Customer ID"
> section under 8188CU, not where you placed it. In addition, I like to keep
> the USB IDs in numerical order within each section. Accordingly, your new
> ID should be between 0846:9041 and 0bda:5088.
OK.
> I am assuming that you have such a device. A statement that this change
> allows it to work should be in the commit message. If you have not tested,
> then you need to state that.
I have one and I tested it, I mostly checked the other commit messages and
wrote the same thing. It's a rather mindless change afterall.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
