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

>>>>> "Shahar" == Shahar Havivi <shaharh@redhat.com> 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 <jacmet@sunsite.dk>

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
--
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: Sergei Shtylyov <sshtylyov@mvista.com>
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)
--
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

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