# HG changeset patch # User Michael Spacefalcon # Date 1409552618 0 # Node ID 42c91c51ca7f02f98eed7202b12b8b89ce1eac3b # Parent d1df7544f102ea94ca9e5e249f5004f1e428b704 rvinterf: display GPF packets as "GPF" rather than "G23" diff -r d1df7544f102 -r 42c91c51ca7f rvinterf/libg23/README --- 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. diff -r d1df7544f102 -r 42c91c51ca7f rvinterf/libg23/fmtdispatch.c --- 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]); diff -r d1df7544f102 -r 42c91c51ca7f rvinterf/libg23/fmtfunc.c --- 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);