altera.programmer: fix Windows-ism in load_bitstream.

whitequark · whitequark · commit 2b76c23e1703 · 2016-11-20T09:21:13.000Z
To specify a particular programmer, one has to specify it as e.g.
"USB-Blaster [USB-0]" on Windows (where 0 is its number on
the system) or e.g. "USB-Blaster [2-1.2]" on Linux (where 2-1.2
is its USB path). The existing default therefore did not work on
Linux.

This commit fixes the default to pick the first USB-Blaster, and
renames the argument to deliberately break old code that relies
on this nonportable behavior.
diff --git a/migen/build/altera/programmer.py b/migen/build/altera/programmer.py
@@ -6,8 +6,7 @@
 class USBBlaster(GenericProgrammer):
     needs_bitreverse = False
 
-    def load_bitstream(self, bitstream_file, port=0):
-        usb_port = "[USB-{}]".format(port)
+    def load_bitstream(self, bitstream_file, cable_suffix=""):
         subprocess.call(["quartus_pgm", "-m", "jtag", "-c",
-                         "USB-Blaster{}".format(usb_port), "-o",
+                         "USB-Blaster{}".format(cable_suffix), "-o",
                          "p;{}".format(bitstream_file)])
