| Message ID | 1324068562-3761-1-git-send-email-aurel@gnuage.org |
|---|---|
| State | Accepted, archived |
| Delegated to: | David Miller |
| Headers | show |
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
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
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
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
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
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
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 };
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(-)