Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]
Groups > linux.kernel > #1355170 > unrolled thread
| Started by | Josh Boyer <jwboyer@fedoraproject.org> |
|---|---|
| First post | 2016-03-10 15:50 +0100 |
| Last post | 2016-03-16 17:20 +0100 |
| Articles | 3 — 2 participants |
Back to article view | Back to linux.kernel
[PATCH] USB: serial: ftdi_sio: Add support for ICP DAS I-756xU devices Josh Boyer <jwboyer@fedoraproject.org> - 2016-03-10 15:50 +0100
Re: [PATCH] USB: serial: ftdi_sio: Add support for ICP DAS I-756xU devices Josh Boyer <jwboyer@fedoraproject.org> - 2016-03-16 17:00 +0100
Re: [PATCH] USB: serial: ftdi_sio: Add support for ICP DAS I-756xU devices Johan Hovold <johan@kernel.org> - 2016-03-16 17:20 +0100
| From | Josh Boyer <jwboyer@fedoraproject.org> |
|---|---|
| Date | 2016-03-10 15:50 +0100 |
| Subject | [PATCH] USB: serial: ftdi_sio: Add support for ICP DAS I-756xU devices |
| Message-ID | <rba4y-8hd-7@gated-at.bofh.it> |
A Fedora user reports that the ftdi_sio driver works properly for the
ICP DAS I-7561U device. Further, the user manual for these devices
instructs users to load the driver and add the ids using the sysfs
interface.
Add support for these in the driver directly so that the devices work
out of the box instead of needing manual configuration.
Reported-by: <thesource@mail.ru>
CC: stable <stable@vger.kernel.org>
Signed-off-by: Josh Boyer <jwboyer@fedoraproject.org>
---
drivers/usb/serial/ftdi_sio.c | 4 ++++
drivers/usb/serial/ftdi_sio_ids.h | 8 ++++++++
2 files changed, 12 insertions(+)
diff --git a/drivers/usb/serial/ftdi_sio.c b/drivers/usb/serial/ftdi_sio.c
index 8c660ae401d8..b61f12160d37 100644
--- a/drivers/usb/serial/ftdi_sio.c
+++ b/drivers/usb/serial/ftdi_sio.c
@@ -1004,6 +1004,10 @@ static const struct usb_device_id id_table_combined[] = {
{ USB_DEVICE(FTDI_VID, CHETCO_SEASMART_DISPLAY_PID) },
{ USB_DEVICE(FTDI_VID, CHETCO_SEASMART_LITE_PID) },
{ USB_DEVICE(FTDI_VID, CHETCO_SEASMART_ANALOG_PID) },
+ /* ICP DAS I-756xU devices */
+ { USB_DEVICE(ICPDAS_VID, ICPDAS_I7560U_PID) },
+ { USB_DEVICE(ICPDAS_VID, ICPDAS_I7561U_PID) },
+ { USB_DEVICE(ICPDAS_VID, ICPDAS_I7563U_PID) },
{ } /* Terminating entry */
};
diff --git a/drivers/usb/serial/ftdi_sio_ids.h b/drivers/usb/serial/ftdi_sio_ids.h
index a84df2513994..a4ec24ce6a11 100644
--- a/drivers/usb/serial/ftdi_sio_ids.h
+++ b/drivers/usb/serial/ftdi_sio_ids.h
@@ -872,6 +872,14 @@
#define NOVITUS_BONO_E_PID 0x6010
/*
+ * ICPDAS I-756*U devices
+ */
+#define ICPDAS_VID 0x1b5c
+#define ICPDAS_I7560U_PID 0x0103
+#define ICPDAS_I7561U_PID 0x0104
+#define ICPDAS_I7563U_PID 0x0105
+
+/*
* RT Systems programming cables for various ham radios
*/
#define RTSYSTEMS_VID 0x2100 /* Vendor ID */
--
2.5.0
[toc] | [next] | [standalone]
| From | Josh Boyer <jwboyer@fedoraproject.org> |
|---|---|
| Date | 2016-03-16 17:00 +0100 |
| Message-ID | <rdm1z-2tH-3@gated-at.bofh.it> |
| In reply to | #1355170 |
On Thu, Mar 10, 2016 at 9:48 AM, Josh Boyer <jwboyer@fedoraproject.org> wrote:
> A Fedora user reports that the ftdi_sio driver works properly for the
> ICP DAS I-7561U device. Further, the user manual for these devices
> instructs users to load the driver and add the ids using the sysfs
> interface.
>
> Add support for these in the driver directly so that the devices work
> out of the box instead of needing manual configuration.
>
> Reported-by: <thesource@mail.ru>
> CC: stable <stable@vger.kernel.org>
> Signed-off-by: Josh Boyer <jwboyer@fedoraproject.org>
> ---
Anything further I need to do on this patch?
josh
> drivers/usb/serial/ftdi_sio.c | 4 ++++
> drivers/usb/serial/ftdi_sio_ids.h | 8 ++++++++
> 2 files changed, 12 insertions(+)
>
> diff --git a/drivers/usb/serial/ftdi_sio.c b/drivers/usb/serial/ftdi_sio.c
> index 8c660ae401d8..b61f12160d37 100644
> --- a/drivers/usb/serial/ftdi_sio.c
> +++ b/drivers/usb/serial/ftdi_sio.c
> @@ -1004,6 +1004,10 @@ static const struct usb_device_id id_table_combined[] = {
> { USB_DEVICE(FTDI_VID, CHETCO_SEASMART_DISPLAY_PID) },
> { USB_DEVICE(FTDI_VID, CHETCO_SEASMART_LITE_PID) },
> { USB_DEVICE(FTDI_VID, CHETCO_SEASMART_ANALOG_PID) },
> + /* ICP DAS I-756xU devices */
> + { USB_DEVICE(ICPDAS_VID, ICPDAS_I7560U_PID) },
> + { USB_DEVICE(ICPDAS_VID, ICPDAS_I7561U_PID) },
> + { USB_DEVICE(ICPDAS_VID, ICPDAS_I7563U_PID) },
> { } /* Terminating entry */
> };
>
> diff --git a/drivers/usb/serial/ftdi_sio_ids.h b/drivers/usb/serial/ftdi_sio_ids.h
> index a84df2513994..a4ec24ce6a11 100644
> --- a/drivers/usb/serial/ftdi_sio_ids.h
> +++ b/drivers/usb/serial/ftdi_sio_ids.h
> @@ -872,6 +872,14 @@
> #define NOVITUS_BONO_E_PID 0x6010
>
> /*
> + * ICPDAS I-756*U devices
> + */
> +#define ICPDAS_VID 0x1b5c
> +#define ICPDAS_I7560U_PID 0x0103
> +#define ICPDAS_I7561U_PID 0x0104
> +#define ICPDAS_I7563U_PID 0x0105
> +
> +/*
> * RT Systems programming cables for various ham radios
> */
> #define RTSYSTEMS_VID 0x2100 /* Vendor ID */
> --
> 2.5.0
>
[toc] | [prev] | [next] | [standalone]
| From | Johan Hovold <johan@kernel.org> |
|---|---|
| Date | 2016-03-16 17:20 +0100 |
| Subject | Re: [PATCH] USB: serial: ftdi_sio: Add support for ICP DAS I-756xU devices |
| Message-ID | <rdmkW-2QE-17@gated-at.bofh.it> |
| In reply to | #1359111 |
On Wed, Mar 16, 2016 at 11:55:49AM -0400, Josh Boyer wrote: > On Thu, Mar 10, 2016 at 9:48 AM, Josh Boyer <jwboyer@fedoraproject.org> wrote: > > A Fedora user reports that the ftdi_sio driver works properly for the > > ICP DAS I-7561U device. Further, the user manual for these devices > > instructs users to load the driver and add the ids using the sysfs > > interface. > > > > Add support for these in the driver directly so that the devices work > > out of the box instead of needing manual configuration. > > > > Reported-by: <thesource@mail.ru> > > CC: stable <stable@vger.kernel.org> > > Signed-off-by: Josh Boyer <jwboyer@fedoraproject.org> > > --- > > Anything further I need to do on this patch? It looks ok so I don't think so, but I'll take a closer look when processing my queue after the merge window closes. Thanks, Johan
[toc] | [prev] | [standalone]
Back to top | Article view | linux.kernel
csiph-web