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

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

Author: zwelch
Date: 2009-06-07 07:23:50 +0200 (Sun, 07 Jun 2009)
New Revision: 2087

Modified:
   trunk/src/jtag/jtag.c
Log:
Factoring of jtag_examine_chain for maintainability:
- Add helper to check for the terminating ID during jtag_examine_chain.


Modified: trunk/src/jtag/jtag.c
===================================================================
--- trunk/src/jtag/jtag.c	2009-06-07 05:23:40 UTC (rev 2086)
+++ trunk/src/jtag/jtag.c	2009-06-07 05:23:50 UTC (rev 2087)
@@ -995,6 +995,11 @@
 		EXTRACT_MFG(idcode), EXTRACT_PART(idcode), EXTRACT_VER(idcode) );
 }
 
+static bool jtag_idcode_is_final(u32 idcode)
+{
+		return idcode == 0x000000FF || idcode == 0xFFFFFFFF;
+}
+
 /**
  * This helper checks that remaining bits in the examined chain data are
  * all as expected, but a single JTAG device requires only 64 bits to be
@@ -1008,7 +1013,7 @@
 	{
 		u32 idcode = buf_get_u32(idcodes, count, 32);
 		// do not trigger the warning if the data looks good
-		if (!triggered && (idcode == 0x000000FF || idcode == 0xFFFFFFFF))
+		if (!triggered && jtag_idcode_is_final(idcode))
 			continue;
 		LOG_WARNING("Unexpected idcode after end of chain: %d 0x%08x",
 				count, idcode);
@@ -1055,7 +1060,7 @@
 			 * as AVR will output all 1's instead of TDI input value at
 			 * end of chain.
 			 */
-			if ((idcode == 0x000000FF)||(idcode == 0xFFFFFFFF))
+			if (jtag_idcode_is_final(idcode))
 			{
 				jtag_examine_chain_end(idcode_buffer,
 						bit_count + 32, JTAG_MAX_CHAIN_SIZE * 32);