[PATCH] [USBNET] DM9501: Add Corega FEther USB-TXC support.

[PATCH] [USBNET] DM9501: Add Corega FEther USB-TXC support.

[YOSHIFUJI Hideaki / 吉藤英明]

Signed-off-by: YOSHIFUJI Hideaki <yoshfuji@linux-ipv6.org>
---
 drivers/usb/net/dm9601.c |    4 ++++
 1 files changed, 4 insertions(+), 0 deletions(-)

diff --git a/drivers/usb/net/dm9601.c b/drivers/usb/net/dm9601.c
index 4a932e1..c0bc52b 100644
--- a/drivers/usb/net/dm9601.c
+++ b/drivers/usb/net/dm9601.c
@@ -571,6 +571,10 @@ static const struct driver_info dm9601_info = {
 
 static const struct usb_device_id products[] = {
 	{
+	 USB_DEVICE(0x07aa, 0x9601),	/* Corega FEther USB-TXC */
+	 .driver_info = (unsigned long)&dm9601_info,
+	 },
+	{
 	 USB_DEVICE(0x0a46, 0x9601),	/* Davicom USB-100 */
 	 .driver_info = (unsigned long)&dm9601_info,
 	 },

-- 
YOSHIFUJI Hideaki @ USAGI Project  <yoshfuji@linux-ipv6.org>
GPG-FP  : 9022 65EB 1ECF 3AD1 0BDF  80D8 4807 F894 E062 0EEA

Re: [PATCH] [USBNET] DM9501: Add Corega FEther USB-TXC support.

[Jeff Garzik]

YOSHIFUJI Hideaki / 吉藤英明 wrote:
> Signed-off-by: YOSHIFUJI Hideaki <yoshfuji@linux-ipv6.org>
> ---
>  drivers/usb/net/dm9601.c |    4 ++++
>  1 files changed, 4 insertions(+), 0 deletions(-)
> 
> diff --git a/drivers/usb/net/dm9601.c b/drivers/usb/net/dm9601.c
> index 4a932e1..c0bc52b 100644
> --- a/drivers/usb/net/dm9601.c
> +++ b/drivers/usb/net/dm9601.c
> @@ -571,6 +571,10 @@ static const struct driver_info dm9601_info = {
>  
>  static const struct usb_device_id products[] = {
>  	{
> +	 USB_DEVICE(0x07aa, 0x9601),	/* Corega FEther USB-TXC */
> +	 .driver_info = (unsigned long)&dm9601_info,
> +	 },
> +	{


ACK the patch, though I wonder if this shouldn't instead go to Greg.

Honestly, I would prefer that the USB net drivers were moved into 
drivers/net with the other net drivers, /then/ I would merge such 
patches.  We don't add drivers for PCI-based hardware to 
drivers/pci/net, after all...

Re: [PATCH] [USBNET] DM9501: Add Corega FEther USB-TXC support.

[Greg KH]

On Fri, Mar 02, 2007 at 08:33:55PM -0500, Jeff Garzik wrote:
> YOSHIFUJI Hideaki / ???????????? wrote:
> >Signed-off-by: YOSHIFUJI Hideaki <yoshfuji@linux-ipv6.org>
> >---
> > drivers/usb/net/dm9601.c |    4 ++++
> > 1 files changed, 4 insertions(+), 0 deletions(-)
> >
> >diff --git a/drivers/usb/net/dm9601.c b/drivers/usb/net/dm9601.c
> >index 4a932e1..c0bc52b 100644
> >--- a/drivers/usb/net/dm9601.c
> >+++ b/drivers/usb/net/dm9601.c
> >@@ -571,6 +571,10 @@ static const struct driver_info dm9601_info = {
> > 
> > static const struct usb_device_id products[] = {
> > 	{
> >+	 USB_DEVICE(0x07aa, 0x9601),	/* Corega FEther USB-TXC */
> >+	 .driver_info = (unsigned long)&dm9601_info,
> >+	 },
> >+	{
> 
> 
> ACK the patch, though I wonder if this shouldn't instead go to Greg.
> 
> Honestly, I would prefer that the USB net drivers were moved into 
> drivers/net with the other net drivers, /then/ I would merge such 
> patches.  We don't add drivers for PCI-based hardware to 
> drivers/pci/net, after all...

I have no objection to that.  Things have been moving out of the
drivers/usb/ directory over time, and if you want to take these under
your umbrella too, that's fine with me.

David, any objections?

thanks,

greg k-h