This is an automated email from Gerrit.

"Evgeniy Naydanov <evgeniy.nayda...@syntacore.com>" just uploaded a new patch 
set to Gerrit, which you can find at https://review.openocd.org/c/openocd/+/8646

-- gerrit

commit d863b3630d2a541240a9eac8dacefb1136d1e369
Author: Evgeniy Naydanov <evgeniy.nayda...@syntacore.com>
Date:   Thu Oct 10 21:05:25 2024 +0300

    jtag: `jtag_tap_next_enabled()` accepts a const pointer
    
    `jtag_tap_next_enabled()` is used to traverse the list of TAPs, skipping
    disabled TAPs. It should be able to operate on constant pointers.
    
    Change-Id: I18bfcf88a658833400d1fc2969999c090228eaed
    Signed-off-by: Evgeniy Naydanov <evgeniy.nayda...@syntacore.com>

diff --git a/src/jtag/core.c b/src/jtag/core.c
index 769e07571f..3070465d49 100644
--- a/src/jtag/core.c
+++ b/src/jtag/core.c
@@ -268,9 +268,9 @@ struct jtag_tap *jtag_tap_by_string(const char *s)
        return t;
 }
 
-struct jtag_tap *jtag_tap_next_enabled(struct jtag_tap *p)
+struct jtag_tap *jtag_tap_next_enabled(const struct jtag_tap *s)
 {
-       p = p ? p->next_tap : jtag_all_taps();
+       struct jtag_tap *p = s ? s->next_tap : jtag_all_taps();
        while (p) {
                if (p->enabled)
                        return p;
diff --git a/src/jtag/jtag.h b/src/jtag/jtag.h
index 86526a09a1..49b902d50c 100644
--- a/src/jtag/jtag.h
+++ b/src/jtag/jtag.h
@@ -151,7 +151,7 @@ const char *jtag_tap_name(const struct jtag_tap *tap);
 struct jtag_tap *jtag_tap_by_string(const char *dotted_name);
 struct jtag_tap *jtag_tap_by_jim_obj(Jim_Interp *interp, Jim_Obj *obj);
 struct jtag_tap *jtag_tap_by_position(unsigned int abs_position);
-struct jtag_tap *jtag_tap_next_enabled(struct jtag_tap *p);
+struct jtag_tap *jtag_tap_next_enabled(const struct jtag_tap *s);
 unsigned int jtag_tap_count_enabled(void);
 
 /*

-- 

Reply via email to