[OpenOCD-commit] Main OpenOCD repository branch, master, updated. v0.12.0-690-g16429f625

[openocd-gerrit <ope...@us...>]

This is an automated email from the git hooks/post-receive script. It was
generated because a ref change was pushed to the repository containing
the project "Main OpenOCD repository".

The branch, master has been updated
       via  16429f6252c0d4b985d882e53ff476fe42fa0125 (commit)
      from  ceae51ad74ba3a16be3e7ee2df4743a6df3cd097 (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.

- Log -----------------------------------------------------------------
commit 16429f6252c0d4b985d882e53ff476fe42fa0125
Author: Marc Schink <de...@za...>
Date:   Thu Aug 1 09:22:02 2024 +0200

    target/arm_cti: Use suitable data types
    
    While at it, fix some small coding style issues.
    
    Change-Id: Ifb8e78b55d29a06d69a3ce71d12d0040777aef13
    Signed-off-by: Marc Schink <de...@za...>
    Reviewed-on: https://review.openocd.org/c/openocd/+/8423
    Tested-by: jenkins
    Reviewed-by: Antonio Borneo <bor...@gm...>

diff --git a/src/target/arm_cti.c b/src/target/arm_cti.c
index 7637ad015..0ea853e49 100644
--- a/src/target/arm_cti.c
+++ b/src/target/arm_cti.c
@@ -196,9 +196,7 @@ static const struct {
 
 static int cti_find_reg_offset(const char *name)
 {
-	unsigned int i;
-
-	for (i = 0; i < ARRAY_SIZE(cti_names); i++) {
+	for (size_t i = 0; i < ARRAY_SIZE(cti_names); i++) {
 		if (!strcmp(name, cti_names[i].label))
 			return cti_names[i].offset;
 	}
@@ -227,7 +225,7 @@ COMMAND_HANDLER(handle_cti_dump)
 	struct adiv5_ap *ap = cti->ap;
 	int retval = ERROR_OK;
 
-	for (int i = 0; (retval == ERROR_OK) && (i < (int)ARRAY_SIZE(cti_names)); i++)
+	for (size_t i = 0; (retval == ERROR_OK) && (i < ARRAY_SIZE(cti_names)); i++)
 		retval = mem_ap_read_u32(ap,
 				cti->spot.base + cti_names[i].offset, cti_names[i].p_val);
 
@@ -237,7 +235,7 @@ COMMAND_HANDLER(handle_cti_dump)
 	if (retval != ERROR_OK)
 		return JIM_ERR;
 
-	for (int i = 0; i < (int)ARRAY_SIZE(cti_names); i++)
+	for (size_t i = 0; i < ARRAY_SIZE(cti_names); i++)
 		command_print(CMD, "%8.8s (0x%04"PRIx32") 0x%08"PRIx32,
 				cti_names[i].label, cti_names[i].offset, *cti_names[i].p_val);
 
@@ -323,7 +321,6 @@ COMMAND_HANDLER(handle_cti_ack)
 
 	int retval = arm_cti_ack_events(cti, 1 << event);
 
-
 	if (retval != ERROR_OK)
 		return retval;
 
@@ -437,6 +434,7 @@ static int cti_configure(struct jim_getopt_info *goi, struct arm_cti *cti)
 
 	return JIM_OK;
 }
+
 static int cti_create(struct jim_getopt_info *goi)
 {
 	struct command_context *cmd_ctx;
@@ -538,7 +536,6 @@ COMMAND_HANDLER(cti_handle_names)
 	return ERROR_OK;
 }
 
-
 static const struct command_registration cti_subcommand_handlers[] = {
 	{
 		.name = "create",

-----------------------------------------------------------------------

Summary of changes:
 src/target/arm_cti.c | 11 ++++-------
 1 file changed, 4 insertions(+), 7 deletions(-)


hooks/post-receive
-- 
Main OpenOCD repository