sim: bfin: update return value in bfin_reg_{fetch,store}

The latest gdb sim-remote.c really wants a return value from the fetch/store
register functions, so update the Blackfin sim to avoid the warnings/errors.

Signed-off-by: Mike Frysinger <vapier@gentoo.org>

diff --git a/sim/bfin/ChangeLog b/sim/bfin/ChangeLog
index 5d09b5a6056ab174f2e151330c3f610cf9455e1d..49f2d6b312d62a4c9b19c452a0639fa30552c5c2 100644 (file)
--- a/sim/bfin/ChangeLog
+++ b/sim/bfin/ChangeLog
@@ -1,3 +1,9 @@
+2012-07-31  Mike Frysinger  <vapier@gentoo.org>
+
+       * machs.c (bfin_reg_fetch): Change return 0 to return -1, and
+       return -1 to return 4.
+       (bfin_reg_store): Likewise.
+
 2012-06-15  Joel Brobecker  <brobecker@adacore.com>
 
        * config.in, configure: Regenerate.

diff --git a/sim/bfin/machs.c b/sim/bfin/machs.c
index 9d0f26314e40f179442c58385cdd87e9ef05ed57..04e39b597cabe3dbe4b3083e2f37651ae4fdf89c 100644 (file)
--- a/sim/bfin/machs.c
+++ b/sim/bfin/machs.c
@@ -1859,7 +1859,7 @@ bfin_reg_fetch (SIM_CPU *cpu, int rn, unsigned char *buf, int len)
   else if (rn == SIM_BFIN_CC_REGNUM)
     value = CCREG;
   else
-    return 0; // will be an error in gdb
+    return -1;
 
   /* Handle our KSP/USP shadowing in SP.  While in supervisor mode, we
      have the normal SP/USP behavior.  User mode is tricky though.  */
@@ -1874,7 +1874,7 @@ bfin_reg_fetch (SIM_CPU *cpu, int rn, unsigned char *buf, int len)
 
   bfin_store_unsigned_integer (buf, 4, value);
 
-  return -1; // disables size checking in gdb
+  return 4;
 }
 
 static int
@@ -1893,9 +1893,9 @@ bfin_reg_store (SIM_CPU *cpu, int rn, unsigned char *buf, int len)
   else if (rn == SIM_BFIN_CC_REGNUM)
     SET_CCREG (value);
   else
-    return 0; // will be an error in gdb
+    return -1;
 
-  return -1; // disables size checking in gdb
+  return 4;
 }
 
 static sim_cia