[Openocd-svn] r2055 - trunk/src/jtag

[<zw...@ma...>]

Author: zwelch
Date: 2009-06-05 01:26:58 +0200 (Fri, 05 Jun 2009)
New Revision: 2055

Modified:
   trunk/src/jtag/jtag.c
   trunk/src/jtag/jtag.h
Log:
Clean up the core JTAG TAP APIs:
- Move jtag_tap_name to same location as other TAP functions; export it.
- Factor new jtag_tap_add() from jim_newtap_cmd(); appends TAP to global list.
- Move static chain position counter to global; use in jtag_NumTotalTaps().
- Use jtag_AllTaps for reading tap list, instead of accessing global directly.


Modified: trunk/src/jtag/jtag.c
===================================================================
--- trunk/src/jtag/jtag.c	2009-06-04 22:43:34 UTC (rev 2054)
+++ trunk/src/jtag/jtag.c	2009-06-04 23:26:58 UTC (rev 2055)
@@ -63,7 +63,15 @@
 int jtag_trst = 0;
 int jtag_srst = 0;
 
+/**
+ * List all TAPs that have been created.
+ */
 static jtag_tap_t *jtag_all_taps = NULL;
+/**
+ * The number of TAPs in the jtag_all_taps list, used to track the
+ * assigned chain position to new TAPs
+ */
+static int jtag_num_taps = 0;
 
 enum reset_types jtag_reset_config = RESET_NONE;
 tap_state_t cmd_queue_end_state = TAP_RESET;
@@ -235,16 +243,7 @@
 
 int jtag_NumTotalTaps(void)
 {
-	jtag_tap_t *t;
-	int n;
-
-	n = 0;
-	t = jtag_AllTaps();
-	while(t){
-		n++;
-		t = t->next_tap;
-	}
-	return n;
+	return jtag_num_taps;
 }
 
 int jtag_NumEnabledTaps(void)
@@ -263,6 +262,17 @@
 	return n;
 }
 
+/// Append a new TAP to the chain of all taps.
+static void jtag_tap_add(struct jtag_tap_s *t)
+{
+	t->abs_chain_position = jtag_num_taps++;
+
+	jtag_tap_t **tap = &jtag_all_taps;
+	while(*tap != NULL)
+		tap = &(*tap)->next_tap;
+	*tap = t;
+}
+
 jtag_tap_t *jtag_TapByString( const char *s )
 {
 	jtag_tap_t *t;
@@ -324,6 +334,12 @@
 	return t;
 }
 
+const char *jtag_tap_name(const jtag_tap_t *tap)
+{
+	return (tap == NULL) ? "(unknown)" : tap->dotted_name;
+}
+
+
 int jtag_register_event_callback(int (*callback)(enum jtag_event event, void *priv), void *priv)
 {
 	jtag_event_callback_t **callbacks_p = &jtag_event_callbacks;
@@ -798,11 +814,6 @@
 	return;
 }
 
-static const char *jtag_tap_name(const jtag_tap_t *tap)
-{
-	return (tap == NULL) ? "(unknown)" : tap->dotted_name;
-}
-
 int jtag_check_value_inner(u8 *captured, u8 *in_check_value, u8 *in_check_mask, int num_bits)
 {
 	int retval = ERROR_OK;
@@ -1262,7 +1273,6 @@
 static int jim_newtap_cmd( Jim_GetOptInfo *goi )
 {
 	jtag_tap_t *pTap;
-	jtag_tap_t **ppTap;
 	jim_wide w;
 	int x;
 	int e;
@@ -1429,22 +1439,13 @@
 
 	jtag_register_event_callback(jtag_reset_callback, pTap );
 
-	ppTap = &(jtag_all_taps);
-	while( (*ppTap) != NULL ){
-		ppTap = &((*ppTap)->next_tap);
-	}
-	*ppTap = pTap;
-	{
-		static int n_taps = 0;
-		pTap->abs_chain_position = n_taps++;
-	}
-	LOG_DEBUG( "Created Tap: %s @ abs position %d, irlen %d, capture: 0x%x mask: 0x%x",
-				(*ppTap)->dotted_name,
-				(*ppTap)->abs_chain_position,
-				(*ppTap)->ir_length,
-				(*ppTap)->ir_capture_value,
-				(*ppTap)->ir_capture_mask );
+	jtag_tap_add(pTap);
 
+	LOG_DEBUG("Created Tap: %s @ abs position %d, "
+		"irlen %d, capture: 0x%x mask: 0x%x", pTap->dotted_name,
+			pTap->abs_chain_position, pTap->ir_length,
+			pTap->ir_capture_value, pTap->ir_capture_mask);
+
 	return ERROR_OK;
 }
 
@@ -1914,7 +1915,7 @@
 {
 	jtag_tap_t *tap;
 
-	tap = jtag_all_taps;
+	tap = jtag_AllTaps();
 	command_print(cmd_ctx, "     TapName            | Enabled |   IdCode      Expected    IrLen IrCap  IrMask Instr     ");
 	command_print(cmd_ctx, "---|--------------------|---------|------------|------------|------|------|------|---------");
 

Modified: trunk/src/jtag/jtag.h
===================================================================
--- trunk/src/jtag/jtag.h	2009-06-04 22:43:34 UTC (rev 2054)
+++ trunk/src/jtag/jtag.h	2009-06-04 23:26:58 UTC (rev 2055)
@@ -176,6 +176,7 @@
 	jtag_tap_t* next_tap;
 };
 extern jtag_tap_t* jtag_AllTaps(void);
+extern const char *jtag_tap_name(const jtag_tap_t *tap);
 extern jtag_tap_t* jtag_TapByPosition(int n);
 extern jtag_tap_t* jtag_TapByString(const char* dotted_name);
 extern jtag_tap_t* jtag_TapByJimObj(Jim_Interp* interp, Jim_Obj* obj);