Mon Aug 11 16:22:36 1997 Geoffrey Noer <noer@cygnus.com>

        * ocd.c (ocd_insert_breakpoint, ocd_remove_breakpoint): macro
        BDM_BREAKPOINT already has braces around it, remove erroneous ones.
        * ser-ocd.c (ocd_write): conditionalize on _WIN32 instead of
        __CYGWIN32__
        * config/powerpc/tm-ppc-eabi.h: remove BDM_NUM_REGS, BDM_REGMAP
        * ppc-bdm.c: move BDM_NUM_REGS, BDM_REGMAP here from tm.h file,
        fill in doc fields of bdm_ppc_ops.
        (bdm_ppc_fetch_registers): don't ask for invalid registers such
        as the MQ or floating point regs not present on ppc 8xx boards
        (bdm_ppc_store_registers): don't write those same invalid registers
        * config/i386/cygwin32.mh: stop including ocd.o ser-ocd.o
        * config/powerpc/ppc-eabi.mt:
        * config/powerpc/ppcle-eabi.mt:
        * config/powerpc/ppc-sim.mt:
        * config/powerpc/ppcle-sim.mt: include ser-ocd.o

1 files changed, 95 insertions, 9 deletions

diff --git a/gdb/ppc-bdm.c b/gdb/ppc-bdm.c
index dcc2690933..5ca012ffad 100644
--- a/gdb/ppc-bdm.c
+++ b/gdb/ppc-bdm.c
@@ -1,5 +1,5 @@
 /* Remote target communications for the Macraigor Systems BDM Wiggler
-   talking to a Motorola PPC 505.
+   talking to a Motorola PPC 8xx ADS board
    Copyright 1996, 1997 Free Software Foundation, Inc.
 
 This file is part of GDB.
@@ -47,11 +47,34 @@ static void bdm_ppc_store_registers PARAMS ((int regno));
 
 extern struct target_ops bdm_ppc_ops;	/* Forward decl */
 
-/* Open a connection to a remote debugger.
-   NAME is the filename used for communication.  */
+/*#define BDM_NUM_REGS 71*/
+#define BDM_NUM_REGS 24
+
+#define BDM_REGMAP \
+	2048, 2049, 2050, 2051, 2052, 2053, 2054, 2055, /* r0-r7 */ \
+	2056, 2057, 2058, 2059, 2060, 2061, 2062, 2063, /* r8-r15 */ \
+	2064, 2065, 2066, 2067, 2068, 2069, 2070, 2071, /* r16-r23 */ \
+	2072, 2073, 2074, 2075, 2076, 2077, 2078, 2079, /* r24-r31 */ \
+\
+	2080, 2082, 2084, 2086, 2088, 2090, 2092, 2094, /* fp0->fp8 */ \
+	2096, 2098, 2100, 2102, 2104, 2106, 2108, 2110, /* fp0->fp8 */ \
+	2112, 2114, 2116, 2118, 2120, 2122, 2124, 2126, /* fp0->fp8 */ \
+	2128, 2130, 2132, 2134, 2136, 2138, 2140, 2142, /* fp0->fp8 */ \
+\
+	26,        /* pc (SRR0 (SPR 26)) */ \
+	2146,      /* ps (MSR) */ \
+	2144,      /* cnd (CR) */ \
+	8,         /* lr (SPR 8) */ \
+	9,         /* cnt (CTR (SPR 9)) */ \
+	1,         /* xer (SPR 1) */ \
+	0,         /* mq (SPR 0) */
 
+
 char nowatchdog[4] = {0xff,0xff,0xff,0x88};
 
+/* Open a connection to a remote debugger.
+   NAME is the filename used for communication.  */
+
 static void
 bdm_ppc_open (name, from_tty)
      char *name;
@@ -107,6 +130,7 @@ bdm_ppc_wait (pid, target_status)
 static int bdm_regmap[] = {BDM_REGMAP};
 
 /* Read the remote registers into regs.
+   Fetch register REGNO, or all registers if REGNO == -1
 
    The Wiggler uses the following codes to access the registers:
 
@@ -118,7 +142,7 @@ static int bdm_regmap[] = {BDM_REGMAP};
 	26 - SPR 26 - SRR0 - pc
    1024 -> 2047		DCR 0 -> DCR 1023 (IBM PPC 4xx only)
    2048 -> 2079		R0 -> R31
-   2080 -> 2143		FP0 -> FP31 (64 bit regs)
+   2080 -> 2143		FP0 -> FP31 (64 bit regs) (IBM PPC 5xx only)
    2144			CR (known as cnd in GDB)
    2145			FPCSR
    2146			MSR (known as ps in GDB)
@@ -129,10 +153,20 @@ bdm_ppc_fetch_registers (regno)
      int regno;
 {
   int i;
-  unsigned char *regs;
+  unsigned char *regs, *beginregs, *endregs, *almostregs;
+  unsigned char midregs[32];
+  unsigned char mqreg[1];
   int first_regno, last_regno;
   int first_bdm_regno, last_bdm_regno;
-  int reglen;
+  int reglen, beginreglen, endreglen;
+
+#if 1
+  for (i = 0; i < (FPLAST_REGNUM - FP0_REGNUM + 1); i++)
+    {
+      midregs[i] = -1;
+    }
+  mqreg[0] = -1;
+#endif
 
   if (regno == -1)
     {
@@ -157,7 +191,44 @@ bdm_ppc_fetch_registers (regno)
       return;			/* Unsupported register */
     }
 
+#if 1
+  /* Can't ask for floating point regs on ppc 8xx, also need to
+     avoid asking for the mq register. */
+  if (first_regno == last_regno)	/* only want one reg */
+    {
+/*      printf("Asking for register %d\n", first_regno); */
+
+      /* if asking for an invalid register */
+      if ((first_regno == MQ_REGNUM) ||
+          ((first_regno >= FP0_REGNUM) && (first_regno <= FPLAST_REGNUM)))
+	{
+/*          printf("invalid reg request!\n"); */
+          supply_register (first_regno, NULL);
+          return;                   /* Unsupported register */
+        }
+      else
+	{
+          regs = ocd_read_bdm_registers (first_bdm_regno,
+						last_bdm_regno, &reglen);
+        }
+    }
+  else		/* want all regs */
+    {
+/*      printf("Asking for registers %d to %d\n", first_regno, last_regno); */
+      beginregs = ocd_read_bdm_registers (first_bdm_regno,
+					FP0_REGNUM - 1, &beginreglen);
+      endregs = (strcat (midregs, 
+				ocd_read_bdm_registers (FPLAST_REGNUM + 1,
+					last_bdm_regno - 1, &endreglen)));
+      almostregs = (strcat (beginregs, endregs));
+      regs = (strcat (almostregs, mqreg));
+      reglen = beginreglen + 32 + endreglen + 1;
+    }
+
+#endif
+#if 0
   regs = ocd_read_bdm_registers (first_bdm_regno, last_bdm_regno, &reglen);
+#endif
 
   for (i = first_regno; i <= last_regno; i++)
     {
@@ -215,7 +286,19 @@ bdm_ppc_store_registers (regno)
 
       bdm_regno = bdm_regmap [i];
 
-      ocd_write_bdm_registers (bdm_regno, registers + REGISTER_BYTE (i), 4);
+      /* only attempt to write if it's a valid ppc 8xx register */
+      /* (need to avoid FP regs and MQ reg) */
+      if ((i != MQ_REGNUM) && ((i < FP0_REGNUM) || (i > FPLAST_REGNUM)))
+	{
+/*          printf("write valid reg %d\n", bdm_regno); */
+          ocd_write_bdm_registers (bdm_regno, registers + REGISTER_BYTE (i), 4);
+        }
+/*
+      else if (i == MQ_REGNUM)
+        printf("don't write invalid reg %d (MQ_REGNUM)\n", bdm_regno);
+      else
+        printf("don't write invalid reg %d\n", bdm_regno);
+*/
     }
 }
 
@@ -223,8 +306,11 @@ bdm_ppc_store_registers (regno)
 
 struct target_ops bdm_ppc_ops = {
   "ocd",			/* to_shortname */
-  "",				/* to_longname */
-  "",				/* to_doc */
+  "Remote target with On-Chip Debugging",	/* to_longname */
+  "Use a remote target with On-Chip Debugging.  To use a target box,\n\
+specify the serial device it is connected to (e.g. /dev/ttya).  To use\n\
+a wiggler, specify wiggler and then the port it is connected to\n\
+(e.g. wiggler lpt1).",	/* to_doc */
   bdm_ppc_open,			/* to_open */
   ocd_close,		/* to_close */
   NULL,				/* to_attach */