From b5539f23a007948b94f7565488b33ca9fd4c6ab0 Mon Sep 17 00:00:00 2001 From: Mike Frysinger Date: Wed, 1 Aug 2012 03:22:32 +0000 Subject: [PATCH] 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 --- sim/bfin/ChangeLog | 6 ++++++ sim/bfin/machs.c | 8 ++++---- 2 files changed, 10 insertions(+), 4 deletions(-) diff --git a/sim/bfin/ChangeLog b/sim/bfin/ChangeLog index 5d09b5a6056..49f2d6b312d 100644 --- a/sim/bfin/ChangeLog +++ b/sim/bfin/ChangeLog @@ -1,3 +1,9 @@ +2012-07-31 Mike Frysinger + + * 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 * config.in, configure: Regenerate. diff --git a/sim/bfin/machs.c b/sim/bfin/machs.c index 9d0f26314e4..04e39b597ca 100644 --- 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 -- 2.30.2