]> git.sur5r.net Git - openocd/commitdiff
Duane Ellis: fix warnings
authoroharboe <oharboe@b42882b7-edfa-0310-969c-e2dbd0fdcd60>
Tue, 5 Aug 2008 12:27:18 +0000 (12:27 +0000)
committeroharboe <oharboe@b42882b7-edfa-0310-969c-e2dbd0fdcd60>
Tue, 5 Aug 2008 12:27:18 +0000 (12:27 +0000)
git-svn-id: svn://svn.berlios.de/openocd/trunk@890 b42882b7-edfa-0310-969c-e2dbd0fdcd60

28 files changed:
configure.in
src/flash/flash.c
src/flash/flash.h
src/helper/command.h
src/helper/log.h
src/helper/time_support.h
src/jtag/amt_jtagaccel.c
src/jtag/gw16012.c
src/jtag/jtag.c
src/jtag/jtag.h
src/main.c
src/server/gdb_server.c
src/server/gdb_server.h
src/server/server.c
src/server/server.h
src/target/arm11.c
src/target/arm720t.c
src/target/arm7tdmi.c
src/target/arm920t.c
src/target/arm926ejs.c
src/target/arm9tdmi.c
src/target/cortex_m3.c
src/target/feroceon.c
src/target/mips32_pracc.c
src/target/mips_m4k.c
src/target/target.c
src/target/target.h
src/target/xscale.c

index 8f9b56f39f8bdb255ebe63a2e89706c99520f6b4..2c2f3cde1c00c60af5dce7b97c0b4944fa92da1b 100644 (file)
@@ -27,6 +27,12 @@ is_cygwin=no
 is_mingw=no
 is_win32=no
 
+
+AC_ARG_ENABLE(gccwarnings,
+       AS_HELP_STRING([--enable-gccwarnings], [Enable compiler warnings, default yes]),
+       [gcc_warnings=$enablevalue], [gcc_warnings=yes])
+
+
 AC_ARG_ENABLE(parport,
   AS_HELP_STRING([--enable-parport], [Enable building the pc parallel port driver]), 
   [build_parport=$enableval], [build_parport=no])
@@ -298,11 +304,17 @@ if test $cross_compiling = yes; then
        CFLAGS_FOR_BUILD="-g -O2"
 else
        CC_FOR_BUILD=$CC
-       CFLAGS_FOR_BUILD=$CFLAGS
+       CFLAGS_FOR_BUILD="$CFLAGS $GCC_WARNINGS"
 fi
+
+if test $gcc_warnings = yes; then
+   CFLAGS_FOR_BUILD="$CFLAGS_FOR_BUILD -Wall -Wmissing-prototypes -Wstrict-prototypes"
+fi
+
 AC_MSG_RESULT([$CC_FOR_BUILD])
 AC_SUBST(CC_FOR_BUILD)
 AC_SUBST(CFLAGS_FOR_BUILD)
+AC_MSG_NOTICE([CFLAGS_FOR_BUILD = $CFLAGS_FOR_BUILD])
 
 AC_MSG_CHECKING([for suffix of executable build tools])
 if test $cross_compiling = yes; then
index 95a983b7433bee680ccac47736e6fd4e8792d72e..71d054f5051bde8f4056a8bbc2b932424d66d65e 100644 (file)
@@ -233,7 +233,7 @@ flash_bank_t *get_flash_bank_by_num_noprobe(int num)
        return NULL;
 }
 
-int flash_get_bank_count()
+int flash_get_bank_count(void)
 {
        flash_bank_t *p;
        int i = 0;
index 807914ade81e1005ced809764db50495ed8f62d2..1f5ee9b3892f3d9fda1e9dcfb8d5096ead66f48d 100644 (file)
@@ -80,7 +80,7 @@ extern int flash_init_drivers(struct command_context_s *cmd_ctx);
 extern int flash_erase_address_range(target_t *target, u32 addr, u32 length);
 extern int flash_write(target_t *target, image_t *image, u32 *written, int erase);
 extern void flash_set_dirty(void);
-extern int flash_get_bank_count();
+extern int flash_get_bank_count(void);
 extern int default_flash_blank_check(struct flash_bank_s *bank);
 extern int default_flash_mem_blank_check(struct flash_bank_s *bank);
 
index d784af76e8ed60803b94e6e84fe6ee5e5751e6e8..097a8d37e6ea1ac474b5a2ae7aec867294b99aa1 100644 (file)
@@ -71,7 +71,7 @@ extern int unregister_all_commands(command_context_t *context);
 extern void command_set_output_handler(command_context_t* context, int (*output_handler)(struct command_context_s *context, const char* line), void *priv);
 extern command_context_t* copy_command_context(command_context_t* context);
 extern int command_context_mode(command_context_t *context, enum command_mode mode);
-extern command_context_t* command_init();
+extern command_context_t* command_init(void);
 extern int command_done(command_context_t *context);
 extern void command_print(command_context_t *context, char *format, ...);
 extern void command_print_sameline(command_context_t *context, char *format, ...);
index f57ed08021875b855d6362355aeb9d46cc604eb9..c329fb5881d6a478cef52315a2aa9a504812c3b4 100644 (file)
@@ -59,8 +59,8 @@ __attribute__ ((format (printf, 5, 6)));
 extern int log_register_commands(struct command_context_s *cmd_ctx);
 extern int log_init(struct command_context_s *cmd_ctx);
 extern int set_log_output(struct command_context_s *cmd_ctx, FILE *output);
-extern void keep_alive();
-extern void kept_alive();
+extern void keep_alive(void);
+extern void kept_alive(void);
 
 typedef void (*log_callback_fn)(void *priv, const char *file, int line,
                const char *function, const char *string);
index 49c41c9d03d6e4ffaa3862c16047e4f41fd219d0..0407a1c351d5ae3fd992341aff7f21d04bcd0b2e 100644 (file)
@@ -38,7 +38,7 @@ extern int timeval_subtract(struct timeval *result, struct timeval *x, struct ti
 extern int timeval_add(struct timeval *result, struct timeval *x, struct timeval *y);
 extern int timeval_add_time(struct timeval *result, int sec, int usec);
 /* gettimeofday() timeval in 64 bit ms */
-extern long long timeval_ms();
+extern long long timeval_ms(void);
 
 typedef struct duration_s
 {
index b94b6f7605ef8dc51f0a74e65319bf6afae0b7ab..d29d092f87cd31e71379aca8338760f9cf1f9ae3 100644 (file)
@@ -166,7 +166,7 @@ void amt_jtagaccel_end_state(state)
        }
 }
 
-void amt_wait_scan_busy()
+void amt_wait_scan_busy(void)
 {
        int timeout = 4096;
        u8 ar_status;
index 8c7dc81b97f4be0fb10eb70fcaee712838a10e00..b964a23a4472048a09d4ca5f2fcbefe20dabd1fa 100644 (file)
@@ -194,7 +194,7 @@ int gw16012_speed(int speed)
        return ERROR_OK;
 }
 
-void gw16012_end_state(state)
+void gw16012_end_state(int state)
 {
        if (tap_move_map[state] != -1)
                end_state = state;
@@ -526,7 +526,7 @@ int gw16012_init(void)
                LOG_WARNING("No gw16012 port specified, using default '0x378' (LPT1)");
        }
        
-       LOG_DEBUG("requesting privileges for parallel port 0x%lx...", gw16012_port);
+       LOG_DEBUG("requesting privileges for parallel port 0x%lx...", (long unsigned)(gw16012_port) );
 #if PARPORT_USE_GIVEIO == 1
        if (gw16012_get_giveio_access() != 0)
 #else /* PARPORT_USE_GIVEIO */
index 04b3892d23fc2d296d6f8007241ba29a347b944f..d34cec5a257b3187221eadb7b9870408ff3f2d87 100644 (file)
@@ -401,7 +401,7 @@ void* cmd_queue_alloc(size_t size)
        return t + offset;
 }
 
-void cmd_queue_free()
+void cmd_queue_free(void)
 {
        cmd_queue_page_t *page = cmd_queue_pages;
 
@@ -416,7 +416,7 @@ void cmd_queue_free()
        cmd_queue_pages = NULL;
 }
 
-static void jtag_prelude1()
+static void jtag_prelude1(void)
 {
        if (jtag_trst == 1)
        {
@@ -1433,7 +1433,7 @@ int jtag_examine_chain()
        return ERROR_OK;
 }
 
-int jtag_validate_chain()
+int jtag_validate_chain(void)
 {
        jtag_device_t *device = jtag_devices;
        int total_ir_length = 0;
index f3f354a3852105a92deeeacf8eb1861b3dc42576..347777e496829aaf50a7a039377ae1420161f8f0 100644 (file)
@@ -282,8 +282,8 @@ extern int interface_jtag_add_plain_dr_scan(int num_fields, scan_field_t *fields
 /* run a TAP_TLR reset. End state is TAP_TLR, regardless
  * of start state.
  */
-extern void jtag_add_tlr();
-extern int interface_jtag_add_tlr();
+extern void jtag_add_tlr(void);
+extern int interface_jtag_add_tlr(void);
 /* Do not use jtag_add_pathmove() unless you need to, but do use it
  * if you have to. 
  *
index 616aacd485a5801dc66b891bff78584413b6f134..b7be8970b81c9df7a9c8d8f99754c5b00e9867ea 100644 (file)
 /* implementations of OpenOCD that uses multithreading needs to know when
  * OpenOCD is sleeping. No-op in vanilla OpenOCD
  */
-void openocd_sleep_prelude()
+void openocd_sleep_prelude(void)
 {
 }
 
-void openocd_sleep_postlude()
+void openocd_sleep_postlude(void)
 {
 }
 
index 040c614fdc2eee98ce627665ad820f9d114c91c2..629995388f7119386f1ec1825beb93b9bcfce384 100644 (file)
@@ -2002,7 +2002,7 @@ int gdb_input(connection_t *connection)
        return ERROR_OK;
 }
 
-int gdb_init()
+int gdb_init(void)
 {
        gdb_service_t *gdb_service;
        target_t *target = targets;
index e81ee694dee95fab30d15710a1b95b8a3a2f1dff..943d9d97a98d97489e866128330adf1d4c1b9d6a 100644 (file)
@@ -46,7 +46,7 @@ typedef struct gdb_service_s
        struct target_s *target;
 } gdb_service_t;
 
-extern int gdb_init();
+extern int gdb_init(void);
 extern int gdb_register_commands(command_context_t *command_context);
 
 #define ERROR_GDB_BUFFER_TOO_SMALL (-800)
index 97762838ac753545ff86abfd4d40207e646d136f..aad98747445bdeb68c42d6c84c88655cf54087bc 100644 (file)
@@ -222,7 +222,7 @@ int remove_service(unsigned short port)
        return ERROR_OK;
 }
 
-int remove_services()
+int remove_services(void)
 {
        service_t *c = services;
 
@@ -249,8 +249,8 @@ int remove_services()
        return ERROR_OK;
 }
 
-extern void openocd_sleep_prelude();
-extern void openocd_sleep_postlude();
+extern void openocd_sleep_prelude(void);
+extern void openocd_sleep_postlude(void);
 
 int server_loop(command_context_t *command_context)
 {
@@ -434,7 +434,7 @@ void sig_handler(int sig) {
 }
 #endif
 
-int server_init()
+int server_init(void)
 {
 #ifdef _WIN32
        WORD wVersionRequested;
@@ -460,7 +460,7 @@ int server_init()
        return ERROR_OK;
 }
 
-int server_quit()
+int server_quit(void)
 {
        remove_services();
 
index 95410e80bec59c167b81228f4758fce52e939780..6f0ec3e326b450d559d7b12ac551cab555f5ac23 100644 (file)
@@ -68,8 +68,8 @@ typedef struct service_s
 } service_t;
 
 extern int add_service(char *name, enum connection_type type, unsigned short port, int max_connections, new_connection_handler_t new_connection_handler, input_handler_t input_handler, connection_closed_handler_t connection_closed_handler, void *priv);
-extern int server_init();
-extern int server_quit();
+extern int server_init(void);
+extern int server_quit(void);
 extern int server_loop(command_context_t *command_context);
 extern int server_register_commands(command_context_t *context);
 
index 9e131d8b5a7379a457ce38a83165d777fa5e73cf..7c947fc2e7ce5ce98d90f915d5fe5467905b19c3 100644 (file)
@@ -1716,7 +1716,7 @@ int arm11_handle_mrc_mcr(struct command_context_s *cmd_ctx, char *cmd, char **ar
        if (values[i] > arm11_coproc_instruction_limits[i])
        {
            LOG_ERROR("Parameter %ld out of bounds (%d max). %s",
-               i + 2, arm11_coproc_instruction_limits[i],
+                     (long)(i + 2), arm11_coproc_instruction_limits[i],
                read ? arm11_mrc_syntax : arm11_mcr_syntax);
            return -1;
        }
index cc45c9e61f2e1ba5ae146946588a1892ca121b89..e253e8eea71ef9b56ec874fd5ae7362d0ab04de2 100644 (file)
@@ -43,7 +43,7 @@ int arm720t_handle_mw_phys_command(struct command_context_s *cmd_ctx, char *cmd,
 /* forward declarations */
 int arm720t_target_command(struct command_context_s *cmd_ctx, char *cmd, char **args, int argc, struct target_s *target);
 int arm720t_init_target(struct command_context_s *cmd_ctx, struct target_s *target);
-int arm720t_quit();
+int arm720t_quit(void);
 int arm720t_arch_state(struct target_s *target);
 int arm720t_read_memory(struct target_s *target, u32 address, u32 size, u32 count, u8 *buffer);
 int arm720t_write_memory(struct target_s *target, u32 address, u32 size, u32 count, u8 *buffer);
@@ -423,7 +423,7 @@ int arm720t_init_target(struct command_context_s *cmd_ctx, struct target_s *targ
        
 }
 
-int arm720t_quit()
+int arm720t_quit(void)
 {
        
        return ERROR_OK;
index 41771ab183e2b3a050b373d180237a949c81c000..af6837e4e4cb93768aecdd87bacb1d536fb287f4 100644 (file)
@@ -46,7 +46,7 @@ int arm7tdmi_register_commands(struct command_context_s *cmd_ctx);
 /* forward declarations */
 int arm7tdmi_target_command(struct command_context_s *cmd_ctx, char *cmd, char **args, int argc, struct target_s *target);
 int arm7tdmi_init_target(struct command_context_s *cmd_ctx, struct target_s *target);
-int arm7tdmi_quit();
+int arm7tdmi_quit(void);
 
 /* target function declarations */
 int arm7tdmi_poll(struct target_s *target);
@@ -753,7 +753,7 @@ int arm7tdmi_init_target(struct command_context_s *cmd_ctx, struct target_s *tar
        
 }
 
-int arm7tdmi_quit()
+int arm7tdmi_quit(void)
 {
        
        return ERROR_OK;
index 3a3c2f1f1369ce0387a5fb0a003cda576506e75b..c14f6ef801c9dd02f03acadc248c1118292fe3af 100644 (file)
@@ -48,7 +48,7 @@ int arm920t_handle_read_mmu_command(struct command_context_s *cmd_ctx, char *cmd
 /* forward declarations */
 int arm920t_target_command(struct command_context_s *cmd_ctx, char *cmd, char **args, int argc, struct target_s *target);
 int arm920t_init_target(struct command_context_s *cmd_ctx, struct target_s *target);
-int arm920t_quit();
+int arm920t_quit(void);
 int arm920t_arch_state(struct target_s *target);
 int arm920t_read_memory(struct target_s *target, u32 address, u32 size, u32 count, u8 *buffer);
 int arm920t_write_memory(struct target_s *target, u32 address, u32 size, u32 count, u8 *buffer);
@@ -685,7 +685,7 @@ int arm920t_init_target(struct command_context_s *cmd_ctx, struct target_s *targ
        
 }
 
-int arm920t_quit()
+int arm920t_quit(void)
 {
        
        return ERROR_OK;
index 386d4471a90df203c87e6335bc6e4057a7086fd4..cec3d0f8790e8457ebd25754fa6ac48fc4d4fc00 100644 (file)
@@ -48,7 +48,7 @@ int arm926ejs_handle_read_mmu_command(struct command_context_s *cmd_ctx, char *c
 /* forward declarations */
 int arm926ejs_target_command(struct command_context_s *cmd_ctx, char *cmd, char **args, int argc, struct target_s *target);
 int arm926ejs_init_target(struct command_context_s *cmd_ctx, struct target_s *target);
-int arm926ejs_quit();
+int arm926ejs_quit(void);
 int arm926ejs_arch_state(struct target_s *target);
 int arm926ejs_read_memory(struct target_s *target, u32 address, u32 size, u32 count, u8 *buffer);
 int arm926ejs_write_memory(struct target_s *target, u32 address, u32 size, u32 count, u8 *buffer);
@@ -666,7 +666,7 @@ int arm926ejs_init_target(struct command_context_s *cmd_ctx, struct target_s *ta
        
 }
 
-int arm926ejs_quit()
+int arm926ejs_quit(void)
 {
        
        return ERROR_OK;
index adb21f79b464e8c2605bca6b74e422402727d594..c6a13adb0dec3938a73b45367bcf1608de4554a1 100644 (file)
@@ -48,7 +48,7 @@ int handle_arm9tdmi_catch_vectors_command(struct command_context_s *cmd_ctx, cha
 /* forward declarations */
 int arm9tdmi_target_command(struct command_context_s *cmd_ctx, char *cmd, char **args, int argc, struct target_s *target);
 int arm9tdmi_init_target(struct command_context_s *cmd_ctx, struct target_s *target);
-int arm9tdmi_quit();
+int arm9tdmi_quit(void);
                
 target_type_t arm9tdmi_target =
 {
@@ -896,7 +896,7 @@ int arm9tdmi_init_target(struct command_context_s *cmd_ctx, struct target_s *tar
        
 }
 
-int arm9tdmi_quit()
+int arm9tdmi_quit(void)
 {
        
        return ERROR_OK;
index 255fcd03453c4cf770cf1831b515621062c04ac2..9e0916ff3ad49856e21647a1bae0a1e514ec08ca 100644 (file)
@@ -47,7 +47,7 @@ void cortex_m3_enable_breakpoints(struct target_s *target);
 void cortex_m3_enable_watchpoints(struct target_s *target);
 int cortex_m3_target_command(struct command_context_s *cmd_ctx, char *cmd, char **args, int argc, struct target_s *target);
 int cortex_m3_init_target(struct command_context_s *cmd_ctx, struct target_s *target);
-int cortex_m3_quit();
+int cortex_m3_quit(void);
 int cortex_m3_load_core_reg_u32(target_t *target, enum armv7m_regtype type, u32 num, u32 *value);
 int cortex_m3_store_core_reg_u32(target_t *target, enum armv7m_regtype type, u32 num, u32 value);
 int cortex_m3_target_request_data(target_t *target, u32 size, u8 *buffer);
@@ -1374,7 +1374,7 @@ int cortex_m3_examine(struct target_s *target)
        return ERROR_OK;
 }
 
-int cortex_m3_quit()
+int cortex_m3_quit(void)
 {
        
        return ERROR_OK;
index 8178e556b1a0bb9ad6ea1f73406b113e6435162d..9c774716cf7ebe5cb9007a9d77a53fe1850fc1d5 100644 (file)
@@ -58,7 +58,7 @@ int feroceon_examine(struct target_s *target);
 int feroceon_target_command(struct command_context_s *cmd_ctx, char *cmd, char **args, int argc, struct target_s *target);
 int feroceon_bulk_write_memory(target_t *target, u32 address, u32 count, u8 *buffer);
 int feroceon_init_target(struct command_context_s *cmd_ctx, struct target_s *target);
-int feroceon_quit();
+int feroceon_quit(void);
 
 target_type_t feroceon_target =
 {
@@ -381,6 +381,7 @@ void feroceon_branch_resume_thumb(target_t *target)
        arm_jtag_t *jtag_info = &arm7_9->jtag_info;
        u32 r0 = buf_get_u32(armv4_5->core_cache->reg_list[0].value, 0, 32);
        u32 pc = buf_get_u32(armv4_5->core_cache->reg_list[15].value, 0, 32);
+       (void)(r0); // use R0... 
 
        arm9tdmi_clock_out(jtag_info, ARMV4_5_NOP, 0, NULL, 0);
        arm9tdmi_clock_out(jtag_info, ARMV4_5_NOP, 0, NULL, 0);
@@ -627,7 +628,7 @@ int feroceon_init_target(struct command_context_s *cmd_ctx, struct target_s *tar
        return ERROR_OK;
 }
 
-int feroceon_quit()
+int feroceon_quit(void)
 {
        return ERROR_OK;
 }
index 400227e93cbb91debbdbf0d8e7cee5cc4a1f4dee..d6351f9ec958df159e0b3072036ddaa4ae381939 100644 (file)
@@ -23,6 +23,7 @@
 #include "config.h"
 #endif
 
+#include <string.h>
 #include "log.h"
 #include "mips32.h"
 #include "mips32_pracc.h"
index 2a34bd0b9fce07ed926648bdf70736149db3e099..4cefbf9244e51c8f823ea349f417ba93339588cf 100644 (file)
@@ -46,7 +46,7 @@ int mips_m4k_init_target(struct command_context_s *cmd_ctx, struct target_s *tar
 int mips_m4k_quit();
 int mips_m4k_target_command(struct command_context_s *cmd_ctx, char *cmd, char **args, int argc, struct target_s *target);
 
-int mips_m4k_examine(struct target_s *target);
+int mips_m4k_examine(struct command_context_s *cmd_ctx, struct target_s *target);
 int mips_m4k_assert_reset(target_t *target);
 int mips_m4k_deassert_reset(target_t *target);
 
@@ -569,7 +569,7 @@ int mips_m4k_init_target(struct command_context_s *cmd_ctx, struct target_s *tar
        return ERROR_OK;
 }
 
-int mips_m4k_quit()
+int mips_m4k_quit(void)
 {
        return ERROR_OK;
 }
@@ -618,7 +618,7 @@ int mips_m4k_target_command(struct command_context_s *cmd_ctx, char *cmd, char *
        return ERROR_OK;
 }
 
-int mips_m4k_examine(struct target_s *target)
+int mips_m4k_examine(struct command_context_s *cmd_ctx, struct target_s *target)
 {
        int retval;
        mips32_common_t *mips32 = target->arch_info;
index 0950e436a41fd94c2eb8f84b616e1dbe45fa5dfd..37e1b3695a10d72df52a1283bdd963ebca0aeefa 100644 (file)
@@ -666,15 +666,15 @@ static int target_call_timer_callbacks_check_time(int checktime)
        return ERROR_OK;
 }
 
-int target_call_timer_callbacks()
+int target_call_timer_callbacks(void)
 {
        return target_call_timer_callbacks_check_time(1);
 }
 
 /* invoke periodic callbacks immediately */
-int target_call_timer_callbacks_now()
+int target_call_timer_callbacks_now(void)
 {
-       return target_call_timer_callbacks(0);
+       return target_call_timer_callbacks();
 }
 
 int target_alloc_working_area(struct target_s *target, u32 size, working_area_t **area)
index 8673ddc03c638a8629242d84f3373cedeeca2e99..2551ede7ba07cfaeb896d070495cd8d397a63fcf 100644 (file)
@@ -266,11 +266,11 @@ extern int target_call_event_callbacks(target_t *target, enum target_event event
  */
 extern int target_register_timer_callback(int (*callback)(void *priv), int time_ms, int periodic, void *priv);
 extern int target_unregister_timer_callback(int (*callback)(void *priv), void *priv);
-extern int target_call_timer_callbacks();
+extern int target_call_timer_callbacks(void);
 /* invoke this to ensure that e.g. polling timer callbacks happen before
  * a syncrhonous command completes.
  */
-extern int target_call_timer_callbacks_now();
+extern int target_call_timer_callbacks_now(void);
 
 extern target_t* get_current_target(struct command_context_s *cmd_ctx);
 extern int get_num_by_target(target_t *query_target);
index 0c99abf78ac227551b5f5eaee3ae735bfd31f53a..39378c01a02baba1ab175ceda4fffe28793e4b60 100644 (file)
@@ -55,7 +55,7 @@ int xscale_register_commands(struct command_context_s *cmd_ctx);
 /* forward declarations */
 int xscale_target_command(struct command_context_s *cmd_ctx, char *cmd, char **args, int argc, struct target_s *target);
 int xscale_init_target(struct command_context_s *cmd_ctx, struct target_s *target);
-int xscale_quit();
+int xscale_quit(void);
 
 int xscale_arch_state(struct target_s *target);
 int xscale_poll(target_t *target);
@@ -1579,7 +1579,7 @@ int xscale_assert_reset(target_t *target)
        jtag_execute_queue();
 
        target->state = TARGET_RESET;
-       
+
     if (target->reset_halt)
     {
        int retval;
@@ -2989,7 +2989,7 @@ int xscale_init_target(struct command_context_s *cmd_ctx, struct target_s *targe
        return ERROR_OK;
 }
 
-int xscale_quit()
+int xscale_quit(void)
 {
 
        return ERROR_OK;