[binutils-gdb] sim: dv-sockser: localize init callback
Michael Frysinger
vapier@sourceware.org
Sun May 2 02:47:45 GMT 2021
https://sourceware.org/git/gitweb.cgi?p=binutils-gdb.git;h=328e805b5a7d4f27552a1062e25a89477bd89009
commit 328e805b5a7d4f27552a1062e25a89477bd89009
Author: Mike Frysinger <vapier@gentoo.org>
Date: Sat Jan 2 10:53:03 2016 -0500
sim: dv-sockser: localize init callback
Now that we don't need to hardcode the module init list in a single
place, move the dv-sockser logic to the place to the one file.
Diff:
---
sim/common/ChangeLog | 8 ++++++++
sim/common/dv-sockser.c | 5 ++++-
sim/common/dv-sockser.h | 2 --
sim/common/sim-module.c | 9 ---------
4 files changed, 12 insertions(+), 12 deletions(-)
diff --git a/sim/common/ChangeLog b/sim/common/ChangeLog
index d00696354dd..55d684bfeab 100644
--- a/sim/common/ChangeLog
+++ b/sim/common/ChangeLog
@@ -1,3 +1,11 @@
+2021-05-01 Mike Frysinger <vapier@gentoo.org>
+
+ * dv-sockser.c (dv_sockser_install): Rename to ...
+ (sim_install_dv_sockser): ... this.
+ * dv-sockser.h (dv_sockser_install): Delete.
+ * sim-module.c: Delete dv-sockser.h include.
+ (early_modules): Delete dv_sockser_install.
+
2021-05-01 Mike Frysinger <vapier@gentoo.org>
* Make-common.in (LIB_OBJS): Add modules.o.
diff --git a/sim/common/dv-sockser.c b/sim/common/dv-sockser.c
index fc552fa0acf..73c473b15c0 100644
--- a/sim/common/dv-sockser.c
+++ b/sim/common/dv-sockser.c
@@ -222,8 +222,11 @@ dv_sockser_uninstall (SIM_DESC sd)
}
}
+/* Provide a prototype to silence -Wmissing-prototypes. */
+extern MODULE_INIT_FN sim_install_dv_sockser;
+
SIM_RC
-dv_sockser_install (SIM_DESC sd)
+sim_install_dv_sockser (SIM_DESC sd)
{
SIM_ASSERT (STATE_MAGIC (sd) == SIM_MAGIC_NUMBER);
if (sim_add_option_table (sd, NULL, sockser_options) != SIM_RC_OK)
diff --git a/sim/common/dv-sockser.h b/sim/common/dv-sockser.h
index c83ed5a9aa6..36ec0a970b1 100644
--- a/sim/common/dv-sockser.h
+++ b/sim/common/dv-sockser.h
@@ -34,8 +34,6 @@ int dv_sockser_write (SIM_DESC, unsigned char);
int dv_sockser_write_buffer (SIM_DESC, const unsigned char *, unsigned);
int dv_sockser_read (SIM_DESC);
-SIM_RC dv_sockser_install (SIM_DESC);
-
#else
/* If dv-sockser isn't available, provide stub functions. */
diff --git a/sim/common/sim-module.c b/sim/common/sim-module.c
index a776a082176..2557697fff2 100644
--- a/sim/common/sim-module.c
+++ b/sim/common/sim-module.c
@@ -29,11 +29,6 @@ along with this program. If not, see <http://www.gnu.org/licenses/>. */
#include "sim-hw.h"
#endif
-#ifdef HAVE_DV_SOCKSER
-/* TODO: Shouldn't have device models here. */
-#include "dv-sockser.h"
-#endif
-
#include "libiberty.h"
#include <stdlib.h>
@@ -60,10 +55,6 @@ static MODULE_INSTALL_FN * const early_modules[] = {
#if WITH_HW
sim_hw_install,
#endif
-#ifdef HAVE_DV_SOCKSER
- /* TODO: Shouldn't have device models here. */
- dv_sockser_install,
-#endif
};
static int early_modules_len = ARRAY_SIZE (early_modules);
More information about the Gdb-cvs
mailing list