This is an automated email from Gerrit.

"Antonio Borneo <borneo.anto...@gmail.com>" just uploaded a new patch set to 
Gerrit, which you can find at https://review.openocd.org/c/openocd/+/8478

-- gerrit

commit 1949d6c3c50c55d84ebc52f0309081869b7bcdf8
Author: Antonio Borneo <borneo.anto...@gmail.com>
Date:   Sun Sep 8 21:44:32 2024 +0200

    target: use 'unsigned int' for smp group
    
    Change the type to 'struct target::smp' and to the initialization
    variable 'smp_group'.
    
    Change-Id: I5f5a30a796aaf4e0014a38e81abdf4fb4afbdf48
    Signed-off-by: Antonio Borneo <borneo.anto...@gmail.com>

diff --git a/src/target/target.c b/src/target/target.c
index c3a4ed6d27..281bbbd408 100644
--- a/src/target/target.c
+++ b/src/target/target.c
@@ -6007,7 +6007,7 @@ static int get_target_with_common_rtos_type(struct 
command_invocation *cmd,
 
 COMMAND_HANDLER(handle_target_smp)
 {
-       static int smp_group = 1;
+       static unsigned int smp_group = 1;
 
        if (CMD_ARGC == 0) {
                LOG_DEBUG("Empty SMP target");
diff --git a/src/target/target.h b/src/target/target.h
index d3077f5716..ecc8a90321 100644
--- a/src/target/target.h
+++ b/src/target/target.h
@@ -184,7 +184,7 @@ struct target {
        bool rtos_auto_detect;                          /* A flag that 
indicates that the RTOS has been specified as "auto"
                                                                                
 * and must be detected when symbols are offered */
        struct backoff_timer backoff;
-       int smp;                                                        /* 
Unique non-zero number for each SMP group */
+       unsigned int smp;                                       /* Unique 
non-zero number for each SMP group */
        struct list_head *smp_targets;          /* list all targets in this smp 
group/cluster
                                                                                
 * The head of the list is shared between the
                                                                                
 * cluster, thus here there is a pointer */

-- 

Reply via email to