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); /* --