opendous: Inhibit unnecessary state transitions13/1113/3
authorRoman Reichel <romanreichel@aol.de>
Tue, 15 Jan 2013 22:13:08 +0000 (15 23:13 +0100)
committerSpencer Oliver <spen@spen-soft.co.uk>
Wed, 16 Jan 2013 11:21:49 +0000 (16 11:21 +0000)
When current tap state and end state are the same, transitions are added which is not what should happen.
The usbprog driver was already patched like this long time ago.

Change-Id: I339e87156bdc7b5c83c10c14025b749605d3871a
Signed-off-by: Roman Reichel <romanreichel@aol.de>
Reviewed-on: http://openocd.zylin.com/1113
Tested-by: jenkins
Reviewed-by: Spencer Oliver <spen@spen-soft.co.uk>
src/jtag/drivers/opendous.c

index 1b74bd5..3a6cf5c 100644 (file)
@@ -422,7 +422,9 @@ void opendous_scan(int ir_scan, enum scan_type type, uint8_t *buffer, int scan_s
        /* Move to appropriate scan state */
        opendous_end_state(ir_scan ? TAP_IRSHIFT : TAP_DRSHIFT);
 
-       opendous_state_move();
+       if (tap_get_state() != tap_get_end_state())
+               opendous_state_move();
+
        opendous_end_state(saved_end_state);
 
        /* Scan */