diff mbox

[OpenWrt-Devel,2/2] firmware-utils: fix various compiler warnings

Message ID 551425fd7e8ef130dd7d5ec0b1eb0c5c5f5994e4.1448384010.git.felix@kaechele.ca
State Accepted
Headers show

Commit Message

Felix Kaechele Nov. 24, 2015, 4:57 p.m. UTC
mostly implicit defines due to missing includes plus one const that
shouldn't be one.

Signed-off-by: Felix Kaechele <felix@kaechele.ca>
---
 tools/firmware-utils/src/dgfirmware.c | 1 +
 tools/firmware-utils/src/lzma2eva.c   | 2 +-
 tools/firmware-utils/src/mkhilinkfw.c | 1 +
 tools/firmware-utils/src/spw303v.c    | 1 +
 tools/firmware-utils/src/srec2bin.c   | 2 +-
 tools/firmware-utils/src/wrt400n.c    | 2 ++
 tools/firmware-utils/src/zyxbcm.c     | 1 +
 7 files changed, 8 insertions(+), 2 deletions(-)

Comments

John Crispin Nov. 24, 2015, 6:42 p.m. UTC | #1
On 24/11/2015 17:57, Felix Kaechele wrote:
> diff --git a/tools/firmware-utils/src/srec2bin.c b/tools/firmware-utils/src/srec2bin.c
> index 1cffbae..5cc71bd 100644
> --- a/tools/firmware-utils/src/srec2bin.c
> +++ b/tools/firmware-utils/src/srec2bin.c
> @@ -513,7 +513,7 @@ int srec2bin(int argc,char *argv[],int verbose)
>      return(1);
>  }
>  
> -main(int argc, char *argv[])
> +int main(int argc, char *argv[])
>  {
>      debug = TRUE;
>      debug = FALSE;


yikees ... its these gems that scare me ;)
diff mbox

Patch

diff --git a/tools/firmware-utils/src/dgfirmware.c b/tools/firmware-utils/src/dgfirmware.c
index 5ff3b69..e3257f1 100644
--- a/tools/firmware-utils/src/dgfirmware.c
+++ b/tools/firmware-utils/src/dgfirmware.c
@@ -1,5 +1,6 @@ 
 #include <stdlib.h>
 #include <stdio.h>
+#include <string.h>
 
 
 #define IMG_SIZE     0x3e0000
diff --git a/tools/firmware-utils/src/lzma2eva.c b/tools/firmware-utils/src/lzma2eva.c
index 0bc13fa..1d7e364 100644
--- a/tools/firmware-utils/src/lzma2eva.c
+++ b/tools/firmware-utils/src/lzma2eva.c
@@ -48,7 +48,7 @@  main(int argc, char *argv[])
 
   const char *infile, *outfile;
   FILE *in, *out;
-  static const uint8_t buf[4096];
+  static uint8_t buf[4096];
   size_t elems;
 
   uint8_t properties;
diff --git a/tools/firmware-utils/src/mkhilinkfw.c b/tools/firmware-utils/src/mkhilinkfw.c
index 99322d4..55908e5 100644
--- a/tools/firmware-utils/src/mkhilinkfw.c
+++ b/tools/firmware-utils/src/mkhilinkfw.c
@@ -22,6 +22,7 @@ 
  *   gcc -lcrypto hlkcrypt.c -o hlkcrypt
  */
  
+#include <arpa/inet.h>
 #include <errno.h>
 #include <fcntl.h>
 #include <getopt.h>
diff --git a/tools/firmware-utils/src/spw303v.c b/tools/firmware-utils/src/spw303v.c
index ae34a1e..654d68d 100644
--- a/tools/firmware-utils/src/spw303v.c
+++ b/tools/firmware-utils/src/spw303v.c
@@ -18,6 +18,7 @@ 
  * Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA 02111-1307 USA
  */
 
+#include <arpa/inet.h>
 #include <stdio.h>
 #include <stdlib.h>
 #include <string.h>
diff --git a/tools/firmware-utils/src/srec2bin.c b/tools/firmware-utils/src/srec2bin.c
index 1cffbae..5cc71bd 100644
--- a/tools/firmware-utils/src/srec2bin.c
+++ b/tools/firmware-utils/src/srec2bin.c
@@ -513,7 +513,7 @@  int srec2bin(int argc,char *argv[],int verbose)
     return(1);
 }
 
-main(int argc, char *argv[])
+int main(int argc, char *argv[])
 {
     debug = TRUE;
     debug = FALSE;
diff --git a/tools/firmware-utils/src/wrt400n.c b/tools/firmware-utils/src/wrt400n.c
index a9a4908..1cf1deb 100644
--- a/tools/firmware-utils/src/wrt400n.c
+++ b/tools/firmware-utils/src/wrt400n.c
@@ -7,12 +7,14 @@ 
  *
  *	Author: Sandeep Mistry
  */
+#include <arpa/inet.h>
 #include <fcntl.h>
 #include <stdio.h>
 #include <stdint.h>
 #include <string.h>
 #include <sys/types.h>
 #include <sys/stat.h>
+#include <unistd.h>
 
 #include "cyg_crc.h"
 
diff --git a/tools/firmware-utils/src/zyxbcm.c b/tools/firmware-utils/src/zyxbcm.c
index cfd00d3..ffeb7cc 100644
--- a/tools/firmware-utils/src/zyxbcm.c
+++ b/tools/firmware-utils/src/zyxbcm.c
@@ -18,6 +18,7 @@ 
  * Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA 02111-1307 USA
  */
 
+#include <arpa/inet.h>
 #include <stdio.h>
 #include <stdlib.h>
 #include <string.h>