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/+/8703

-- gerrit

commit 416a3d9b62a550f954ef7ef3f3c33d3facd04580
Author: Antonio Borneo <borneo.anto...@gmail.com>
Date:   Fri Jan 10 15:12:38 2025 +0100

    target: etm: drop useless typedefs
    
    No need to use a typedef for an enum.
    Drop etmv1_pipestat_t and etmv1_branch_reason_t.
    
    Change-Id: I03ae4de3efe699d9635fc4f162649f6bedcef4c0
    Signed-off-by: Antonio Borneo <borneo.anto...@gmail.com>

diff --git a/src/target/etm.h b/src/target/etm.h
index be5f2c7d07..6c5b5e5953 100644
--- a/src/target/etm.h
+++ b/src/target/etm.h
@@ -175,7 +175,7 @@ struct etm_context {
 };
 
 /* PIPESTAT values */
-typedef enum {
+enum etmv1_pipestat {
        STAT_IE = 0x0,
        STAT_ID = 0x1,
        STAT_IN = 0x2,
@@ -184,10 +184,10 @@ typedef enum {
        STAT_BD = 0x5,
        STAT_TR = 0x6,
        STAT_TD = 0x7
-} etmv1_pipestat_t;
+};
 
 /* branch reason values */
-typedef enum {
+enum etmv1_branch_reason {
        BR_NORMAL  = 0x0, /* Normal PC change : periodic synchro (ETMv1.1) */
        BR_ENABLE  = 0x1, /* Trace has been enabled */
        BR_RESTART = 0x2, /* Trace restarted after a FIFO overflow */
@@ -196,7 +196,7 @@ typedef enum {
        BR_RSVD5   = 0x5, /* reserved */
        BR_RSVD6   = 0x6, /* reserved */
        BR_RSVD7   = 0x7, /* reserved */
-} etmv1_branch_reason_t;
+};
 
 struct reg_cache *etm_build_reg_cache(struct target *target,
                struct arm_jtag *jtag_info, struct etm_context *etm_ctx);

-- 

Reply via email to