From 402cf0534665db83e19c00f130184e94552a75fb Mon Sep 17 00:00:00 2001 From: Mike Frysinger Date: Mon, 4 Jan 2016 00:25:46 -0500 Subject: [PATCH] sim: msp430: drop duplicate sim_load_file call There's no need, or desire, to call sim_load_file from sim_open. The higher levels (gdb/run) take care of calling sim_load for us already. --- sim/msp430/ChangeLog | 7 +++++++ sim/msp430/msp430-sim.c | 33 +++++++-------------------------- 2 files changed, 14 insertions(+), 26 deletions(-) diff --git a/sim/msp430/ChangeLog b/sim/msp430/ChangeLog index 4f4b938fb6..02892801e8 100644 --- a/sim/msp430/ChangeLog +++ b/sim/msp430/ChangeLog @@ -1,3 +1,10 @@ +2016-01-05 Mike Frysinger + + * msp430-sim.c (loader_write_mem): Delete. + (lookup_symbol): Return -1 when abfd is NULL. + (sim_open): Delete prog_bfd variable. Delete call to sim_load_file. + Delete prog_bfd check. + 2016-01-05 Mike Frysinger * Makefile.in (SIM_OBJS): Delete trace.o. diff --git a/sim/msp430/msp430-sim.c b/sim/msp430/msp430-sim.c index a074587625..74a600d2d8 100644 --- a/sim/msp430/msp430-sim.c +++ b/sim/msp430/msp430-sim.c @@ -32,16 +32,6 @@ #include "sim-syscall.h" #include "targ-vals.h" -static int -loader_write_mem (SIM_DESC sd, - SIM_ADDR taddr, - const unsigned char *buf, - int bytes) -{ - SIM_CPU *cpu = MSP430_CPU (sd); - return sim_core_write_buffer (sd, cpu, write_map, buf, taddr, bytes); -} - static sim_cia msp430_pc_fetch (SIM_CPU *cpu) { @@ -62,6 +52,9 @@ lookup_symbol (SIM_DESC sd, const char *name) long number_of_symbols = STATE_NUM_SYMBOLS (sd); long i; + if (abfd == NULL) + return -1; + if (symbol_table == NULL) { long storage_needed; @@ -148,7 +141,6 @@ sim_open (SIM_OPEN_KIND kind, { SIM_DESC sd = sim_state_alloc (kind, callback); char c; - struct bfd *prog_bfd; /* Initialise the simulator. */ @@ -198,14 +190,6 @@ sim_open (SIM_OPEN_KIND kind, return 0; } - prog_bfd = sim_load_file (sd, argv[0], callback, - "the program", - STATE_PROG_BFD (sd), - 0 /* verbose */, - 1 /* use LMA instead of VMA */, - loader_write_mem); - /* Allow prog_bfd to be NULL - this is needed by the GDB testsuite. */ - /* Establish any remaining configuration options. */ if (sim_config (sd) != SIM_RC_OK) { @@ -223,13 +207,10 @@ sim_open (SIM_OPEN_KIND kind, assert (MAX_NR_PROCESSORS == 1); msp430_initialize_cpu (sd, MSP430_CPU (sd)); - if (prog_bfd != NULL) - { - MSP430_CPU (sd)->state.cio_breakpoint = lookup_symbol (sd, "C$$IO$$"); - MSP430_CPU (sd)->state.cio_buffer = lookup_symbol (sd, "__CIOBUF__"); - if (MSP430_CPU (sd)->state.cio_buffer == -1) - MSP430_CPU (sd)->state.cio_buffer = lookup_symbol (sd, "_CIOBUF_"); - } + MSP430_CPU (sd)->state.cio_breakpoint = lookup_symbol (sd, "C$$IO$$"); + MSP430_CPU (sd)->state.cio_buffer = lookup_symbol (sd, "__CIOBUF__"); + if (MSP430_CPU (sd)->state.cio_buffer == -1) + MSP430_CPU (sd)->state.cio_buffer = lookup_symbol (sd, "_CIOBUF_"); return sd; } -- 2.34.1