This is an automated email from Gerrit.

Christopher Head (ch...@zaber.com) just uploaded a new patch set to Gerrit, 
which you can find at http://openocd.zylin.com/5219

-- gerrit

commit dbe8d1a5b6862526887cc02c537d360757c3bde8
Author: Christopher Head <ch...@zaber.com>
Date:   Thu Jun 6 11:40:12 2019 -0700

    target/target: parse value as proper type
    
    The `value` variable is passed into `target_fill_mem` as its
    second-to-last parameter. That parameter is of type `uint64_t`. It is
    appropriate to parse the value as that type, since otherwise a target
    with a 32-bit address space but 64-bit data write capabilities would not
    be able to exercise those capabilities.
    
    Change-Id: Ib336d47d42c27cd2b5ba1206b04e8f740f167dba
    Signed-off-by: Christopher Head <ch...@zaber.com>

diff --git a/src/target/target.c b/src/target/target.c
index 1e42c5e..2b53f64 100644
--- a/src/target/target.c
+++ b/src/target/target.c
@@ -3318,8 +3318,8 @@ COMMAND_HANDLER(handle_mw_command)
        target_addr_t address;
        COMMAND_PARSE_ADDRESS(CMD_ARGV[0], address);
 
-       target_addr_t value;
-       COMMAND_PARSE_ADDRESS(CMD_ARGV[1], value);
+       uint64_t value;
+       COMMAND_PARSE_NUMBER(u64, CMD_ARGV[1], value);
 
        unsigned count = 1;
        if (CMD_ARGC == 3)

-- 


_______________________________________________
OpenOCD-devel mailing list
OpenOCD-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/openocd-devel

Reply via email to