On Mon, Jan 30, 2017 at 07:26:40PM +0100, Marcel J.E. Mol wrote:
> 
> Seems that ATEN serial-to-usb devices using pl2303 exist with
> different device ids. This patch adds a missing device ID so it
> is recognised by the driver.
> 
> Signed-off-by: Marcel J.E. Mol <mar...@mesa.nl>
> ---

Thanks for the patch. I dropped that stray "Subject: " from the commit
summary and changed it to

        "USB: serial: pl2303: add ATEN device ID"

before applying.

When replying to this one the first time I noticed that you'd also
gotten the linux-usb mailing list address wrong. Fixed up and resent
with full context now.

Johan

>  drivers/usb/serial/pl2303.c | 2 ++
>  drivers/usb/serial/pl2303.h | 1 +
>  2 files changed, 3 insertions(+)
> 
> diff --git a/drivers/usb/serial/pl2303.c b/drivers/usb/serial/pl2303.c
> index 46fca6b..8e61975 100644
> --- a/drivers/usb/serial/pl2303.c
> +++ b/drivers/usb/serial/pl2303.c
> @@ -49,7 +49,8 @@ static const struct usb_device_id id_table[] = {
>       { USB_DEVICE(IODATA_VENDOR_ID, IODATA_PRODUCT_ID) },
>       { USB_DEVICE(IODATA_VENDOR_ID, IODATA_PRODUCT_ID_RSAQ5) },
>       { USB_DEVICE(ATEN_VENDOR_ID, ATEN_PRODUCT_ID) },
> +     { USB_DEVICE(ATEN_VENDOR_ID, ATEN_PRODUCT_ID2) },
>       { USB_DEVICE(ATEN_VENDOR_ID2, ATEN_PRODUCT_ID) },
>       { USB_DEVICE(ELCOM_VENDOR_ID, ELCOM_PRODUCT_ID) },
>       { USB_DEVICE(ELCOM_VENDOR_ID, ELCOM_PRODUCT_ID_UCSGT) },
>       { USB_DEVICE(ITEGNO_VENDOR_ID, ITEGNO_PRODUCT_ID) },
> diff --git a/drivers/usb/serial/pl2303.h b/drivers/usb/serial/pl2303.h
> index e3b7af8..09d9be8 100644
> --- a/drivers/usb/serial/pl2303.h
> +++ b/drivers/usb/serial/pl2303.h
> @@ -27,6 +27,7 @@
>  #define ATEN_VENDOR_ID               0x0557
>  #define ATEN_VENDOR_ID2              0x0547
>  #define ATEN_PRODUCT_ID              0x2008
> +#define ATEN_PRODUCT_ID2     0x2118
>  
>  #define IODATA_VENDOR_ID     0x04bb
>  #define IODATA_PRODUCT_ID    0x0a03
--
To unsubscribe from this list: send the line "unsubscribe linux-usb" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to