This is an automated email from Gerrit.

"Walter J. <walter...@oss.cipunited.com>" just uploaded a new patch set to 
Gerrit, which you can find at https://review.openocd.org/c/openocd/+/8179

-- gerrit

commit 3222396c74abcae9bd18b1854a8917111d2b87b8
Author: Walter Ji <walter...@oss.cipunited.com>
Date:   Mon Mar 18 17:24:51 2024 +0800

    target/mips32: fix clang sbuild check fail
    
    Initialized `value` variables that could only be set in a branch.
    
    Change-Id: Iec7413ade9d053c93352a58ff954ad49a6545923
    Signed-off-by: Walter Ji <walter...@oss.cipunited.com>

diff --git a/src/target/mips32.c b/src/target/mips32.c
index 6bbf71bd82..81faab72da 100644
--- a/src/target/mips32.c
+++ b/src/target/mips32.c
@@ -2109,7 +2109,7 @@ static int mips32_dsp_find_register_by_name(const char 
*reg_name)
  */
 static int mips32_dsp_get_all_regs(struct command_invocation *cmd, struct 
mips_ejtag *ejtag_info)
 {
-       uint32_t value;
+       uint32_t value = 0;
        for (int i = 0; i < MIPS32NUMDSPREGS; i++) {
                int retval = mips32_pracc_read_dsp_reg(ejtag_info, &value, i);
                if (retval != ERROR_OK) {
@@ -2134,7 +2134,7 @@ static int mips32_dsp_get_all_regs(struct 
command_invocation *cmd, struct mips_e
  */
 static int mips32_dsp_get_register(struct command_invocation *cmd, struct 
mips_ejtag *ejtag_info)
 {
-       uint32_t value;
+       uint32_t value = 0;
        int index = mips32_dsp_find_register_by_name(CMD_ARGV[0]);
        if (index == MIPS32NUMDSPREGS) {
                command_print(CMD, "ERROR: register '%s' not found", 
CMD_ARGV[0]);

-- 

Reply via email to