target/mips: Remove typedef'd struct 33/6033/2
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 */

Linking to existing account procedure

If you already have an account and want to add another login method you MUST first sign in with your existing account and then change URL to read https://review.openocd.org/login/?link to get to this page again but this time it'll work for linking. Thank you.

SSH host keys fingerprints

1024 SHA256:YKx8b7u5ZWdcbp7/4AeXNaqElP49m6QrwfXaqQGJAOk gerrit-code-review@openocd.zylin.com (DSA)
384 SHA256:jHIbSQa4REvwCFG4cq5LBlBLxmxSqelQPem/EXIrxjk gerrit-code-review@openocd.org (ECDSA)
521 SHA256:UAOPYkU9Fjtcao0Ul/Rrlnj/OsQvt+pgdYSZ4jOYdgs gerrit-code-review@openocd.org (ECDSA)
256 SHA256:A13M5QlnozFOvTllybRZH6vm7iSt0XLxbA48yfc2yfY gerrit-code-review@openocd.org (ECDSA)
256 SHA256:spYMBqEYoAOtK7yZBrcwE8ZpYt6b68Cfh9yEVetvbXg gerrit-code-review@openocd.org (ED25519)
+--[ED25519 256]--+
|=..              |
|+o..   .         |
|*.o   . .        |
|+B . . .         |
|Bo. = o S        |
|Oo.+ + =         |
|oB=.* = . o      |
| =+=.+   + E     |
|. .=o   . o      |
+----[SHA256]-----+
2048 SHA256:0Onrb7/PHjpo6iVZ7xQX2riKN83FJ3KGU0TvI0TaFG4 gerrit-code-review@openocd.zylin.com (RSA)