Hello Dave,

This works fine for me! Thanks!

But other files had also the same problem. I managed to solve them myself, using the same trick you did.
So for these other files, I created a patch file.

Thanks again!

Ferdinand


David Brownell schreef:
On Sunday 19 July 2009, Ferdinand Postema wrote:
When I try to compile the latest version on cygwin, I get the following warnings and a error:

arm_disassembler.c: In function `evaluate_load_literal_thumb':
arm_disassembler.c:1715: warning: unsigned int format, uint32_t arg (arg 8)
arm_disassembler.c:1715: warning: unsigned int format, uint32_t arg (arg 8)
        ... etc ...

I think that compiler is misbehaving ... it's generating each
warning twice!  And that's on top of generating warnings that
I can't reproduce here (non-cygwin); looks like type conversion
is acting strange too (uint32_t masked with int not giving int).

Does the attached patch resolve things?

- Dave


Index: src/jtag/amt_jtagaccel.c
===================================================================
--- src/jtag/amt_jtagaccel.c    (revision 2557)
+++ src/jtag/amt_jtagaccel.c    (working copy)
@@ -374,7 +374,7 @@
                                break;
                        case JTAG_SLEEP:
 #ifdef _DEBUG_JTAG_IO_
-                               LOG_DEBUG("sleep %i", cmd->cmd.sleep->us);
+                               LOG_DEBUG("sleep %" PRIi32, cmd->cmd.sleep->us);
 #endif
                                jtag_sleep(cmd->cmd.sleep->us);
                                break;
Index: src/jtag/bitbang.c
===================================================================
--- src/jtag/bitbang.c  (revision 2557)
+++ src/jtag/bitbang.c  (working copy)
@@ -308,7 +308,7 @@
                                break;
                        case JTAG_SLEEP:
 #ifdef _DEBUG_JTAG_IO_
-                               LOG_DEBUG("sleep %i", cmd->cmd.sleep->us);
+                               LOG_DEBUG("sleep %" PRIi32, cmd->cmd.sleep->us);
 #endif
                                jtag_sleep(cmd->cmd.sleep->us);
                                break;
Index: src/target/arm_adi_v5.c
===================================================================
--- src/target/arm_adi_v5.c     (revision 2557)
+++ src/target/arm_adi_v5.c     (working copy)
@@ -1144,7 +1144,7 @@
 
        dap_ap_read_reg_u32(swjdp, 0xF8, &baseaddr);
        retval = swjdp_transaction_endcheck(swjdp);
-       command_print(cmd_ctx, "0x%8.8x", baseaddr);
+       command_print(cmd_ctx, "0x%8.8" PRIx32, baseaddr);
 
        if (apselsave != apsel)
                dap_ap_select(swjdp, apselsave);
@@ -1162,7 +1162,7 @@
                memaccess_tck = strtoul(args[0], NULL, 0);
 
        swjdp->memaccess_tck = memaccess_tck;
-       command_print(cmd_ctx, "memory bus access delay set to %i tck",
+       command_print(cmd_ctx, "memory bus access delay set to %" PRIi32 " tck",
                        swjdp->memaccess_tck);
 
        return ERROR_OK;
@@ -1181,7 +1181,7 @@
        dap_ap_select(swjdp, apsel);
        dap_ap_read_reg_u32(swjdp, 0xFC, &apid);
        retval = swjdp_transaction_endcheck(swjdp);
-       command_print(cmd_ctx, "ap %i selected, identification register 
0x%8.8x",
+       command_print(cmd_ctx, "ap %" PRIi32 " selected, identification 
register 0x%8.8" PRIx32,
                        apsel, apid);
 
        return retval;
@@ -1203,7 +1203,7 @@
 
        dap_ap_read_reg_u32(swjdp, 0xFC, &apid);
        retval = swjdp_transaction_endcheck(swjdp);
-       command_print(cmd_ctx, "0x%8.8x", apid);
+       command_print(cmd_ctx, "0x%8.8" PRIx32, apid);
        if (apselsave != apsel)
                dap_ap_select(swjdp, apselsave);
 
_______________________________________________
Openocd-development mailing list
[email protected]
https://lists.berlios.de/mailman/listinfo/openocd-development

Reply via email to