Commit 77c529e8 authored by Damien George's avatar Damien George

tools/mpremote: Use available ports instead of auto-connect list.

Using just the list of available ports, instead of a hard-coded list of
possible ports, means that all ports will be available for auto connection.
And the order that they will be attempted in will match what's printed by
"mpremote connect list" (and will be the same as before, trying ACMx before
USBx).  Auto-connect will also now work on Mac, and will allow all COM
ports on Windows.
Signed-off-by: default avatarDamien George <damien@micropython.org>
parent b0b8ebc4
......@@ -25,21 +25,6 @@ from .console import Console, ConsolePosix
_PROG = "mpremote"
_AUTO_CONNECT_SEARCH_LIST = [
"/dev/ttyACM0",
"/dev/ttyACM1",
"/dev/ttyACM2",
"/dev/ttyACM3",
"/dev/ttyUSB0",
"/dev/ttyUSB1",
"/dev/ttyUSB2",
"/dev/ttyUSB3",
"COM0",
"COM1",
"COM2",
"COM3",
]
_BUILTIN_COMMAND_EXPANSIONS = {
# Device connection shortcuts.
"a0": "connect /dev/ttyACM0",
......@@ -187,11 +172,9 @@ def do_connect(args):
return None
elif dev == "auto":
# Auto-detect and auto-connect to the first available device.
ports = serial.tools.list_ports.comports()
for dev in _AUTO_CONNECT_SEARCH_LIST:
if any(p.device == dev for p in ports):
for p in sorted(serial.tools.list_ports.comports()):
try:
return pyboard.PyboardExtended(dev, baudrate=115200)
return pyboard.PyboardExtended(p.device, baudrate=115200)
except pyboard.PyboardError as er:
if not er.args[0].startswith("failed to access"):
raise er
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment