linux-kernel - Re: [PATCH]usb: serial: pl2303: rm id

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:	Mon, 14 Nov 2011 09:11:30 +0800
From:	wangyanqing <udknight@...il.com>
To:	gregkh@...e.de
Cc:	wd@...x.de, stable@...nel.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH]usb: serial: pl2303: rm id

On Thu, Nov 10, 2011 at 02:04:08PM +0800, wangyanqing wrote:
> I get report from customer that his usb-serial
> converter doesn't work well,it sometimes work,
> but sometimes it doesn't.
> 
> The usb-serial converter's id:
> vendor_id product_id
> 0x4348    0x5523
> 
> Then I search the usb-serial codes, and there are
> two drivers announce support this device, pl2303
> and ch341, commit 026dfaf1 cause it. Through many
> times to test, ch341 works well with this device,
> and pl2303 doesn't work quite often(it just work quite little).
> 
> ch341 works well with this device, so we doesn't
> need pl2303 to support.I try to revert 026dfaf1 first,
> but it failed. So I prepare this patch by hand to revert it.
> 
> Signed-off-by: Wang YanQing <Udknight@...il.com>
> ---
>  drivers/usb/serial/pl2303.c |    1 -
>  drivers/usb/serial/pl2303.h |    4 ----
>  2 files changed, 0 insertions(+), 5 deletions(-)
> 
> diff --git a/drivers/usb/serial/pl2303.c b/drivers/usb/serial/pl2303.c
> index 9083d1e..fc2d66f 100644
> --- a/drivers/usb/serial/pl2303.c
> +++ b/drivers/usb/serial/pl2303.c
> @@ -91,7 +91,6 @@ static const struct usb_device_id id_table[] = {
>  	{ USB_DEVICE(SONY_VENDOR_ID, SONY_QN3USB_PRODUCT_ID) },
>  	{ USB_DEVICE(SANWA_VENDOR_ID, SANWA_PRODUCT_ID) },
>  	{ USB_DEVICE(ADLINK_VENDOR_ID, ADLINK_ND6530_PRODUCT_ID) },
> -	{ USB_DEVICE(WINCHIPHEAD_VENDOR_ID, WINCHIPHEAD_USBSER_PRODUCT_ID) },
>  	{ USB_DEVICE(SMART_VENDOR_ID, SMART_PRODUCT_ID) },
>  	{ }					/* Terminating entry */
>  };
> diff --git a/drivers/usb/serial/pl2303.h b/drivers/usb/serial/pl2303.h
> index 3d10d7f..c38b8c0 100644
> --- a/drivers/usb/serial/pl2303.h
> +++ b/drivers/usb/serial/pl2303.h
> @@ -145,10 +145,6 @@
>  #define ADLINK_VENDOR_ID		0x0b63
>  #define ADLINK_ND6530_PRODUCT_ID	0x6530
>  
> -/* WinChipHead USB->RS 232 adapter */
> -#define WINCHIPHEAD_VENDOR_ID		0x4348
> -#define WINCHIPHEAD_USBSER_PRODUCT_ID	0x5523
> -
>  /* SMART USB Serial Adapter */
>  #define SMART_VENDOR_ID	0x0b8c
>  #define SMART_PRODUCT_ID	0x2303
> -- 
> 1.7.3.4
> 
Hi all.
Maybe it should be noticed that not only one usb-serial converter base 
this chip have this issue, but there are about 50(even more) usb-serial 
coverter lines have the same issue, so we can't ignore it.
Of course we can make all the drivers work well, but we should make it
useful at the first.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/