Skip to content
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

Add detection for boards using CP210x/CH340 bridge. #197

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

jose1711
Copy link

This means many ESP32/ESP8266-based dev. boards are now automatically
recognized.

This means many ESP32/ESP8266-based dev. boards are now automatically
recognized.
@dhylands
Copy link
Owner

dhylands commented Sep 6, 2022

The issue I have with this is that non-Micropython boards which also use CP310/CH340 chips will also be detected.

What I'd rather see is having something like a .rshell configuration file where you could enter things like PID/VID and have the autoconnect work off of that. Then if you have other boards, you can tweak the settings to work for your own situation.

The configuration could also have things like don't automatically set the clock, and any options which are available via the command line. I wouldn't necessarily put all of that in a the beginning, but knowing the direction to expand in would be useful.

Peter Hinch has a fork where he adds some macro capabilities that I've been meaning to review and merge, and that mechanism should probably be looked at as well.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants