asix: new device id

Submitted by Aurelien Jacobs on Dec. 16, 2011, 8:49 p.m.

Details

Message ID 1324068562-3761-1-git-send-email-aurel@gnuage.org
State Accepted
Delegated to: David Miller
Headers show

Commit Message

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(-)

Comments

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 hide | download patch | download mbox

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
 };