-
Notifications
You must be signed in to change notification settings - Fork 254
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Expanded support for CP210x family USB serial devices #329
Comments
@dwhinham Thank you for the comprehensive information as always! I will try to implement the second option. I'll need a few days. |
There is a modified CP210x driver on the develop branch. This went quicker than expected. I tested it with a CP2102. Please let me know, if more modifications are necessary. I'm not sure, if we should add more vendor/device ID combinations. The Linux driver has a lot of them. I wouldn't include them all, but perhaps the missing SiLabs factory defaults should be added (0xEA61, 0xEA70). |
Perfect solution, thanks Rene. I personally can't think of any further changes necessary, and the way you have implemented this makes it very easy to extend later on if need be. Cheers! |
OK, thanks. Then we should wait, if somebody wants another modification in the coming time. Cheers! |
This feature is in Circle 45.2 now. |
This issue was reported by a user with a CP2104 device (following on from dwhinham/mt32-pi#305).
Removing a part number check in the CP2102 driver (relevant code) apparently gets it working for CP2104, and possibly other members of the CP210x family as they appear to be very similar.
This PDF (page 2) suggests that the differences are very minor (e.g. number of stop bits supported).
Searching for
partnum
within the Linux driver also reveals where behaviour varies depending on the part number.It would probably make sense to:
I would offer to help, but sadly I don't possess any of the devices concerned, hence I'm only able to ensure the issue is tracked.
Many thanks as always!
The text was updated successfully, but these errors were encountered: