aboutsummaryrefslogtreecommitdiff
path: root/sim
diff options
context:
space:
mode:
authorMike Frysinger <vapier@gentoo.org>2016-08-13 01:41:57 -0700
committerMike Frysinger <vapier@gentoo.org>2016-08-13 22:46:27 -0700
commit527aaa4a3143afedd8dd85aa70862328a9dbe627 (patch)
tree5110c46b53b2d0cfb648a3fab594c5b89aa5b2df /sim
parent4c171e25a8c83fc26b78430fa632fa9e64f61050 (diff)
downloadgdb-527aaa4a3143afedd8dd85aa70862328a9dbe627.zip
gdb-527aaa4a3143afedd8dd85aa70862328a9dbe627.tar.gz
gdb-527aaa4a3143afedd8dd85aa70862328a9dbe627.tar.bz2
sim: m68hc11: fix up various prototype related warnings
A few funcs are only used locally, so mark them static to avoid warnings due to -Wmissing-prototypes. Some funcs cast the return value wrong, so drop them (and let void * just work by default). Update some prototypes to be new style.
Diffstat (limited to 'sim')
-rw-r--r--sim/m68hc11/ChangeLog17
-rw-r--r--sim/m68hc11/dv-m68hc11eepr.c2
-rw-r--r--sim/m68hc11/dv-m68hc11sio.c4
-rw-r--r--sim/m68hc11/dv-m68hc11spi.c2
-rw-r--r--sim/m68hc11/dv-m68hc11tim.c2
-rw-r--r--sim/m68hc11/dv-nvram.c2
-rw-r--r--sim/m68hc11/emulos.c6
-rw-r--r--sim/m68hc11/gencode.c6
8 files changed, 29 insertions, 12 deletions
diff --git a/sim/m68hc11/ChangeLog b/sim/m68hc11/ChangeLog
index b8f9ffc..7fcb535 100644
--- a/sim/m68hc11/ChangeLog
+++ b/sim/m68hc11/ChangeLog
@@ -1,3 +1,20 @@
+2016-08-13 Mike Frysinger <vapier@gentoo.org>
+
+ * dv-m68hc11eepr.c (attach_m68hc11eepr_regs): Drop cast with
+ return value of hw_malloc.
+ * dv-m68hc11sio.c (m68hc11sio_rx_poll): Mark static.
+ (m68hc11sio_tx_poll): Likewise.
+ * dv-m68hc11spi.c (m68hc11spi_clock): Likewise.
+ * dv-m68hc11tim.c (m68hc11tim_timer_event): Likewise.
+ * dv-nvram.c (attach_nvram_regs): Drop cast with
+ return value of hw_malloc.
+ * emulos.c (emul_bench): Mark static.
+ (emul_write): Likewise.
+ (emul_exit): Likewise.
+ * gencode.c: Include stdio.h.
+ (gen_function_entry): Unify two print lines.
+ (cmp_opcode): Mark args const.
+
2016-01-10 Mike Frysinger <vapier@gentoo.org>
* config.in, configure: Regenerate.
diff --git a/sim/m68hc11/dv-m68hc11eepr.c b/sim/m68hc11/dv-m68hc11eepr.c
index 6fee524..6c03a99 100644
--- a/sim/m68hc11/dv-m68hc11eepr.c
+++ b/sim/m68hc11/dv-m68hc11eepr.c
@@ -217,7 +217,7 @@ attach_m68hc11eepr_regs (struct hw *me,
controller->attach_space = attach_space;
controller->base_address = attach_address;
- controller->eeprom = (char*) hw_malloc (me, attach_size + 1);
+ controller->eeprom = hw_malloc (me, attach_size + 1);
controller->eeprom_min_cycles = 10000;
controller->size = attach_size + 1;
controller->mapped = 0;
diff --git a/sim/m68hc11/dv-m68hc11sio.c b/sim/m68hc11/dv-m68hc11sio.c
index dd05c28..fe850e4 100644
--- a/sim/m68hc11/dv-m68hc11sio.c
+++ b/sim/m68hc11/dv-m68hc11sio.c
@@ -240,7 +240,7 @@ m68hc11sio_port_event (struct hw *me,
}
-void
+static void
m68hc11sio_rx_poll (struct hw *me, void *data)
{
SIM_DESC sd;
@@ -313,7 +313,7 @@ m68hc11sio_rx_poll (struct hw *me, void *data)
}
-void
+static void
m68hc11sio_tx_poll (struct hw *me, void *data)
{
SIM_DESC sd;
diff --git a/sim/m68hc11/dv-m68hc11spi.c b/sim/m68hc11/dv-m68hc11spi.c
index 4bcfb95..a4e9148 100644
--- a/sim/m68hc11/dv-m68hc11spi.c
+++ b/sim/m68hc11/dv-m68hc11spi.c
@@ -234,7 +234,7 @@ set_bit_port (struct hw *me, sim_cpu *cpu, int port, int mask, int value)
#define SPI_START_BIT 1
#define SPI_MIDDLE_BIT 2
-void
+static void
m68hc11spi_clock (struct hw *me, void *data)
{
SIM_DESC sd;
diff --git a/sim/m68hc11/dv-m68hc11tim.c b/sim/m68hc11/dv-m68hc11tim.c
index 317d464..c4e6c9d 100644
--- a/sim/m68hc11/dv-m68hc11tim.c
+++ b/sim/m68hc11/dv-m68hc11tim.c
@@ -236,7 +236,7 @@ enum event_type
COMPARE_EVENT
};
-void
+static void
m68hc11tim_timer_event (struct hw *me, void *data)
{
SIM_DESC sd;
diff --git a/sim/m68hc11/dv-nvram.c b/sim/m68hc11/dv-nvram.c
index 6a27a32..6a72c96 100644
--- a/sim/m68hc11/dv-nvram.c
+++ b/sim/m68hc11/dv-nvram.c
@@ -186,7 +186,7 @@ attach_nvram_regs (struct hw *me, struct nvram *controller)
case NVRAM_SAVE_MODIFIED:
case NVRAM_SAVE_ALL:
- controller->data = (char*) hw_malloc (me, attach_size);
+ controller->data = hw_malloc (me, attach_size);
if (controller->data == 0)
hw_abort (me, "Not enough memory, try to use the mode 'map'");
diff --git a/sim/m68hc11/emulos.c b/sim/m68hc11/emulos.c
index 77607c7..e11c618 100644
--- a/sim/m68hc11/emulos.c
+++ b/sim/m68hc11/emulos.c
@@ -33,7 +33,7 @@ static int bench_mode = -1;
static struct timeval bench_start;
static struct timeval bench_stop;
-void
+static void
emul_bench (struct _sim_cpu* cpu)
{
int op;
@@ -90,7 +90,7 @@ emul_bench (struct _sim_cpu* cpu)
}
#endif
-void
+static void
emul_write(struct _sim_cpu* state)
{
int addr = cpu_get_x (state) & 0x0FFFF;
@@ -115,7 +115,7 @@ emul_write(struct _sim_cpu* state)
But doing an exit () on a real target is really a non-sense.
exit () is important for the validation of GCC. The exit status
is passed in 'D' register. */
-void
+static void
emul_exit (sim_cpu *cpu)
{
sim_engine_halt (CPU_STATE (cpu), cpu,
diff --git a/sim/m68hc11/gencode.c b/sim/m68hc11/gencode.c
index 1b1844d..a3032f8 100644
--- a/sim/m68hc11/gencode.c
+++ b/sim/m68hc11/gencode.c
@@ -18,6 +18,7 @@ You should have received a copy of the GNU General Public License
along with this program. If not, see <http://www.gnu.org/licenses/>. */
#include <stdio.h>
+#include <stdlib.h>
#include <string.h>
#include <stdarg.h>
#include <errno.h>
@@ -1987,8 +1988,7 @@ void
gen_function_entry (FILE *fp, const char *name, int locals)
{
/* Generate interpretor entry point. */
- print (fp, 0, "%s (proc)\n", name);
- print (fp, indent_level, "struct _sim_cpu* proc;");
+ print (fp, 0, "%s (sim_cpu *proc)\n", name);
print (fp, indent_level, "{\n");
/* Interpretor local variables. */
@@ -2007,7 +2007,7 @@ gen_function_close (FILE *fp)
}
int
-cmp_opcode (void* e1, void* e2)
+cmp_opcode (const void *e1, const void *e2)
{
struct m6811_opcode_def* op1 = (struct m6811_opcode_def*) e1;
struct m6811_opcode_def* op2 = (struct m6811_opcode_def*) e2;