2014-02-02 09:22:41

by Ulrich Hahn

[permalink] [raw]
Subject: ..another device for the ftdi_sio driver, 2nd infusion

Thanks, Greg to your bot for the hint,

gmail seems to screw up tabs for spaces. Good to know, learning by doing..
Lets give it another try:

>finally I prepared a patch for the famous ftdi_sio usb serial driver.
>Adding two Tagsys RFID readers I am using.
>After having asked Bill Ryder he suggested adressing the
>linux-usb-devel mailing list. (what?)

>Sorry for the mess below - I am not doing this frequently - being
>rather a leecher over the last 20 years:-)


Signed-off-by: Ulrich Hahn <[email protected]>

diff -uprN linux-source-3.2.0/drivers/usb/serial/ftdi_sio.c linux-source-3.2.0+tagsysRFID/drivers/usb/serial/ftdi_sio.c
--- linux-source-3.2.0/drivers/usb/serial/ftdi_sio.c 2013-12-03 18:40:26.000000000 +0100
+++ linux-source-3.2.0+tagsysRFID/drivers/usb/serial/ftdi_sio.c 2014-02-01 16:32:40.146698791 +0100
@@ -202,6 +202,8 @@ static struct usb_device_id id_table_com
{ USB_DEVICE(INTERBIOMETRICS_VID, INTERBIOMETRICS_IOBOARD_PID) },
{ USB_DEVICE(INTERBIOMETRICS_VID, INTERBIOMETRICS_MINI_IOBOARD_PID) },
{ USB_DEVICE(FTDI_VID, FTDI_SPROG_II) },
+ { USB_DEVICE(FTDI_VID, FTDI_TAGSYS_LP101_PID) },
+ { USB_DEVICE(FTDI_VID, FTDI_TAGSYS_P200X_PID) },
{ USB_DEVICE(FTDI_VID, FTDI_LENZ_LIUSB_PID) },
{ USB_DEVICE(FTDI_VID, FTDI_XF_632_PID) },
{ USB_DEVICE(FTDI_VID, FTDI_XF_634_PID) },
diff -uprN linux-source-3.2.0/drivers/usb/serial/ftdi_sio_ids.h linux-source-3.2.0+tagsysRFID/drivers/usb/serial/ftdi_sio_ids.h
--- linux-source-3.2.0/drivers/usb/serial/ftdi_sio_ids.h 2013-12-03 18:40:26.000000000 +0100
+++ linux-source-3.2.0+tagsysRFID/drivers/usb/serial/ftdi_sio_ids.h 2014-02-02 09:11:35.183155244 +0100
@@ -363,6 +363,13 @@
/* Sprog II (Andrew Crosland's SprogII DCC interface) */
#define FTDI_SPROG_II 0xF0C8

+/*
+ * Tagsys RFID Readers
+ * survey (two of them) by Ulrich Hahn
+ */
+#define FTDI_TAGSYS_LP101_PID 0xF0E9 /* Tagsys L-P101 RFID*/
+#define FTDI_TAGSYS_P200X_PID 0xF0EE /* Tagsys Medio P200x RFID*/
+
/* an infrared receiver for user access control with IR tags */
#define FTDI_PIEGROUP_PID 0xF208 /* Product Id */


2014-02-02 09:31:30

by Greg Kroah-Hartman

[permalink] [raw]
Subject: Re: ..another device for the ftdi_sio driver, 2nd infusion

On Sun, Feb 02, 2014 at 10:22:16AM +0100, Ulrich Hahn wrote:
> Thanks, Greg to your bot for the hint,
>
> gmail seems to screw up tabs for spaces. Good to know, learning by doing..
> Lets give it another try:
>
> >finally I prepared a patch for the famous ftdi_sio usb serial driver.
> >Adding two Tagsys RFID readers I am using.
> >After having asked Bill Ryder he suggested adressing the
> >linux-usb-devel mailing list. (what?)
>
> >Sorry for the mess below - I am not doing this frequently - being
> >rather a leecher over the last 20 years:-)
>
>
> Signed-off-by: Ulrich Hahn <[email protected]>

That's better, but I have to edit the above by hand, which isn't a big
deal, but when you deal with tens of thousands of patches a year,
doesn't scale :(

>
> diff -uprN linux-source-3.2.0/drivers/usb/serial/ftdi_sio.c linux-source-3.2.0+tagsysRFID/drivers/usb/serial/ftdi_sio.c
> --- linux-source-3.2.0/drivers/usb/serial/ftdi_sio.c 2013-12-03 18:40:26.000000000 +0100
> +++ linux-source-3.2.0+tagsysRFID/drivers/usb/serial/ftdi_sio.c 2014-02-01 16:32:40.146698791 +0100

3.2.0 is over 2 years old, things have moved on a bit since then :)

Can you redo this patch against the 3.13.0 kernel release, to have a
chance for it to be able to be applied?

thanks,

greg k-h