LKML Archive on lore.kernel.org
help / color / mirror / Atom feed
* [PATCH] USB 10/100 RJ45 Ethernet Network Adapter
@ 2011-01-27 20:37 Shahar Havivi
  0 siblings, 0 replies; 6+ messages in thread
From: Shahar Havivi @ 2011-01-27 20:37 UTC (permalink / raw)
  To: linux-kernel

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


^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH] USB 10/100 RJ45 Ethernet Network Adapter
  2011-02-22 19:24   ` David Miller
@ 2011-02-22 20:18     ` Sergei Shtylyov
  0 siblings, 0 replies; 6+ messages in thread
From: Sergei Shtylyov @ 2011-02-22 20:18 UTC (permalink / raw)
  To: David Miller
  Cc: sshtylyov, shaharh, linux-usb, linux-kernel, jacmet, gregkh, netdev

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

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH] USB 10/100 RJ45 Ethernet Network Adapter
  2011-02-22 19:18 ` Sergei Shtylyov
@ 2011-02-22 19:24   ` David Miller
  2011-02-22 20:18     ` Sergei Shtylyov
  0 siblings, 1 reply; 6+ messages in thread
From: David Miller @ 2011-02-22 19:24 UTC (permalink / raw)
  To: sshtylyov; +Cc: shaharh, linux-usb, linux-kernel, jacmet, gregkh, netdev

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)

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH] USB 10/100 RJ45 Ethernet Network Adapter
  2011-02-22 14:07 Shahar Havivi
  2011-02-22 14:21 ` Peter Korsgaard
@ 2011-02-22 19:18 ` Sergei Shtylyov
  2011-02-22 19:24   ` David Miller
  1 sibling, 1 reply; 6+ messages in thread
From: Sergei Shtylyov @ 2011-02-22 19:18 UTC (permalink / raw)
  To: Shahar Havivi
  Cc: linux-usb, linux-kernel, Peter Korsgaard, Greg Kroah-Hartman, netdev

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

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH] USB 10/100 RJ45 Ethernet Network Adapter
  2011-02-22 14:07 Shahar Havivi
@ 2011-02-22 14:21 ` Peter Korsgaard
  2011-02-22 19:18 ` Sergei Shtylyov
  1 sibling, 0 replies; 6+ messages in thread
From: Peter Korsgaard @ 2011-02-22 14:21 UTC (permalink / raw)
  To: Shahar Havivi; +Cc: linux-usb, linux-kernel, Greg Kroah-Hartman, netdev

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

-- 
Bye, Peter Korsgaard

^ permalink raw reply	[flat|nested] 6+ messages in thread

* [PATCH] USB 10/100 RJ45 Ethernet Network Adapter
@ 2011-02-22 14:07 Shahar Havivi
  2011-02-22 14:21 ` Peter Korsgaard
  2011-02-22 19:18 ` Sergei Shtylyov
  0 siblings, 2 replies; 6+ messages in thread
From: Shahar Havivi @ 2011-02-22 14:07 UTC (permalink / raw)
  To: linux-usb, linux-kernel; +Cc: Peter Korsgaard, Greg Kroah-Hartman, netdev

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


^ permalink raw reply	[flat|nested] 6+ messages in thread

end of thread, other threads:[~2011-02-22 20:19 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2011-01-27 20:37 [PATCH] USB 10/100 RJ45 Ethernet Network Adapter Shahar Havivi
2011-02-22 14:07 Shahar Havivi
2011-02-22 14:21 ` Peter Korsgaard
2011-02-22 19:18 ` Sergei Shtylyov
2011-02-22 19:24   ` David Miller
2011-02-22 20:18     ` Sergei Shtylyov

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).