linux-kernel - Re: [PATCH] rtlwifi: rtl8192cu: Add new device ID

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-Id: <201503260213.42322.marex@denx.de>
Date:	Thu, 26 Mar 2015 02:13:42 +0100
From:	Marek Vasut <marex@...x.de>
To:	Larry Finger <Larry.Finger@...inger.net>
Cc:	linux-wireless@...r.kernel.org, linux-kernel@...r.kernel.org,
	"John W. Linville" <linville@...driver.com>,
	Chaoming Li <chaoming_li@...lsil.com.cn>
Subject: Re: [PATCH] rtlwifi: rtl8192cu: Add new device ID

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 <marex@...x.de>
> > Cc: Larry Finger <Larry.Finger@...inger.net>
> > Cc: John W. Linville <linville@...driver.com>
> > Cc: Chaoming Li <chaoming_li@...lsil.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 majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/