target/mips: Remove typedef'd struct
authorMarc Schink <dev@zapb.de>
Tue, 26 Jan 2021 08:15:47 +0000 (09:15 +0100)
committerAntonio Borneo <borneo.antonio@gmail.com>
Wed, 10 Mar 2021 21:30:04 +0000 (21:30 +0000)
The C style guide forbids typedef'd structs, see 'Naming Rules'.

Change-Id: I449590251056c478c05105cdc18014ab4eb77ed8
Signed-off-by: Marc Schink <dev@zapb.de>
Reviewed-on: http://openocd.zylin.com/6033
Tested-by: jenkins
Reviewed-by: Oleksij Rempel <linux@rempel-privat.de>
Reviewed-by: Antonio Borneo <borneo.antonio@gmail.com>
src/target/mips32_pracc.c
src/target/mips32_pracc.h
src/target/mips64_pracc.c
src/target/mips_ejtag.c

index d6bd1c58a504c59fdebe4fa4f55012ad86a05a55..f8643fa2d78304b5f75df6d272d79819b4bbd028 100644 (file)
@@ -317,7 +317,7 @@ void pracc_add(struct pracc_queue_info *ctx, uint32_t addr, uint32_t instr)
        if (ctx->retval != ERROR_OK)    /* On previous out of memory, return */
                return;
        if (ctx->code_count == ctx->max_code) {
-               void *p = realloc(ctx->pracc_list, sizeof(pa_list) * (ctx->max_code + PRACC_BLOCK));
+               void *p = realloc(ctx->pracc_list, sizeof(struct pa_list) * (ctx->max_code + PRACC_BLOCK));
                if (p) {
                        ctx->max_code += PRACC_BLOCK;
                        ctx->pracc_list = p;
index 911a69c5b0be3610b82ebaa6a05d2e0d634fc179..30edaec0a28a91806e954c544f54c8b55fca5e32 100644 (file)
 
 #define PRACC_BLOCK    128     /* 1 Kbyte */
 
-typedef struct {
+struct pa_list {
        uint32_t instr;
        uint32_t addr;
-} pa_list;
+};
 
 struct pracc_queue_info {
        struct mips_ejtag *ejtag_info;
@@ -59,7 +59,7 @@ struct pracc_queue_info {
        int code_count;
        int store_count;
        int max_code;           /* max instructions with currently allocated memory */
-       pa_list *pracc_list;    /* Code and store addresses at dmseg */
+       struct pa_list *pracc_list;     /* Code and store addresses at dmseg */
 };
 
 void pracc_queue_init(struct pracc_queue_info *ctx);
index 3080046f3b80dbe87d8735740c2cae7ee464f893..dbad248fcd4288b55c930e15d19e40ad1e1e96cb 100644 (file)
@@ -24,7 +24,7 @@
 
 #define STACK_DEPTH    32
 
-typedef struct {
+struct mips64_pracc_context {
        uint64_t *local_iparam;
        unsigned num_iparam;
        uint64_t *local_oparam;
@@ -34,7 +34,7 @@ typedef struct {
        uint64_t stack[STACK_DEPTH];
        unsigned stack_offset;
        struct mips_ejtag *ejtag_info;
-} mips64_pracc_context;
+};
 
 static int wait_for_pracc_rw(struct mips_ejtag *ejtag_info, uint32_t *ctrl)
 {
@@ -61,7 +61,7 @@ static int wait_for_pracc_rw(struct mips_ejtag *ejtag_info, uint32_t *ctrl)
        return ERROR_OK;
 }
 
-static int mips64_pracc_exec_read(mips64_pracc_context *ctx, uint64_t address)
+static int mips64_pracc_exec_read(struct mips64_pracc_context *ctx, uint64_t address)
 {
        struct mips_ejtag *ejtag_info = ctx->ejtag_info;
        unsigned offset;
@@ -149,7 +149,7 @@ static int mips64_pracc_exec_read(mips64_pracc_context *ctx, uint64_t address)
        return jtag_execute_queue();
 }
 
-static int mips64_pracc_exec_write(mips64_pracc_context *ctx, uint64_t address)
+static int mips64_pracc_exec_write(struct mips64_pracc_context *ctx, uint64_t address)
 {
        uint32_t ejtag_ctrl;
        uint64_t data;
@@ -214,7 +214,7 @@ int mips64_pracc_exec(struct mips_ejtag *ejtag_info,
 {
        uint32_t ejtag_ctrl;
        uint64_t address = 0, address_prev = 0, data;
-       mips64_pracc_context ctx;
+       struct mips64_pracc_context ctx;
        int retval;
        int pass = 0;
        bool first_time_call = true;
index 4b049fb4e468d5ae41ec5635a975effa3f7b3454..d4c019fbe213a188ad04a1f29de0c1e5ba1ab006 100644 (file)
@@ -268,7 +268,7 @@ error:
 
 int mips_ejtag_exit_debug(struct mips_ejtag *ejtag_info)
 {
-       pa_list pracc_list = {.instr = MIPS32_DRET(ejtag_info->isa), .addr = 0};
+       struct pa_list pracc_list = {.instr = MIPS32_DRET(ejtag_info->isa), .addr = 0};
        struct pracc_queue_info ctx = {.max_code = 1, .pracc_list = &pracc_list, .code_count = 1, .store_count = 0};
 
        /* execute our dret instruction */