From 02e18316d400a4b4fcc26f2e180de5d521721f4d Mon Sep 17 00:00:00 2001 From: Antonio Ospite Date: Thu, 29 Sep 2011 23:05:58 +0200 Subject: [PATCH] kinect_upload_fw: fix -Wmissing-prototypes warning MIME-Version: 1.0 Content-Type: text/plain; charset=utf8 Content-Transfer-Encoding: 8bit kinect_upload_fw.c:60:6: warning: no previous prototype for ‘dump_bl_cmd’ [-Wmissing-prototypes] kinect_upload_fw.c:67:5: warning: no previous prototype for ‘get_first_reply’ [-Wmissing-prototypes] kinect_upload_fw.c:86:5: warning: no previous prototype for ‘get_reply’ [-Wmissing-prototypes] GCC emits "missing-prototypes" argument for _public_ functions without a prototype, making them static is the right fix here. --- kinect_upload_fw/kinect_upload_fw.c | 7 +++---- 1 file changed, 3 insertions(+), 4 deletions(-) diff --git a/kinect_upload_fw/kinect_upload_fw.c b/kinect_upload_fw/kinect_upload_fw.c index 626b963..c932908 100644 --- a/kinect_upload_fw/kinect_upload_fw.c +++ b/kinect_upload_fw/kinect_upload_fw.c @@ -57,14 +57,14 @@ typedef struct { #define fn_le32(x) (x) // TODO: support architectures that aren't little-endian -void dump_bl_cmd(bootloader_command cmd) { +static void dump_bl_cmd(bootloader_command cmd) { int i; for(i = 0; i < 24; i++) LOG("%02X ", ((unsigned char*)(&cmd))[i]); LOG("\n"); } -int get_first_reply(void) { +static int get_first_reply(void) { unsigned char buffer[512]; int res; int transferred; @@ -82,8 +82,7 @@ int get_first_reply(void) { return res; } - -int get_reply(void) { +static int get_reply(void) { unsigned char dump[512]; status_code buffer = ((status_code*)dump)[0]; int res; -- 2.1.4