* [PATCH/committed 2/4] sim: m68hc11: tweak printf-style funcs
2021-01-30 18:40 [PATCH/committed 1/4] sim: m68hc11: include stdlib.h for prototypes Mike Frysinger
@ 2021-01-30 18:40 ` Mike Frysinger
2021-01-30 18:40 ` [PATCH/committed 3/4] sim: m68hc11: localize a few functions Mike Frysinger
2021-01-30 18:40 ` [PATCH/committed 4/4] sim: m68hc11: fix printf size warnings Mike Frysinger
2 siblings, 0 replies; 4+ messages in thread
From: Mike Frysinger @ 2021-01-30 18:40 UTC (permalink / raw)
To: gdb-patches
GCC complains that we past non-string literals to a printf style func,
so put a %s in here to keep it quiet.
---
sim/m68hc11/ChangeLog | 4 ++++
sim/m68hc11/interp.c | 4 ++--
2 files changed, 6 insertions(+), 2 deletions(-)
diff --git a/sim/m68hc11/ChangeLog b/sim/m68hc11/ChangeLog
index 90d73df898a7..7c956d1b6278 100644
--- a/sim/m68hc11/ChangeLog
+++ b/sim/m68hc11/ChangeLog
@@ -1,3 +1,7 @@
+2021-01-30 Mike Frysinger <vapier@gentoo.org>
+
+ * interp.c (sim_get_info, sim_board_reset): Pass %s to sim_hw_parse.
+
2021-01-30 Mike Frysinger <vapier@gentoo.org>
* dv-m68hc11.c, m68hc11_sim.c: Include stdlib.h.
diff --git a/sim/m68hc11/interp.c b/sim/m68hc11/interp.c
index 26f2d1527beb..7dc8e25c0462 100644
--- a/sim/m68hc11/interp.c
+++ b/sim/m68hc11/interp.c
@@ -100,7 +100,7 @@ sim_get_info (SIM_DESC sd, char *cmd)
sim_io_eprintf (sd, "Valid devices: cpu timer sio eeprom\n");
return;
}
- hw_dev = sim_hw_parse (sd, dev_list[i].device);
+ hw_dev = sim_hw_parse (sd, "%s", dev_list[i].device);
if (hw_dev == 0)
{
sim_io_eprintf (sd, "Device '%s' not found\n", dev_list[i].device);
@@ -138,7 +138,7 @@ sim_board_reset (SIM_DESC sd)
cpu_type = "/m68hc12";
}
- hw_cpu = sim_hw_parse (sd, cpu_type);
+ hw_cpu = sim_hw_parse (sd, "%s", cpu_type);
if (hw_cpu == 0)
{
sim_io_eprintf (sd, "%s cpu not found in device tree.", cpu_type);
--
2.30.0
^ permalink raw reply [flat|nested] 4+ messages in thread
* [PATCH/committed 3/4] sim: m68hc11: localize a few functions
2021-01-30 18:40 [PATCH/committed 1/4] sim: m68hc11: include stdlib.h for prototypes Mike Frysinger
2021-01-30 18:40 ` [PATCH/committed 2/4] sim: m68hc11: tweak printf-style funcs Mike Frysinger
@ 2021-01-30 18:40 ` Mike Frysinger
2021-01-30 18:40 ` [PATCH/committed 4/4] sim: m68hc11: fix printf size warnings Mike Frysinger
2 siblings, 0 replies; 4+ messages in thread
From: Mike Frysinger @ 2021-01-30 18:40 UTC (permalink / raw)
To: gdb-patches
These are only used in this file and lack prototypes, so gcc
complains about it. Add static everywhere to clean that up.
---
sim/m68hc11/ChangeLog | 6 ++++++
sim/m68hc11/m68hc11_sim.c | 12 ++++++------
2 files changed, 12 insertions(+), 6 deletions(-)
diff --git a/sim/m68hc11/ChangeLog b/sim/m68hc11/ChangeLog
index 7c956d1b6278..e83e11ebb70b 100644
--- a/sim/m68hc11/ChangeLog
+++ b/sim/m68hc11/ChangeLog
@@ -1,3 +1,9 @@
+2021-01-30 Mike Frysinger <vapier@gentoo.org>
+
+ * m68hc11_sim.c (cpu_set_sp): Add static markings.
+ (cpu_get_reg, cpu_get_src_reg, cpu_set_dst_reg,
+ cpu_get_indexed_operand_addr, cpu_get_indexed_operand8): Likewise.
+
2021-01-30 Mike Frysinger <vapier@gentoo.org>
* interp.c (sim_get_info, sim_board_reset): Pass %s to sim_hw_parse.
diff --git a/sim/m68hc11/m68hc11_sim.c b/sim/m68hc11/m68hc11_sim.c
index 8eaa421a68fc..0d7860170bc7 100644
--- a/sim/m68hc11/m68hc11_sim.c
+++ b/sim/m68hc11/m68hc11_sim.c
@@ -115,7 +115,7 @@ cpu_set_sp (sim_cpu *cpu, uint16 val)
cpu->cpu_regs.sp = val;
}
-uint16
+static uint16
cpu_get_reg (sim_cpu *cpu, uint8 reg)
{
switch (reg)
@@ -137,7 +137,7 @@ cpu_get_reg (sim_cpu *cpu, uint8 reg)
}
}
-uint16
+static uint16
cpu_get_src_reg (sim_cpu *cpu, uint8 reg)
{
switch (reg)
@@ -171,7 +171,7 @@ cpu_get_src_reg (sim_cpu *cpu, uint8 reg)
}
}
-void
+static void
cpu_set_dst_reg (sim_cpu *cpu, uint8 reg, uint16 val)
{
switch (reg)
@@ -213,7 +213,7 @@ cpu_set_dst_reg (sim_cpu *cpu, uint8 reg, uint16 val)
}
}
-void
+static void
cpu_set_reg (sim_cpu *cpu, uint8 reg, uint16 val)
{
switch (reg)
@@ -346,7 +346,7 @@ cpu_get_indexed_operand_addr (sim_cpu *cpu, int restricted)
return addr;
}
-uint8
+static uint8
cpu_get_indexed_operand8 (sim_cpu *cpu, int restricted)
{
uint16 addr;
@@ -355,7 +355,7 @@ cpu_get_indexed_operand8 (sim_cpu *cpu, int restricted)
return memory_read8 (cpu, addr);
}
-uint16
+static uint16
cpu_get_indexed_operand16 (sim_cpu *cpu, int restricted)
{
uint16 addr;
--
2.30.0
^ permalink raw reply [flat|nested] 4+ messages in thread
* [PATCH/committed 4/4] sim: m68hc11: fix printf size warnings
2021-01-30 18:40 [PATCH/committed 1/4] sim: m68hc11: include stdlib.h for prototypes Mike Frysinger
2021-01-30 18:40 ` [PATCH/committed 2/4] sim: m68hc11: tweak printf-style funcs Mike Frysinger
2021-01-30 18:40 ` [PATCH/committed 3/4] sim: m68hc11: localize a few functions Mike Frysinger
@ 2021-01-30 18:40 ` Mike Frysinger
2 siblings, 0 replies; 4+ messages in thread
From: Mike Frysinger @ 2021-01-30 18:40 UTC (permalink / raw)
To: gdb-patches
GCC complains %llu is wrong for signed64, so switch to PRIi64.
---
sim/m68hc11/ChangeLog | 4 ++++
sim/m68hc11/dv-m68hc11tim.c | 2 +-
2 files changed, 5 insertions(+), 1 deletion(-)
diff --git a/sim/m68hc11/ChangeLog b/sim/m68hc11/ChangeLog
index e83e11ebb70b..0063e8e8b464 100644
--- a/sim/m68hc11/ChangeLog
+++ b/sim/m68hc11/ChangeLog
@@ -1,3 +1,7 @@
+2021-01-30 Mike Frysinger <vapier@gentoo.org>
+
+ * dv-m68hc11tim.c (cycle_to_string): Change llu to PRIi64.
+
2021-01-30 Mike Frysinger <vapier@gentoo.org>
* m68hc11_sim.c (cpu_set_sp): Add static markings.
diff --git a/sim/m68hc11/dv-m68hc11tim.c b/sim/m68hc11/dv-m68hc11tim.c
index b0d032e19ae5..cd8274ffc34f 100644
--- a/sim/m68hc11/dv-m68hc11tim.c
+++ b/sim/m68hc11/dv-m68hc11tim.c
@@ -503,7 +503,7 @@ cycle_to_string (sim_cpu *cpu, signed64 t, int flags)
if (t < LONG_MAX)
sprintf (buf, "%9lu%s%s", (unsigned long) t, cycle_buf, time_buf);
else
- sprintf (buf, "%llu%s%s", t, cycle_buf, time_buf);
+ sprintf (buf, "%" PRIi64 "%s%s", t, cycle_buf, time_buf);
return buf;
}
--
2.30.0
^ permalink raw reply [flat|nested] 4+ messages in thread