This is an automated email from Gerrit.

Colin Baumgarten ([email protected]) just uploaded a new patch set to 
Gerrit, which you can find at http://openocd.zylin.com/3940

-- gerrit

commit 5c78ad9a82210f6e74f3565aa53f300bd9b8aaeb
Author: Colin Baumgarten <[email protected]>
Date:   Fri Jan 6 00:04:04 2017 +0100

    contrib/itmdump: fix another compiler warning
    
    Fix the following warning by changing all port instances to unsigned.
    
    itmdump.c: In function ‘show_swit’:
    itmdump.c:279:22: warning: comparison between signed and unsigned integer 
expressions [-Wsign-compare]
       if (format[i].port == port) {
                          ^
    
    Change-Id: I7b46fee5e558e1c2db132b04ed6bbf11b8d219e1
    Signed-off-by: Colin Baumgarten <[email protected]>

diff --git a/contrib/itmdump.c b/contrib/itmdump.c
index f829f0e..71f2a02 100644
--- a/contrib/itmdump.c
+++ b/contrib/itmdump.c
@@ -56,7 +56,7 @@ unsigned int dump_swit;
  * NOTE that this specific encoding could be space-optimized; and that
  * trace data streams could also be history-sensitive.
  */
-static void show_task(int port, unsigned data)
+static void show_task(unsigned port, unsigned data)
 {
        unsigned code = data >> 16;
        char buf[16];
@@ -242,8 +242,8 @@ static void show_hard(FILE *f, int c)
  * REVISIT there can be up to 256 trace ports, via "ITM Extension" packets
  */
 struct {
-       int port;
-       void (*show)(int port, unsigned data);
+       unsigned port;
+       void (*show)(unsigned port, unsigned data);
 } format[] = {
        { .port = 31,  .show = show_task, },
 };

-- 

------------------------------------------------------------------------------
Check out the vibrant tech community on one of the world's most 
engaging tech sites, SlashDot.org! http://sdm.link/slashdot
_______________________________________________
OpenOCD-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/openocd-devel

Reply via email to