changeset 627:42c91c51ca7f

rvinterf: display GPF packets as "GPF" rather than "G23"
author Michael Spacefalcon <msokolov@ivan.Harhan.ORG>
date Mon, 01 Sep 2014 06:23:38 +0000
parents d1df7544f102
children c2f2f7d78451
files rvinterf/libg23/README rvinterf/libg23/fmtdispatch.c rvinterf/libg23/fmtfunc.c
diffstat 3 files changed, 13 insertions(+), 5 deletions(-) [+]
line wrap: on
line diff
--- a/rvinterf/libg23/README	Sun Aug 31 05:04:56 2014 +0000
+++ b/rvinterf/libg23/README	Mon Sep 01 06:23:38 2014 +0000
@@ -1,3 +1,11 @@
 The library built in this directory is a host side library, not for the target.
-This library implements some functions for handling packet exchanges with G23,
+This library implements some functions for handling packet exchanges with GPF,
 and it will be linked by some of the programs in the rvinterf suite.
+
+It needs to be noted that the RVTMUX channel belonging to GPF was named
+RVT_L23_HEADER by TI, and as a result, I thought that these packets related
+specifically to the higher layers of the protocol stack.  But now we know that
+hierarchically speaking, GPF sits *below* L1, not above, and GPF packets should
+not be automatically associated with G23.  This realization was made fairly
+late, thus "g23" appears in a bunch of function names, and in the name of this
+library.
--- a/rvinterf/libg23/fmtdispatch.c	Sun Aug 31 05:04:56 2014 +0000
+++ b/rvinterf/libg23/fmtdispatch.c	Mon Sep 01 06:23:38 2014 +0000
@@ -62,7 +62,7 @@
 	char *dp;
 
 	dp = outbuf;
-	strcpy(dp, "G23 UNK:");
+	strcpy(dp, "GPF UNK:");
 	dp += 8;
 	for (i = 1; i < rxpkt_len; i++) {
 		sprintf(dp, " %02X", rxpkt[i]);
--- a/rvinterf/libg23/fmtfunc.c	Sun Aug 31 05:04:56 2014 +0000
+++ b/rvinterf/libg23/fmtfunc.c	Mon Sep 01 06:23:38 2014 +0000
@@ -1,9 +1,9 @@
 /*
  * This libg23 module exports functions for formatting 3 different kinds
- * of G23 packets into human-readable form: traces, system primitives
+ * of GPF packets into human-readable form: traces, system primitives
  * and protocol stack primitives.
  *
- * G23 packets passed to these functions to decoding MUST have already
+ * GPF packets passed to these functions for decoding MUST have already
  * been verified to be well-formed for their respective type.
  */
 
@@ -52,7 +52,7 @@
 	u_char *rxpkt;
 	char **outp, *typestr;
 {
-	sprintf(*outp, "G23 %s id=%02X ts=%02X%02X%02X%02X ", typestr,
+	sprintf(*outp, "GPF %s id=%02X ts=%02X%02X%02X%02X ", typestr,
 		rxpkt[1], rxpkt[7], rxpkt[6], rxpkt[5], rxpkt[4]);
 	*outp = index(*outp, '\0');
 	print_entity_name(rxpkt + 8, outp);