infernalheaven / libmpsse

Automatically exported from code.google.com/p/libmpsse
0 stars 0 forks source link

SetMode sets all high byte pins to output #18

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
The comment says that pins are set to input/low, however, they are actually set 
to output (trish = 0xFF). This might be dangerous for some hardware.

Section 3.6.2 from:
http://www.ftdichip.com/Support/Documents/AppNotes/AN_108_Command_Processor_for_
MPSSE_and_MCU_Host_Bus_Emulation_Modes.pdf

Original issue reported on code.google.com by p.voorth...@gmail.com on 17 Jan 2013 at 1:57

GoogleCodeExporter commented 9 years ago
Thanks, that comment is clearly incorrect.

Currently libmpsse only supports writing to the ACBUS pins and does not provide 
any mechanism to read from the ACBUS pins, so setting those pins as inputs 
wouldn't make sense. They are initialized as outputs, and set low. Nothing 
should be connected to the ACBUS pins with the expectation that those pins are 
configured as inputs.

I've updated the code comments appropriately.

Original comment by heffne...@gmail.com on 28 Jan 2013 at 6:58