Re: [PATCH v2] pl2303: distinguish between original and cloned HX chips

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Am 14.09.2013 14:19, schrieb Greg KH:
> On Sat, Sep 14, 2013 at 12:13:03PM +0200, Frank Schäfer wrote:
>> According to Prolific, several (unauthorized) cheap and less functional
>> clones of the PL2303HX chip are in circulation. [1]
> No footnote showed up in this changelog comment, care to fix this up?
Urgh... it got lost when updated the patch, sorry. Will fix that.

> And really, cloned chips for such a cheap device?  Ugh.
"Cloned" is the friendly appellation. You could also call it a crappy
fake device. ;)
Maximum baud rate is 115200 and it even doesn't support sending breaks.
But at least the standard baud rates from 75 to 115200, different parity
and data byte settings and the 4 handshaking lines seem to work.

>
>> I've had the chance to test such a cloned device and it turned out that
>> it doesn't support any baud rates above 115200 baud (original: 6 Mbaud)
>> It also doesn't support the divisior based baud rate encoding method,
>> so no continuous baud rate adjustment is possible.
>> Nevertheless, these devices have been working (unintentionally) with
>> the driver up to commit 61fa8d694b ("pl2303: also use the divisor based
>> baud rate encoding method for baud rates < 115200 with HX chips"), and
>> this commit broke support for them.
>> Fortunately, it is pretty simple to distinguish between the original
>> and the cloned HX chips, so I've added a check and an extra chip type
>> to keep the clones working.
>> The same check is used by the latest Prolific Windows driver, so it
>> should be solid.
> Patch looks reasonable, but can you add the link you refer to above
> somewhere?
v3 is coming...

Regards,
Frank
>
> thanks,
>
> greg k-h

--
To unsubscribe from this list: send the line "unsubscribe linux-usb" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Index of Archives]     [Linux Media]     [Linux Input]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [Old Linux USB Devel Archive]

  Powered by Linux