FT2232H support
See original GitHub issueHi! I’m working on a board called Tigard at the moment, which is based off the FTDI FT2232H. The FT2232H is similar to the FT232H, but with dual MPSSEs. It would be awesome if I could use Tigard (or some of the other FT2232H-based boards such as timep) with CircuitPython/Blinka, and I see there is already support for the FT232H! I’d be happy to take this work on myself, but wanted to ask some questions before throwing a PR at you so my work can hopefully be merged upstream if this is something Blinka wants to support:
There are a few options for implementing AFAICT:
- Make a generic class that’s used for both the FT232H and the FT2232H classes (similar to how the “amlogic” folder is structured) . Since the two chips are really similar, I think this makes the most sense. I’m not sure about the name, maybe something along the lines of
ftdi
,ftdi-mpsse
, orftxxxx
? - Just copy the FT232H class and make a FT2232H class. This is easy and keeps the FT232H code as-is but super redundant.
- Simply modify the FT232H class to look for
ftdi:///1
instead offtdi://ftdi:ft232h/1
. This technically works (I have tested it), but you can only use the first MPSSE so it’s really not ideal.
I noticed there’s also issue #260, which is pretty similar to this, but I don’t think makes sense given how Blinka works? The difference is that this issue describes adding support for a new “microcontroller” (the FT2232H) that happens to have two MPSSEs, instead of adding support for using two of the same “microcontroller” (two FT232Hs) each with one MPSSE. However, if support for the latter was ever going to be added, I think a more generic FTDI class would help with making that easier.
I think the first is the clear choice, but I’m not super familiar with this library, so let me know what you think, thanks!
Edit: implemented the first, still needs some more testing but check it out here: https://github.com/tigard-tools/Adafruit_Blinka
Issue Analytics
- State:
- Created 3 years ago
- Comments:28 (17 by maintainers)
Top GitHub Comments
yes! we’d look at a PR that adds FT2232H support without breaking FT232H support, on mac/win/linux… but none have been submitted yet 😃
ok we also dont have an FT2232 right now, but we’ll find a dev board for one!