From 3474aa223a712ac848a8a8e58633106477db0641 Mon Sep 17 00:00:00 2001 From: Antonio Borneo Date: Mon, 6 May 2019 12:31:57 +0200 Subject: [PATCH] coding style: prototype of functions with no parameters Functions that have no parameters should use "void" as parameter in the function declaration. Issue identified and fixed by script checkpatch from Linux kernel v5.1 using the command find src/ -type f -exec ./tools/scripts/checkpatch.pl \ -q --types FUNCTION_WITHOUT_ARGS --fix-inplace -f {} \; Change-Id: If104ac75b44e939ec86155ff7b5720f2e33c6b39 Signed-off-by: Antonio Borneo Reviewed-on: http://openocd.zylin.com/5621 Reviewed-by: Andreas Fritiofson Tested-by: jenkins --- src/helper/log.c | 4 ++-- src/jtag/core.c | 4 ++-- src/jtag/drivers/arm-jtag-ew.c | 2 +- src/jtag/interface.c | 4 ++-- src/jtag/minidummy/minidummy.c | 2 +- src/jtag/zy1000/zy1000.c | 2 +- 6 files changed, 9 insertions(+), 9 deletions(-) diff --git a/src/helper/log.c b/src/helper/log.c index ae26df5a1c..380f548f42 100644 --- a/src/helper/log.c +++ b/src/helper/log.c @@ -401,7 +401,7 @@ char *alloc_printf(const char *format, ...) * fast when invoked more often than every 500ms. * */ -void keep_alive() +void keep_alive(void) { current_time = timeval_ms(); if (current_time-last_time > 1000) { @@ -436,7 +436,7 @@ void keep_alive() } /* reset keep alive timer without sending message */ -void kept_alive() +void kept_alive(void) { current_time = timeval_ms(); last_time = current_time; diff --git a/src/jtag/core.c b/src/jtag/core.c index f12ab59249..884a743650 100644 --- a/src/jtag/core.c +++ b/src/jtag/core.c @@ -1872,7 +1872,7 @@ void jtag_set_verify(bool enable) jtag_verify = enable; } -bool jtag_will_verify() +bool jtag_will_verify(void) { return jtag_verify; } @@ -1882,7 +1882,7 @@ void jtag_set_verify_capture_ir(bool enable) jtag_verify_capture_ir = enable; } -bool jtag_will_verify_capture_ir() +bool jtag_will_verify_capture_ir(void) { return jtag_verify_capture_ir; } diff --git a/src/jtag/drivers/arm-jtag-ew.c b/src/jtag/drivers/arm-jtag-ew.c index d35e15128d..405278b9a1 100644 --- a/src/jtag/drivers/arm-jtag-ew.c +++ b/src/jtag/drivers/arm-jtag-ew.c @@ -683,7 +683,7 @@ static int armjtagew_tap_execute(void) /**************************************************************************** * JLink USB low-level functions */ -static struct armjtagew *armjtagew_usb_open() +static struct armjtagew *armjtagew_usb_open(void) { usb_init(); diff --git a/src/jtag/interface.c b/src/jtag/interface.c index de132bbe8e..56bbf6e51b 100644 --- a/src/jtag/interface.c +++ b/src/jtag/interface.c @@ -45,7 +45,7 @@ void tap_set_state_impl(tap_state_t new_state) state_follower = new_state; } -tap_state_t tap_get_state() +tap_state_t tap_get_state(void) { return state_follower; } @@ -64,7 +64,7 @@ void tap_set_end_state(tap_state_t new_end_state) end_state_follower = new_end_state; } -tap_state_t tap_get_end_state() +tap_state_t tap_get_end_state(void) { return end_state_follower; } diff --git a/src/jtag/minidummy/minidummy.c b/src/jtag/minidummy/minidummy.c index 7d5e6a98a6..7ee206732b 100644 --- a/src/jtag/minidummy/minidummy.c +++ b/src/jtag/minidummy/minidummy.c @@ -83,7 +83,7 @@ int interface_jtag_add_plain_dr_scan(int num_bits, const uint8_t *out_bits, return ERROR_OK; } -int interface_jtag_add_tlr() +int interface_jtag_add_tlr(void) { /* synchronously do the operation here */ diff --git a/src/jtag/zy1000/zy1000.c b/src/jtag/zy1000/zy1000.c index 48a3d4d039..37af2f7ae4 100644 --- a/src/jtag/zy1000/zy1000.c +++ b/src/jtag/zy1000/zy1000.c @@ -486,7 +486,7 @@ int interface_jtag_add_plain_dr_scan(int num_bits, return ERROR_OK; } -int interface_jtag_add_tlr() +int interface_jtag_add_tlr(void) { setCurrentState(TAP_RESET); return ERROR_OK; -- 2.30.2