2011-02-22 14:07:24

by Shahar Havivi

Subject: [PATCH] USB 10/100 RJ45 Ethernet Network Adapter

The device is very similar to (0x0fe6, 0x8101),
And works well with dm9601 driver.
---
drivers/net/usb/dm9601.c | 5 +++++
1 files changed, 5 insertions(+), 0 deletions(-)

diff --git a/drivers/net/usb/dm9601.c b/drivers/net/usb/dm9601.c
index 02b622e..2d1c8ae 100644
--- a/drivers/net/usb/dm9601.c
+++ b/drivers/net/usb/dm9601.c
@@ -654,6 +654,11 @@ static const struct usb_device_id products[] = {
USB_DEVICE(0x0a46, 0x9000), /* DM9000E */
.driver_info = (unsigned long)&dm9601_info,
},
+ {
+ USB_DEVICE(0x0fe6, 0x9700), /* DM9601 USB to Fast Ethernet Adapter */
+
+ .driver_info = (unsigned long)&dm9601_info,
+ },
{}, // END
};

--
1.7.3.4

2011-02-22 14:21:13

by Peter Korsgaard

Subject: Re: [PATCH] USB 10/100 RJ45 Ethernet Network Adapter

>>>>> "Shahar" == Shahar Havivi <[email protected]> writes:

Shahar> The device is very similar to (0x0fe6, 0x8101),
Shahar> And works well with dm9601 driver.
Shahar> ---
Shahar> drivers/net/usb/dm9601.c | 5 +++++
Shahar> 1 files changed, 5 insertions(+), 0 deletions(-)

Shahar> diff --git a/drivers/net/usb/dm9601.c b/drivers/net/usb/dm9601.c
Shahar> index 02b622e..2d1c8ae 100644
Shahar> --- a/drivers/net/usb/dm9601.c
Shahar> +++ b/drivers/net/usb/dm9601.c
Shahar> @@ -654,6 +654,11 @@ static const struct usb_device_id products[] = {
Shahar> USB_DEVICE(0x0a46, 0x9000), /* DM9000E */
Shahar> .driver_info = (unsigned long)&dm9601_info,
Shahar> },
Shahar> + {
Shahar> + USB_DEVICE(0x0fe6, 0x9700), /* DM9601 USB to Fast Ethernet Adapter */
Shahar> +
Shahar> + .driver_info = (unsigned long)&dm9601_info,
Shahar> + },
Shahar> {}, // END

Please put it after the 0x8100 device and get rid of the empty line
before .driver_info - Otherwise it looks good.

Acked-by: Peter Korsgaard <[email protected]>

--
Bye, Peter Korsgaard

2011-02-22 19:20:13

by Sergei Shtylyov

Subject: Re: [PATCH] USB 10/100 RJ45 Ethernet Network Adapter

Hello.

Shahar Havivi wrote:

> The device is very similar to (0x0fe6, 0x8101),
> And works well with dm9601 driver.

You should sign off your patch in order for it to be merged.

> ---
> drivers/net/usb/dm9601.c | 5 +++++
> 1 files changed, 5 insertions(+), 0 deletions(-)

> diff --git a/drivers/net/usb/dm9601.c b/drivers/net/usb/dm9601.c
> index 02b622e..2d1c8ae 100644
> --- a/drivers/net/usb/dm9601.c
> +++ b/drivers/net/usb/dm9601.c
> @@ -654,6 +654,11 @@ static const struct usb_device_id products[] = {
> USB_DEVICE(0x0a46, 0x9000), /* DM9000E */
> .driver_info = (unsigned long)&dm9601_info,
> },
> + {
> + USB_DEVICE(0x0fe6, 0x9700), /* DM9601 USB to Fast Ethernet Adapter */
> +

Extra empty line shouldn't be here I guess...

> + .driver_info = (unsigned long)&dm9601_info,
> + },
> {}, // END
> };

WBR, Sergei

2011-02-22 19:23:29

by David Miller

Subject: Re: [PATCH] USB 10/100 RJ45 Ethernet Network Adapter

From: Sergei Shtylyov <[email protected]>
Date: Tue, 22 Feb 2011 22:18:39 +0300

> Hello.
>
> Shahar Havivi wrote:
>
>> The device is very similar to (0x0fe6, 0x8101),
>> And works well with dm9601 driver.
>
> You should sign off your patch in order for it to be merged.

He did, see the follow-up patch with subject

[PATCH-v2] Added support for usb ethernet (0x0fe6, 0x9700)

2011-02-22 20:19:49

by Sergei Shtylyov

Subject: Re: [PATCH] USB 10/100 RJ45 Ethernet Network Adapter

Hello.

David Miller wrote:

>> Shahar Havivi wrote:

>>> The device is very similar to (0x0fe6, 0x8101),
>>> And works well with dm9601 driver.
>> You should sign off your patch in order for it to be merged.

> He did, see the follow-up patch with subject

> [PATCH-v2] Added support for usb ethernet (0x0fe6, 0x9700)

Yeah, I've somewhat hasted with following up.

WBR, Sergei