This is an automated email from Gerrit.

Diego Herranz ([email protected]) just uploaded a new patch set to 
Gerrit, which you can find at http://openocd.zylin.com/4908

-- gerrit

commit 5d40e87ed5fca48947cc1945060b3bb885f698cd
Author: Diego Herranz <[email protected]>
Date:   Wed Feb 13 08:33:17 2019 +0000

    src/jtag/drivers/ftdi: fix swd pin comment and links
    
    - TDI/DO is pin 1 and not pin 2. That matches with the mask used
    for that operation: 0x0002U, that is, pin 1.
    
    - FT2232D link didn't work anymore. It could have been updated to the
    current working FT2232D link but changed to link the FT2232H instead
    which is the latest generation and the most commonly used these days.
    
    - Taking advantage of the change, link to the MPSSE document updated.
    It still worked but through a redirection, so updated just in case the
    redirection stops working.
    
    Change-Id: Ia7d01bbcac2158026831edb010a6b50f58351e42
    Signed-off-by: Diego Herranz <[email protected]>

diff --git a/src/jtag/drivers/ftdi.c b/src/jtag/drivers/ftdi.c
index 1afedd1..9b7d4b2 100644
--- a/src/jtag/drivers/ftdi.c
+++ b/src/jtag/drivers/ftdi.c
@@ -53,11 +53,11 @@
  *
  * This code uses information contained in the MPSSE specification which was
  * found here:
- * http://www.ftdichip.com/Documents/AppNotes/AN2232C-01_MPSSE_Cmnd.pdf
+ * 
https://www.ftdichip.com/Support/Documents/AppNotes/AN2232C-01_MPSSE_Cmnd.pdf
  * Hereafter this is called the "MPSSE Spec".
  *
- * The datasheet for the ftdichip.com's FT2232D part is here:
- * http://www.ftdichip.com/Documents/DataSheets/DS_FT2232D.pdf
+ * The datasheet for the ftdichip.com's FT2232H part is here:
+ * https://www.ftdichip.com/Support/Documents/DataSheets/ICs/DS_FT2232H.pdf
  *
  * Also note the issue with code 0x4b (clock data to TMS) noted in
  * http://developer.intra2net.com/mailarchive/html/libftdi/2009/msg00292.html
@@ -1080,7 +1080,7 @@ static void ftdi_swd_swdio_en(bool enable)
                if (oe->data_mask)
                        ftdi_set_signal(oe, enable ? '1' : '0');
                else {
-                       /* Sets TDI/DO pin (pin 2) to input during rx when both 
pins are connected
+                       /* Sets TDI/DO pin (pin 1) to input during rx when both 
pins are connected
                           to SWDIO */
                        if (enable)
                                direction |= jtag_direction_init & 0x0002U;

-- 


_______________________________________________
OpenOCD-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/openocd-devel

Reply via email to