Dick Hollenbeck <dick@softplc.com> comments & 7 clock TMS reset for good measure
git-svn-id: svn://svn.berlios.de/openocd/trunk@1618 b42882b7-edfa-0310-969c-e2dbd0fdcd60
This commit is contained in:
parent
cd0512befd
commit
c8e52f9e15
|
@ -938,7 +938,7 @@ int MINIDRIVER(interface_jtag_add_tlr)(void)
|
|||
|
||||
void jtag_add_pathmove(int num_states, tap_state_t *path)
|
||||
{
|
||||
tap_state_t cur_state=cmd_queue_cur_state;
|
||||
tap_state_t cur_state = cmd_queue_cur_state;
|
||||
int i;
|
||||
int retval;
|
||||
|
||||
|
@ -956,6 +956,7 @@ void jtag_add_pathmove(int num_states, tap_state_t *path)
|
|||
LOG_ERROR("BUG: TAP_RESET is not a valid state for pathmove sequences");
|
||||
exit(-1);
|
||||
}
|
||||
|
||||
if ( tap_state_transition(cur_state, true) != path[i]
|
||||
&& tap_state_transition(cur_state, false) != path[i])
|
||||
{
|
||||
|
@ -967,7 +968,7 @@ void jtag_add_pathmove(int num_states, tap_state_t *path)
|
|||
|
||||
jtag_prelude1();
|
||||
|
||||
retval=interface_jtag_add_pathmove(num_states, path);
|
||||
retval = interface_jtag_add_pathmove(num_states, path);
|
||||
cmd_queue_cur_state = path[num_states - 1];
|
||||
if (retval!=ERROR_OK)
|
||||
jtag_error=retval;
|
||||
|
@ -1490,7 +1491,6 @@ int jtag_examine_chain(void)
|
|||
|
||||
field.in_handler = NULL;
|
||||
|
||||
|
||||
for (i = 0; i < JTAG_MAX_CHAIN_SIZE; i++)
|
||||
{
|
||||
buf_set_u32(idcode_buffer, i * 32, 32, 0x000000FF);
|
||||
|
@ -1668,7 +1668,6 @@ int jtag_validate_chain(void)
|
|||
|
||||
field.in_handler = NULL;
|
||||
|
||||
|
||||
jtag_add_plain_ir_scan(1, &field, TAP_RESET);
|
||||
jtag_execute_queue();
|
||||
|
||||
|
@ -3127,18 +3126,29 @@ static struct
|
|||
some long-standing problems.
|
||||
Jeff
|
||||
|
||||
I added the bit count into the table
|
||||
I added the bit count into the table, reduced RESET column to 7 bits from 8.
|
||||
Dick
|
||||
|
||||
state specific comments:
|
||||
------------------------
|
||||
*->RESET tried the 5 bit reset and it gave me problems, 7 bits seems to
|
||||
work better on ARM9 with ft2232 driver. (Dick)
|
||||
|
||||
RESET->DRSHIFT add 1 extra clock cycles in the RESET state before advancing.
|
||||
needed on ARM9 with ft2232 driver. (Dick)
|
||||
|
||||
RESET->IRSHIFT add 1 extra clock cycles in the RESET state before advancing.
|
||||
needed on ARM9 with ft2232 driver. (Dick)
|
||||
*/
|
||||
|
||||
/* to state: */
|
||||
/* RESET IDLE DRSHIFT DRPAUSE IRSHIFT IRPAUSE */ /* from state: */
|
||||
{ B8(11111,5), B8(0,1), B8(0010,4), B8(01010,5), B8(00110,5), B8(010110,6) }, /* RESET */
|
||||
{ B8(11111,5), B8(0,1), B8(001,3), B8(0101,4), B8(0011,4), B8(01011,5) }, /* IDLE */
|
||||
{ B8(11111,5), B8(011,3), B8(00111,5), B8(01,2), B8(001111,6), B8(0101111,7) }, /* DRSHIFT */
|
||||
{ B8(11111,5), B8(011,3), B8(01,2), B8(0,1), B8(001111,6), B8(0101111,7) }, /* DRPAUSE */
|
||||
{ B8(11111,5), B8(011,3), B8(00111,5), B8(010111,6), B8(001111,6), B8(01,2) }, /* IRSHIFT */
|
||||
{ B8(11111,5), B8(011,3), B8(00111,5), B8(010111,6), B8(01,2), B8(0,1) } /* IRPAUSE */
|
||||
{ B8(1111111,7), B8(0,1), B8(00101,5), B8(01010,5), B8(001101,6), B8(010110,6) }, /* RESET */
|
||||
{ B8(1111111,7), B8(0,1), B8(001,3), B8(0101,4), B8(0011,4), B8(01011,5) }, /* IDLE */
|
||||
{ B8(1111111,7), B8(011,3), B8(00111,5), B8(01,2), B8(001111,6), B8(0101111,7) }, /* DRSHIFT */
|
||||
{ B8(1111111,7), B8(011,3), B8(01,2), B8(0,1), B8(001111,6), B8(0101111,7) }, /* DRPAUSE */
|
||||
{ B8(1111111,7), B8(011,3), B8(00111,5), B8(010111,6), B8(001111,6), B8(01,2) }, /* IRSHIFT */
|
||||
{ B8(1111111,7), B8(011,3), B8(00111,5), B8(010111,6), B8(01,2), B8(0,1) } /* IRPAUSE */
|
||||
|
||||
#else /* this is the old table, converted from hex and with the bit_count set to 7 for each combo, like before */
|
||||
|
||||
|
@ -3149,7 +3159,7 @@ static struct
|
|||
{ B8(1111111,7), B8(0110001,7), B8(0000000,7), B8(0000001,7), B8(0001111,7), B8(0101111,7) }, /* DRSHIFT */
|
||||
{ B8(1111111,7), B8(0110000,7), B8(0100000,7), B8(0010111,7), B8(0011110,7), B8(0101111,7) }, /* DRPAUSE */
|
||||
{ B8(1111111,7), B8(0110001,7), B8(0000111,7), B8(0010111,7), B8(0000000,7), B8(0000001,7) }, /* IRSHIFT */
|
||||
{ B8(1111111,7), B8(0110000,7), B8(0011100,7), B8(0010111,7), B8(0100000,7), B8(0101111,7) }, /* IRPAUSE */
|
||||
{ B8(1111111,7), B8(0110000,7), B8(0011100,7), B8(0010111,7), B8(0100000,7), B8(0101111,7) } /* IRPAUSE */
|
||||
|
||||
#endif
|
||||
|
||||
|
|
Loading…
Reference in New Issue