Patchwork asix: new device id

login
register
mail settings
Submitter Aurelien Jacobs
Date Dec. 16, 2011, 8:49 p.m.
Message ID <1324068562-3761-1-git-send-email-aurel@gnuage.org>
Download mbox | patch
Permalink /patch/131907/
State Accepted
Delegated to: David Miller
Headers show

Comments

Aurelien Jacobs - Dec. 16, 2011, 8:49 p.m.
Adds the device id needed for the USB Ethernet Adapter delivered by
ASUS with their Zenbook.

Signed-off-by: Aurelien Jacobs <aurel@gnuage.org>
---
 drivers/net/usb/asix.c |    4 ++++
 1 files changed, 4 insertions(+), 0 deletions(-)
David Miller - Dec. 19, 2011, 9 p.m.
From: Aurelien Jacobs <aurel@gnuage.org>
Date: Fri, 16 Dec 2011 21:49:22 +0100

> Adds the device id needed for the USB Ethernet Adapter delivered by
> ASUS with their Zenbook.
> 
> Signed-off-by: Aurelien Jacobs <aurel@gnuage.org>

Grant, ACK/NACK?
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Grant Grundler - Dec. 19, 2011, 9:22 p.m.
On Mon, Dec 19, 2011 at 1:00 PM, David Miller <davem@davemloft.net> wrote:
> From: Aurelien Jacobs <aurel@gnuage.org>
> Date: Fri, 16 Dec 2011 21:49:22 +0100
>
>> Adds the device id needed for the USB Ethernet Adapter delivered by
>> ASUS with their Zenbook.
>>
>> Signed-off-by: Aurelien Jacobs <aurel@gnuage.org>
>
> Grant, ACK/NACK?

Dave,
I'm totally ok with it if Aurelien is confident it "works" (well enough).

I'm really the wrong guy to ask though. I don't have the device
and I'm not associated with the mfg.

The right people to ask (CC'd) are Allan Chou and Freddy Xin (both from ASIX).

cheers!
grant
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
David Miller - Dec. 19, 2011, 9:25 p.m.
From: Grant Grundler <grundler@google.com>
Date: Mon, 19 Dec 2011 13:22:35 -0800

> I'm really the wrong guy to ask though. I don't have the device
> and I'm not associated with the mfg.

Strongly disagree.

When you make changes to a driver, in fact you've made the most
recent 5 or 6, it's like "tag, you're it" :-)

Just so you understand why I am not going to ask random people who
have never submitted a change to the driver whether a change makes
sense or not.
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Grant Grundler - Dec. 19, 2011, 9:34 p.m.
On Mon, Dec 19, 2011 at 1:25 PM, David Miller <davem@davemloft.net> wrote:
> From: Grant Grundler <grundler@google.com>
> Date: Mon, 19 Dec 2011 13:22:35 -0800
>
>> I'm really the wrong guy to ask though. I don't have the device
>> and I'm not associated with the mfg.
>
> Strongly disagree.
>
> When you make changes to a driver, in fact you've made the most
> recent 5 or 6, it's like "tag, you're it" :-)

Fair enough. :)   But I'm trying to change that. :) (Have ASIX to
participate in asix.c devel/support).


Acked-by: Grant Grundler <grundler@chromium.org>

The change is "harmless" in that it won't interfere with anything that
works today.

> Just so you understand why I am not going to ask random people who
> have never submitted a change to the driver whether a change makes
> sense or not.

Understood. But I have no clue what driver functionality *will* work
with the new device.
I guess this is the first step to finding out. :)

cheers,
grant
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
David Miller - Dec. 19, 2011, 9:38 p.m.
From: Grant Grundler <grundler@chromium.org>
Date: Mon, 19 Dec 2011 13:34:24 -0800

> On Mon, Dec 19, 2011 at 1:25 PM, David Miller <davem@davemloft.net> wrote:
>> From: Grant Grundler <grundler@google.com>
>> Date: Mon, 19 Dec 2011 13:22:35 -0800
>>
>>> I'm really the wrong guy to ask though. I don't have the device
>>> and I'm not associated with the mfg.
>>
>> Strongly disagree.
>>
>> When you make changes to a driver, in fact you've made the most
>> recent 5 or 6, it's like "tag, you're it" :-)
> 
> Fair enough. :)   But I'm trying to change that. :) (Have ASIX to
> participate in asix.c devel/support).
> 
> 
> Acked-by: Grant Grundler <grundler@chromium.org>

That's the spirit :-)

Aplied, thanks.
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
allan - Dec. 20, 2011, 12:41 a.m.
Dear David,

ASUS's USB to Ethernet adapter (delivered with Zenbook) implemented ASIX's latest USB 2.0 to Fast Ethernet solution, AX88772B inside. ASUS hopes to enable the WOL function on Windows system by default so we assigned a new VID/PID (0B95h/7E2Bh) to meet their requirement. 

For Linux native driver, you can easily add the new VID/PID (0B95h/7E2Bh) to work as the AX88772B (with default VID/PID, 0B95h/772Bh). Please feel free to let us know if you have any problems or need more information. Thanks a lot for your great helps.


---
Best regards,
Allan Chou
Technical Support Division
ASIX Electronics Corporation
TEL: 886-3-5799500 ext.228
FAX: 886-3-5799558
E-mail: allan@asix.com.tw 
http://www.asix.com.tw/ 

-----Original Message-----
From: Grant Grundler [mailto:grundler@google.com] 
Sent: Tuesday, December 20, 2011 5:23 AM
To: David Miller
Cc: aurel@gnuage.org; netdev@vger.kernel.org; Allan Chou; Freddy Xin
Subject: Re: [PATCH] asix: new device id

On Mon, Dec 19, 2011 at 1:00 PM, David Miller <davem@davemloft.net> wrote:
> From: Aurelien Jacobs <aurel@gnuage.org>
> Date: Fri, 16 Dec 2011 21:49:22 +0100
>
>> Adds the device id needed for the USB Ethernet Adapter delivered by
>> ASUS with their Zenbook.
>>
>> Signed-off-by: Aurelien Jacobs <aurel@gnuage.org>
>
> Grant, ACK/NACK?

Dave,
I'm totally ok with it if Aurelien is confident it "works" (well enough).

I'm really the wrong guy to ask though. I don't have the device
and I'm not associated with the mfg.

The right people to ask (CC'd) are Allan Chou and Freddy Xin (both from ASIX).

cheers!
grant

--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Patch

diff --git a/drivers/net/usb/asix.c b/drivers/net/usb/asix.c
index e6fed4d..e95f0e6 100644
--- a/drivers/net/usb/asix.c
+++ b/drivers/net/usb/asix.c
@@ -1655,6 +1655,10 @@  static const struct usb_device_id	products [] = {
 	// ASIX 88772a
 	USB_DEVICE(0x0db0, 0xa877),
 	.driver_info = (unsigned long) &ax88772_info,
+}, {
+	// Asus USB Ethernet Adapter
+	USB_DEVICE (0x0b95, 0x7e2b),
+	.driver_info = (unsigned long) &ax88772_info,
 },
 	{ },		// END
 };