[PATCH] USB: plusb: Add support for PL-2501

STALE5164d REVIEWED: 1 (0M)

1 review trailer.

4 messages, 2 authors, 2012-07-24 · open the first message on its own page

[PATCH] USB: plusb: Add support for PL-2501

From: kyak <hidden>
Date: 2012-07-23 11:44:16

From: Mikhail Peselnik <redacted>

This patch adds support for PL-2501 by adding the appropriate USB
ID's. This chip is used in several USB 'Easy Trasfer' Cables.

Signed-off-by: Mikhail Peselnik <redacted>
Tested-by: Mikhail Peselnik <redacted>
---
Now with proper sign-offs and right people in cc and unwrapped lines.
plusb driver (drivers/net/usb/plusb.c) doesn't recognize PL2501 chip.
Since PL2501 uses the same code as PL2301/PL2302 (PL2501 works in
compatibility mode with PL2301/PL2302), the fix is trivial and
attached as a patch.

Just to note: the patch is not mine, it can be found here and there on
Internet.
I've tested the patch and it works great.

Thank you.
--- linux-3.5/drivers/net/usb/plusb.c.orig	2012-07-22 21:06:41.905802795 +0400
+++ linux-3.5/drivers/net/usb/plusb.c	2012-07-22 21:07:00.345552404 +0400
@@ -107,7 +107,7 @@ static int pl_reset(struct usbnet *dev)
  }

  static const struct driver_info	prolific_info = {
-	.description =	"Prolific PL-2301/PL-2302/PL-25A1",
+	.description =	"Prolific PL-2301/PL-2302/PL-25A1/PL-2501",
  	.flags =	FLAG_POINTTOPOINT | FLAG_NO_SETINT,
  		/* some PL-2302 versions seem to fail usb_set_interface() */
  	.reset =	pl_reset,
@@ -139,6 +139,9 @@ static const struct usb_device_id	produc
  }, {
  	USB_DEVICE(0x050d, 0x258a),     /* Belkin F5U258/F5U279 (PL-25A1) */
  	.driver_info =  (unsigned long) &prolific_info,
+}, {
+	USB_DEVICE(0x067b, 0x2501),     /* PL-2501 */
+	.driver_info =  (unsigned long) &prolific_info,
  },

  	{ },		// END
@@ -158,5 +161,5 @@ static struct usb_driver plusb_driver =
  module_usb_driver(plusb_driver);

  MODULE_AUTHOR("David Brownell");
-MODULE_DESCRIPTION("Prolific PL-2301/2302/25A1 USB Host to Host Link Driver");
+MODULE_DESCRIPTION("Prolific PL-2301/2302/25A1/2501 USB Host to Host Link Driver");
  MODULE_LICENSE("GPL");

Re: [PATCH] USB: plusb: Add support for PL-2501

From: David Miller <davem@davemloft.net>
Date: 2012-07-23 20:08:40

From: kyak <redacted>
Date: Mon, 23 Jul 2012 15:44:11 +0400 (MSK)
From: Mikhail Peselnik <redacted>

This patch adds support for PL-2501 by adding the appropriate USB
ID's. This chip is used in several USB 'Easy Trasfer' Cables.

Signed-off-by: Mikhail Peselnik <redacted>
Tested-by: Mikhail Peselnik <redacted>
This does not apply cleanly to my net-next tree at all.

Re: [PATCH] USB: plusb: Add support for PL-2501

From: kyak <hidden>
Date: 2012-07-24 05:39:46

This patch was created against linux-3.5, but it applies perfectly 
against net-next tree, i just checked..

I'm sorry for not being able to submit the correct patch from the first 
attempt (and not even from the third attempt). Could you be more specific 
about "doesn't apply cleanly at all"? By the way, i'm perfectly fine if 
you just make this trivial change by yourself and take the credit, because 
our exchange of e-mails has become 20 times bigger than the patch 
itself. Probably sending another version of this patch from my side would 
be just another waste of (your) time.

On Mon, 23 Jul 2012, David Miller wrote:
From: kyak <redacted>
Date: Mon, 23 Jul 2012 15:44:11 +0400 (MSK)
quoted
From: Mikhail Peselnik <redacted>

This patch adds support for PL-2501 by adding the appropriate USB
ID's. This chip is used in several USB 'Easy Trasfer' Cables.

Signed-off-by: Mikhail Peselnik <redacted>
Tested-by: Mikhail Peselnik <redacted>
This does not apply cleanly to my net-next tree at all.

Re: [PATCH] USB: plusb: Add support for PL-2501

From: David Miller <davem@davemloft.net>
Date: 2012-07-24 06:57:49

From: kyak <redacted>
Date: Tue, 24 Jul 2012 09:39:40 +0400 (MSK)
This patch was created against linux-3.5, but it applies perfectly
against net-next tree, i just checked..

I'm sorry for not being able to submit the correct patch from the
first attempt (and not even from the third attempt). Could you be more
specific about "doesn't apply cleanly at all"?
"git am your_patch" fails.
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help