Fix all compilation warnings
authorOlivier Gay <ogay@logitech.com>
Sun, 27 Oct 2013 16:05:13 +0000 (17:05 +0100)
committerOlivier Gay <ogay@logitech.com>
Sun, 27 Oct 2013 16:07:01 +0000 (17:07 +0100)
gdbserver/gdb-server.c
src/st-term.c
src/stlink-common.h
src/stlink-usb.c

index 69eadda497a8f95176e0609f26c277116dfaf5fd..7e8bb685a7ab7e7e12a931ce7fd7fd6cd86e834d 100644 (file)
@@ -172,7 +172,7 @@ int parse_options(int argc, char** argv, st_state_t *st) {
 
 
 int main(int argc, char** argv) {
-       uint32_t voltage;
+       int32_t voltage;
 
        stlink_t *sl = NULL;
 
index 346cca58ffb398fd66f5f3bcc7e8d9c7760d1484..bbf7ea37ef714a7e39062c5ee6ee5f173d44e0b0 100644 (file)
 
 #define STLINKY_MAGIC 0xDEADF00D
 
+#define READ_UINT32_LE(buf)  ((uint32_t) (   buf[0]         \
+                                           | buf[1] <<  8   \
+                                           | buf[2] << 16   \
+                                           | buf[3] << 24))
+
 struct stlinky {
        stlink_t *sl;
        uint32_t off;
@@ -26,11 +31,11 @@ struct stlinky*  stlinky_detect(stlink_t* sl)
        static const uint32_t sram_base = 0x20000000;
        struct stlinky* st = malloc(sizeof(struct stlinky));
        st->sl = sl;
-       printf("sram: 0x%x bytes @ 0x%x\n", sl->sram_base, sl->sram_size);
+       printf("sram: 0x%x bytes @ 0x%zx\n", sl->sram_base, sl->sram_size);
        uint32_t off;
        for (off = 0; off < sl->sram_size; off += 4) {
                stlink_read_mem32(sl, sram_base + off, 4);
-               if ( STLINKY_MAGIC== *(uint32_t*) sl->q_buf)
+               if (STLINKY_MAGIC == READ_UINT32_LE(sl->q_buf))
                {
                        printf("stlinky detected at 0x%x\n", sram_base + off);
                        st->off = sram_base + off;
@@ -121,6 +126,7 @@ static int keep_running = 1;
 static int sigcount=0;
 void cleanup(int dummy)
 {
+       (void) dummy;
        sigcount++;
        keep_running = 0;
        printf("\n\nGot a signal\n");
index edf8c6733b46a43288a6ccc1670c884fd45d8d8e..579622892b38ccb4858a50636f454f0537003663 100644 (file)
@@ -397,7 +397,7 @@ static const chip_params_t devices[] = {
         void (*step) (stlink_t * stl);
         int (*current_mode) (stlink_t * stl);
         void (*force_debug) (stlink_t *sl);
-        uint32_t (*target_voltage) (stlink_t *sl);
+        int32_t (*target_voltage) (stlink_t *sl);
     } stlink_backend_t;
 
     struct _stlink {
index fa2bd7e081c50154fbd3164ce7250ed334b737fe..ec4053ac7bd050bd3d4f0f91b750084902e7a94a 100644 (file)
@@ -203,7 +203,7 @@ void _stlink_usb_version(stlink_t *sl) {
     }
 }
 
-int _stlink_usb_target_voltage(stlink_t *sl) {
+int32_t _stlink_usb_target_voltage(stlink_t *sl) {
     struct stlink_libusb * const slu = sl->backend_data;
     unsigned char* const rdata = sl->q_buf;
     unsigned char* const cmd  = sl->c_buf;