]> www.infradead.org Git - users/borneoa/openocd-next.git/commitdiff
target: riscv: convert 'unsigned' to 'unsigned int'
authorAntonio Borneo <borneo.antonio@gmail.com>
Sun, 8 Sep 2024 19:23:28 +0000 (21:23 +0200)
committerAntonio Borneo <borneo.antonio@gmail.com>
Sun, 20 Oct 2024 09:22:52 +0000 (09:22 +0000)
Conversion done with
checkpatch --fix-inplace -types UNSPECIFIED_INT

Ignore the cast as they could be better addressed.
Fix only minor additional checkpatch issue (spacing and line
length).

Change-Id: I11f10eddadc21e051c96eb3d4d4c0554a2cddd15
Signed-off-by: Antonio Borneo <borneo.antonio@gmail.com>
Reviewed-on: https://review.openocd.org/c/openocd/+/8483
Tested-by: jenkins
src/target/riscv/batch.c
src/target/riscv/batch.h
src/target/riscv/opcodes.h
src/target/riscv/program.c
src/target/riscv/riscv-011.c
src/target/riscv/riscv-013.c
src/target/riscv/riscv.c
src/target/riscv/riscv.h

index d4bdadf1700cc2b261a71ff5a3b62907d461e5e3..859e2275e503f821909f0efdc9fd6e990409f9f9 100644 (file)
@@ -127,7 +127,7 @@ int riscv_batch_run(struct riscv_batch *batch)
        return ERROR_OK;
 }
 
-void riscv_batch_add_dmi_write(struct riscv_batch *batch, unsigned address, uint64_t data)
+void riscv_batch_add_dmi_write(struct riscv_batch *batch, unsigned int address, uint64_t data)
 {
        assert(batch->used_scans < batch->allocated_scans);
        struct scan_field *field = batch->fields + batch->used_scans;
@@ -140,7 +140,7 @@ void riscv_batch_add_dmi_write(struct riscv_batch *batch, unsigned address, uint
        batch->used_scans++;
 }
 
-size_t riscv_batch_add_dmi_read(struct riscv_batch *batch, unsigned address)
+size_t riscv_batch_add_dmi_read(struct riscv_batch *batch, unsigned int address)
 {
        assert(batch->used_scans < batch->allocated_scans);
        struct scan_field *field = batch->fields + batch->used_scans;
@@ -156,7 +156,7 @@ size_t riscv_batch_add_dmi_read(struct riscv_batch *batch, unsigned address)
        return batch->read_keys_used++;
 }
 
-unsigned riscv_batch_get_dmi_read_op(struct riscv_batch *batch, size_t key)
+unsigned int riscv_batch_get_dmi_read_op(struct riscv_batch *batch, size_t key)
 {
        assert(key < batch->read_keys_used);
        size_t index = batch->read_keys[key];
index 9c42ba81ea02a57777aab989cce584a4ea0cb697..e5b94cc2194fb271f8b3ff27d2d6d89578ff4f1c 100644 (file)
@@ -59,13 +59,13 @@ bool riscv_batch_full(struct riscv_batch *batch);
 int riscv_batch_run(struct riscv_batch *batch);
 
 /* Adds a DMI write to this batch. */
-void riscv_batch_add_dmi_write(struct riscv_batch *batch, unsigned address, uint64_t data);
+void riscv_batch_add_dmi_write(struct riscv_batch *batch, unsigned int address, uint64_t data);
 
 /* DMI reads must be handled in two parts: the first one schedules a read and
  * provides a key, the second one actually obtains the result of the read -
  * status (op) and the actual data. */
-size_t riscv_batch_add_dmi_read(struct riscv_batch *batch, unsigned address);
-unsigned riscv_batch_get_dmi_read_op(struct riscv_batch *batch, size_t key);
+size_t riscv_batch_add_dmi_read(struct riscv_batch *batch, unsigned int address);
+unsigned int riscv_batch_get_dmi_read_op(struct riscv_batch *batch, size_t key);
 uint32_t riscv_batch_get_dmi_read_data(struct riscv_batch *batch, size_t key);
 
 /* Scans in a NOP. */
index 8faa154bae9632a43a42e7b002c9d0f9f539a935..7d17e6eea0acbf6b5aea9d233535de4bb796c68e 100644 (file)
@@ -191,26 +191,26 @@ static uint32_t fld(unsigned int dest, unsigned int base, uint16_t offset)
        return imm_i(offset) | inst_rs1(base) | inst_rd(dest) | MATCH_FLD;
 }
 
-static uint32_t fmv_x_w(unsigned dest, unsigned src) __attribute__ ((unused));
-static uint32_t fmv_x_w(unsigned dest, unsigned src)
+static uint32_t fmv_x_w(unsigned int dest, unsigned int src) __attribute__ ((unused));
+static uint32_t fmv_x_w(unsigned int dest, unsigned int src)
 {
        return inst_rs1(src) | inst_rd(dest) | MATCH_FMV_X_W;
 }
 
-static uint32_t fmv_x_d(unsigned dest, unsigned src) __attribute__ ((unused));
-static uint32_t fmv_x_d(unsigned dest, unsigned src)
+static uint32_t fmv_x_d(unsigned int dest, unsigned int src) __attribute__ ((unused));
+static uint32_t fmv_x_d(unsigned int dest, unsigned int src)
 {
        return inst_rs1(src) | inst_rd(dest) | MATCH_FMV_X_D;
 }
 
-static uint32_t fmv_w_x(unsigned dest, unsigned src) __attribute__ ((unused));
-static uint32_t fmv_w_x(unsigned dest, unsigned src)
+static uint32_t fmv_w_x(unsigned int dest, unsigned int src) __attribute__ ((unused));
+static uint32_t fmv_w_x(unsigned int dest, unsigned int src)
 {
        return inst_rs1(src) | inst_rd(dest) | MATCH_FMV_W_X;
 }
 
-static uint32_t fmv_d_x(unsigned dest, unsigned src) __attribute__ ((unused));
-static uint32_t fmv_d_x(unsigned dest, unsigned src)
+static uint32_t fmv_d_x(unsigned int dest, unsigned int src) __attribute__ ((unused));
+static uint32_t fmv_d_x(unsigned int dest, unsigned int src)
 {
        return inst_rs1(src) | inst_rd(dest) | MATCH_FMV_D_X;
 }
index 0976539b3c73f894397fc449ae3ab25912a649d1..1014137f4f69610aa826e3ecbc22251b2eecc041 100644 (file)
@@ -31,7 +31,7 @@ int riscv_program_init(struct riscv_program *p, struct target *target)
 
 int riscv_program_write(struct riscv_program *program)
 {
-       for (unsigned i = 0; i < program->instruction_count; ++i) {
+       for (unsigned int i = 0; i < program->instruction_count; ++i) {
                LOG_DEBUG("debug_buffer[%02x] = DASM(0x%08x)", i, program->debug_buffer[i]);
                if (riscv_write_debug_buffer(program->target, i,
                                        program->debug_buffer[i]) != ERROR_OK)
index 565721c28bc99ffa7af2a341731b769cf5b26973..302ace86982cc52ba63f4b6c6babfc50519dc9fc 100644 (file)
@@ -474,7 +474,7 @@ static uint64_t dbus_read(struct target *target, uint16_t address)
         * While somewhat nonintuitive, this is an efficient way to get the data.
         */
 
-       unsigned i = 0;
+       unsigned int i = 0;
        do {
                status = dbus_scan(target, &address_in, &value, DBUS_OP_READ, address, 0);
                if (status == DBUS_STATUS_BUSY)
@@ -495,7 +495,7 @@ static uint64_t dbus_read(struct target *target, uint16_t address)
 static void dbus_write(struct target *target, uint16_t address, uint64_t value)
 {
        dbus_status_t status = DBUS_STATUS_BUSY;
-       unsigned i = 0;
+       unsigned int i = 0;
        while (status == DBUS_STATUS_BUSY && i++ < 256) {
                status = dbus_scan(target, NULL, NULL, DBUS_OP_WRITE, address, value);
                if (status == DBUS_STATUS_BUSY)
@@ -656,13 +656,13 @@ static void scans_add_read(scans_t *scans, slot_t slot, bool set_interrupt)
 }
 
 static uint32_t scans_get_u32(scans_t *scans, unsigned int index,
-               unsigned first, unsigned num)
+               unsigned int first, unsigned int num)
 {
        return buf_get_u32(scans->in + scans->scan_size * index, first, num);
 }
 
 static uint64_t scans_get_u64(scans_t *scans, unsigned int index,
-               unsigned first, unsigned num)
+               unsigned int first, unsigned int num)
 {
        return buf_get_u64(scans->in + scans->scan_size * index, first, num);
 }
@@ -699,7 +699,7 @@ static bits_t read_bits(struct target *target)
        };
 
        do {
-               unsigned i = 0;
+               unsigned int i = 0;
                do {
                        status = dbus_scan(target, &address_in, &value, DBUS_OP_READ, 0, 0);
                        if (status == DBUS_STATUS_BUSY) {
@@ -1296,7 +1296,7 @@ static int register_write(struct target *target, unsigned int number,
                int result = update_mstatus_actual(target);
                if (result != ERROR_OK)
                        return result;
-               unsigned i = 0;
+               unsigned int i = 0;
                if ((info->mstatus_actual & MSTATUS_FS) == 0) {
                        info->mstatus_actual = set_field(info->mstatus_actual, MSTATUS_FS, 1);
                        cache_set_load(target, i++, S0, SLOT1);
@@ -1352,7 +1352,7 @@ static int get_register(struct target *target, riscv_reg_t *value, int regid)
                int result = update_mstatus_actual(target);
                if (result != ERROR_OK)
                        return result;
-               unsigned i = 0;
+               unsigned int i = 0;
                if ((info->mstatus_actual & MSTATUS_FS) == 0) {
                        info->mstatus_actual = set_field(info->mstatus_actual, MSTATUS_FS, 1);
                        cache_set_load(target, i++, S0, SLOT1);
@@ -1538,7 +1538,7 @@ static int examine(struct target *target)
        /* 0x00000000  0x00000000:00000003  0x00000000:00000003:ffffffff:ffffffff */
        cache_set32(target, 4, sw(S1, ZERO, DEBUG_RAM_START + 4));
        cache_set_jump(target, 5);
-       for (unsigned i = 6; i < info->dramsize; i++)
+       for (unsigned int i = 6; i < info->dramsize; i++)
                cache_set32(target, i, i * 0x01020304);
 
        cache_write(target, 0, false);
@@ -1569,7 +1569,7 @@ static int examine(struct target *target)
        LOG_DEBUG("Discovered XLEN is %d", riscv_xlen(target));
 
        if (read_remote_csr(target, &r->misa, CSR_MISA) != ERROR_OK) {
-               const unsigned old_csr_misa = 0xf10;
+               const unsigned int old_csr_misa = 0xf10;
                LOG_WARNING("Failed to read misa at 0x%x; trying 0x%x.", CSR_MISA,
                                old_csr_misa);
                if (read_remote_csr(target, &r->misa, old_csr_misa) != ERROR_OK) {
@@ -1651,7 +1651,7 @@ static riscv_error_t handle_halt_routine(struct target *target)
 
        unsigned int dbus_busy = 0;
        unsigned int interrupt_set = 0;
-       unsigned result = 0;
+       unsigned int result = 0;
        uint64_t value = 0;
        reg_cache_set(target, 0, 0);
        /* The first scan result is the result from something old we don't care
@@ -2016,7 +2016,7 @@ static int read_memory(struct target *target, target_addr_t address,
        cache_write(target, CACHE_NO_READ, false);
 
        riscv011_info_t *info = get_info(target);
-       const unsigned max_batch_size = 256;
+       const unsigned int max_batch_size = 256;
        scans_t *scans = scans_new(target, max_batch_size);
        if (!scans)
                return ERROR_FAIL;
@@ -2174,7 +2174,7 @@ static int write_memory(struct target *target, target_addr_t address,
        if (setup_write_memory(target, size) != ERROR_OK)
                return ERROR_FAIL;
 
-       const unsigned max_batch_size = 256;
+       const unsigned int max_batch_size = 256;
        scans_t *scans = scans_new(target, max_batch_size);
        if (!scans)
                return ERROR_FAIL;
index dbf9aad1d95fde83c04f60b3f150c5eca78d1aaa..16eced2d8947ee9c63ae312e912c56889357acdc 100644 (file)
@@ -48,7 +48,7 @@ static int riscv013_on_step(struct target *target);
 static int riscv013_resume_prep(struct target *target);
 static bool riscv013_is_halted(struct target *target);
 static enum riscv_halt_reason riscv013_halt_reason(struct target *target);
-static int riscv013_write_debug_buffer(struct target *target, unsigned index,
+static int riscv013_write_debug_buffer(struct target *target, unsigned int index,
                riscv_insn_t d);
 static riscv_insn_t riscv013_read_debug_buffer(struct target *target, unsigned
                index);
@@ -59,7 +59,7 @@ static int riscv013_dmi_write_u64_bits(struct target *target);
 static void riscv013_fill_dmi_nop_u64(struct target *target, char *buf);
 static int register_read(struct target *target, uint64_t *value, uint32_t number);
 static int register_read_direct(struct target *target, uint64_t *value, uint32_t number);
-static int register_write_direct(struct target *target, unsigned number,
+static int register_write_direct(struct target *target, unsigned int number,
                uint64_t value);
 static int read_memory(struct target *target, target_addr_t address,
                uint32_t size, uint32_t count, uint8_t *buffer, uint32_t increment);
@@ -156,13 +156,13 @@ typedef struct {
 
 typedef struct {
        /* The indexed used to address this hart in its DM. */
-       unsigned index;
+       unsigned int index;
        /* Number of address bits in the dbus register. */
-       unsigned abits;
+       unsigned int abits;
        /* Number of abstract command data registers. */
-       unsigned datacount;
+       unsigned int datacount;
        /* Number of words in the Program Buffer. */
-       unsigned progbufsize;
+       unsigned int progbufsize;
 
        /* We cache the read-only bits of sbcs here. */
        uint32_t sbcs;
@@ -209,7 +209,7 @@ typedef struct {
        int16_t dataaddr;
 
        /* The width of the hartsel field. */
-       unsigned hartsellen;
+       unsigned int hartsellen;
 
        /* DM that provides access to this target. */
        dm013_info_t *dm;
@@ -290,10 +290,10 @@ static uint32_t set_hartsel(uint32_t initial, uint32_t index)
        return initial;
 }
 
-static void decode_dmi(char *text, unsigned address, unsigned data)
+static void decode_dmi(char *text, unsigned int address, unsigned int data)
 {
        static const struct {
-               unsigned address;
+               unsigned int address;
                uint64_t mask;
                const char *name;
        } description[] = {
@@ -350,10 +350,10 @@ static void decode_dmi(char *text, unsigned address, unsigned data)
        };
 
        text[0] = 0;
-       for (unsigned i = 0; i < ARRAY_SIZE(description); i++) {
+       for (unsigned int i = 0; i < ARRAY_SIZE(description); i++) {
                if (description[i].address == address) {
                        uint64_t mask = description[i].mask;
-                       unsigned value = get_field(data, mask);
+                       unsigned int value = get_field(data, mask);
                        if (value) {
                                if (i > 0)
                                        *(text++) = ' ';
@@ -468,7 +468,7 @@ static dmi_status_t dmi_scan(struct target *target, uint32_t *address_in,
 {
        riscv013_info_t *info = get_info(target);
        RISCV_INFO(r);
-       unsigned num_bits = info->abits + DTM_DMI_OP_LENGTH + DTM_DMI_DATA_LENGTH;
+       unsigned int num_bits = info->abits + DTM_DMI_OP_LENGTH + DTM_DMI_DATA_LENGTH;
        size_t num_bytes = (num_bits + 7) / 8;
        uint8_t in[num_bytes];
        uint8_t out[num_bytes];
@@ -680,7 +680,7 @@ static int dmi_write_exec(struct target *target, uint32_t address,
 }
 
 static int dmstatus_read_timeout(struct target *target, uint32_t *dmstatus,
-               bool authenticated, unsigned timeout_sec)
+               bool authenticated, unsigned int timeout_sec)
 {
        int result = dmi_op_timeout(target, dmstatus, NULL, DMI_OP_READ,
                        DM_DMSTATUS, 0, timeout_sec, false, true);
@@ -717,7 +717,7 @@ static void increase_ac_busy_delay(struct target *target)
                        info->ac_busy_delay);
 }
 
-static uint32_t __attribute__((unused)) abstract_register_size(unsigned width)
+static uint32_t __attribute__((unused)) abstract_register_size(unsigned int width)
 {
        switch (width) {
                case 32:
@@ -807,12 +807,12 @@ static int execute_abstract_command(struct target *target, uint32_t command)
        return ERROR_OK;
 }
 
-static riscv_reg_t read_abstract_arg(struct target *target, unsigned index,
-               unsigned size_bits)
+static riscv_reg_t read_abstract_arg(struct target *target, unsigned int index,
+               unsigned int size_bits)
 {
        riscv_reg_t value = 0;
        uint32_t v;
-       unsigned offset = index * size_bits / 32;
+       unsigned int offset = index * size_bits / 32;
        switch (size_bits) {
                default:
                        LOG_ERROR("Unsupported size: %d bits", size_bits);
@@ -828,10 +828,10 @@ static riscv_reg_t read_abstract_arg(struct target *target, unsigned index,
        return value;
 }
 
-static int write_abstract_arg(struct target *target, unsigned index,
-               riscv_reg_t value, unsigned size_bits)
+static int write_abstract_arg(struct target *target, unsigned int index,
+               riscv_reg_t value, unsigned int size_bits)
 {
-       unsigned offset = index * size_bits / 32;
+       unsigned int offset = index * size_bits / 32;
        switch (size_bits) {
                default:
                        LOG_ERROR("Unsupported size: %d bits", size_bits);
@@ -849,7 +849,7 @@ static int write_abstract_arg(struct target *target, unsigned index,
  * @par size in bits
  */
 static uint32_t access_register_command(struct target *target, uint32_t number,
-               unsigned size, uint32_t flags)
+               unsigned int size, uint32_t flags)
 {
        uint32_t command = set_field(0, DM_COMMAND_CMDTYPE, 0);
        switch (size) {
@@ -891,7 +891,7 @@ static uint32_t access_register_command(struct target *target, uint32_t number,
 }
 
 static int register_read_abstract(struct target *target, uint64_t *value,
-               uint32_t number, unsigned size)
+               uint32_t number, unsigned int size)
 {
        RISCV013_INFO(info);
 
@@ -929,7 +929,7 @@ static int register_read_abstract(struct target *target, uint64_t *value,
 }
 
 static int register_write_abstract(struct target *target, uint32_t number,
-               uint64_t value, unsigned size)
+               uint64_t value, unsigned int size)
 {
        RISCV013_INFO(info);
 
@@ -968,7 +968,7 @@ static int register_write_abstract(struct target *target, uint32_t number,
  * Sets the AAMSIZE field of a memory access abstract command based on
  * the width (bits).
  */
-static uint32_t abstract_memory_size(unsigned width)
+static uint32_t abstract_memory_size(unsigned int width)
 {
        switch (width) {
                case 8:
@@ -991,7 +991,7 @@ static uint32_t abstract_memory_size(unsigned width)
  * Creates a memory access abstract command.
  */
 static uint32_t access_memory_command(struct target *target, bool virtual,
-               unsigned width, bool postincrement, bool write)
+               unsigned int width, bool postincrement, bool write)
 {
        uint32_t command = set_field(0, AC_ACCESS_MEMORY_CMDTYPE, 2);
        command = set_field(command, AC_ACCESS_MEMORY_AAMVIRTUAL, virtual);
@@ -1134,7 +1134,7 @@ typedef struct {
 static int scratch_reserve(struct target *target,
                scratch_mem_t *scratch,
                struct riscv_program *program,
-               unsigned size_bytes)
+               unsigned int size_bytes)
 {
        riscv_addr_t alignment = 1;
        while (alignment < size_bytes)
@@ -1166,7 +1166,7 @@ static int scratch_reserve(struct target *target,
                return ERROR_FAIL;
 
        /* Allow for ebreak at the end of the program. */
-       unsigned program_size = (program->instruction_count + 1) * 4;
+       unsigned int program_size = (program->instruction_count + 1) * 4;
        scratch->hart_address = (info->progbuf_address + program_size + alignment - 1) &
                ~(alignment - 1);
        if ((info->progbuf_writable == YNM_YES) &&
@@ -1271,7 +1271,7 @@ static int scratch_write64(struct target *target, scratch_mem_t *scratch,
 }
 
 /** Return register size in bits. */
-static unsigned register_size(struct target *target, unsigned number)
+static unsigned int register_size(struct target *target, unsigned int number)
 {
        /* If reg_cache hasn't been initialized yet, make a guess. We need this for
         * when this function is called during examine(). */
@@ -1281,7 +1281,7 @@ static unsigned register_size(struct target *target, unsigned number)
                return riscv_xlen(target);
 }
 
-static bool has_sufficient_progbuf(struct target *target, unsigned size)
+static bool has_sufficient_progbuf(struct target *target, unsigned int size)
 {
        RISCV013_INFO(info);
        RISCV_INFO(r);
@@ -1293,7 +1293,7 @@ static bool has_sufficient_progbuf(struct target *target, unsigned size)
  * Immediately write the new value to the requested register. This mechanism
  * bypasses any caches.
  */
-static int register_write_direct(struct target *target, unsigned number,
+static int register_write_direct(struct target *target, unsigned int number,
                uint64_t value)
 {
        LOG_DEBUG("{%d} %s <- 0x%" PRIx64, riscv_current_hartid(target),
@@ -1834,7 +1834,7 @@ static int riscv013_hart_count(struct target *target)
 }
 
 /* Try to find out the widest memory access size depending on the selected memory access methods. */
-static unsigned riscv013_data_bits(struct target *target)
+static unsigned int riscv013_data_bits(struct target *target)
 {
        RISCV013_INFO(info);
        RISCV_INFO(r);
@@ -1904,12 +1904,12 @@ static COMMAND_HELPER(riscv013_print_info, struct target *target)
 }
 
 static int prep_for_vector_access(struct target *target, uint64_t *vtype,
-               uint64_t *vl, unsigned *debug_vl)
+               uint64_t *vl, unsigned int *debug_vl)
 {
        RISCV_INFO(r);
        /* TODO: this continuous save/restore is terrible for performance. */
        /* Write vtype and vl. */
-       unsigned encoded_vsew;
+       unsigned int encoded_vsew;
        switch (riscv_xlen(target)) {
                case 32:
                        encoded_vsew = 2;
@@ -1965,12 +1965,12 @@ static int riscv013_get_register_buf(struct target *target,
                return ERROR_FAIL;
 
        uint64_t vtype, vl;
-       unsigned debug_vl;
+       unsigned int debug_vl;
        if (prep_for_vector_access(target, &vtype, &vl, &debug_vl) != ERROR_OK)
                return ERROR_FAIL;
 
-       unsigned vnum = regno - GDB_REGNO_V0;
-       unsigned xlen = riscv_xlen(target);
+       unsigned int vnum = regno - GDB_REGNO_V0;
+       unsigned int xlen = riscv_xlen(target);
 
        struct riscv_program program;
        riscv_program_init(&program, target);
@@ -1978,7 +1978,7 @@ static int riscv013_get_register_buf(struct target *target,
        riscv_program_insert(&program, vslide1down_vx(vnum, vnum, S0, true));
 
        int result = ERROR_OK;
-       for (unsigned i = 0; i < debug_vl; i++) {
+       for (unsigned int i = 0; i < debug_vl; i++) {
                /* Executing the program might result in an exception if there is some
                 * issue with the vector implementation/instructions we're using. If that
                 * happens, attempt to restore as usual. We may have clobbered the
@@ -2024,18 +2024,18 @@ static int riscv013_set_register_buf(struct target *target,
                return ERROR_FAIL;
 
        uint64_t vtype, vl;
-       unsigned debug_vl;
+       unsigned int debug_vl;
        if (prep_for_vector_access(target, &vtype, &vl, &debug_vl) != ERROR_OK)
                return ERROR_FAIL;
 
-       unsigned vnum = regno - GDB_REGNO_V0;
-       unsigned xlen = riscv_xlen(target);
+       unsigned int vnum = regno - GDB_REGNO_V0;
+       unsigned int xlen = riscv_xlen(target);
 
        struct riscv_program program;
        riscv_program_init(&program, target);
        riscv_program_insert(&program, vslide1down_vx(vnum, vnum, S0, true));
        int result = ERROR_OK;
-       for (unsigned i = 0; i < debug_vl; i++) {
+       for (unsigned int i = 0; i < debug_vl; i++) {
                if (register_write_direct(target, GDB_REGNO_S0,
                                        buf_get_u64(value, xlen * i, xlen)) != ERROR_OK)
                        return ERROR_FAIL;
@@ -2476,7 +2476,7 @@ static int execute_fence(struct target *target)
 }
 
 static void log_memory_access(target_addr_t address, uint64_t value,
-               unsigned size_bytes, bool read)
+               unsigned int size_bytes, bool read)
 {
        if (debug_level < LOG_LVL_DEBUG)
                return;
@@ -2524,7 +2524,7 @@ static int read_memory_bus_word(struct target *target, target_addr_t address,
 static target_addr_t sb_read_address(struct target *target)
 {
        RISCV013_INFO(info);
-       unsigned sbasize = get_field(info->sbcs, DM_SBCS_SBASIZE);
+       unsigned int sbasize = get_field(info->sbcs, DM_SBCS_SBASIZE);
        target_addr_t address = 0;
        uint32_t v;
        if (sbasize > 32) {
@@ -2717,7 +2717,7 @@ static int read_memory_bus_v1(struct target *target, target_addr_t address,
                for (uint32_t i = (next_address - address) / size; i < count - 1; i++) {
                        for (int j = (size - 1) / 4; j >= 0; j--) {
                                uint32_t value;
-                               unsigned attempt = 0;
+                               unsigned int attempt = 0;
                                while (1) {
                                        if (attempt++ > 100) {
                                                LOG_ERROR("DMI keeps being busy in while reading memory just past " TARGET_ADDR_FMT,
@@ -2745,7 +2745,7 @@ static int read_memory_bus_v1(struct target *target, target_addr_t address,
                uint32_t sbcs_read = 0;
                if (count > 1) {
                        uint32_t value;
-                       unsigned attempt = 0;
+                       unsigned int attempt = 0;
                        while (1) {
                                if (attempt++ > 100) {
                                        LOG_ERROR("DMI keeps being busy in while reading memory just past " TARGET_ADDR_FMT,
@@ -2793,7 +2793,7 @@ static int read_memory_bus_v1(struct target *target, target_addr_t address,
                        continue;
                }
 
-               unsigned error = get_field(sbcs_read, DM_SBCS_SBERROR);
+               unsigned int error = get_field(sbcs_read, DM_SBCS_SBERROR);
                if (error == 0) {
                        next_address = end_address;
                } else {
@@ -2959,7 +2959,7 @@ static int read_memory_abstract(struct target *target, target_addr_t address,
        memset(buffer, 0, count * size);
 
        /* Convert the size (bytes) to width (bits) */
-       unsigned width = size << 3;
+       unsigned int width = size << 3;
 
        /* Create the command (physical address, postincrement, read) */
        uint32_t command = access_memory_command(target, false, width, use_aampostincrement, false);
@@ -3035,7 +3035,7 @@ static int write_memory_abstract(struct target *target, target_addr_t address,
                          size, address);
 
        /* Convert the size (bytes) to width (bits) */
-       unsigned width = size << 3;
+       unsigned int width = size << 3;
 
        /* Create the command (physical address, postincrement, write) */
        uint32_t command = access_memory_command(target, false, width, use_aampostincrement, true);
@@ -3145,7 +3145,7 @@ static int read_memory_progbuf_inner(struct target *target, target_addr_t addres
 
        /* read_addr is the next address that the hart will read from, which is the
         * value in s0. */
-       unsigned index = 2;
+       unsigned int index = 2;
        while (index < count) {
                riscv_addr_t read_addr = address + index * increment;
                LOG_DEBUG("i=%d, count=%d, read_addr=0x%" PRIx64, index, count, read_addr);
@@ -3162,8 +3162,8 @@ static int read_memory_progbuf_inner(struct target *target, target_addr_t addres
                if (!batch)
                        return ERROR_FAIL;
 
-               unsigned reads = 0;
-               for (unsigned j = index; j < count; j++) {
+               unsigned int reads = 0;
+               for (unsigned int j = index; j < count; j++) {
                        if (size > 4)
                                riscv_batch_add_dmi_read(batch, DM_DATA1);
                        riscv_batch_add_dmi_read(batch, DM_DATA0);
@@ -3186,8 +3186,8 @@ static int read_memory_progbuf_inner(struct target *target, target_addr_t addres
                                return ERROR_FAIL;
                info->cmderr = get_field(abstractcs, DM_ABSTRACTCS_CMDERR);
 
-               unsigned next_index;
-               unsigned ignore_last = 0;
+               unsigned int next_index;
+               unsigned int ignore_last = 0;
                switch (info->cmderr) {
                        case CMDERR_NONE:
                                LOG_DEBUG("successful (partial?) memory read");
@@ -3256,9 +3256,9 @@ static int read_memory_progbuf_inner(struct target *target, target_addr_t addres
 
                /* Now read whatever we got out of the batch. */
                dmi_status_t status = DMI_STATUS_SUCCESS;
-               unsigned read = 0;
+               unsigned int read = 0;
                assert(index >= 2);
-               for (unsigned j = index - 2; j < index + reads; j++) {
+               for (unsigned int j = index - 2; j < index + reads; j++) {
                        assert(j < count);
                        LOG_DEBUG("index=%d, reads=%d, next_index=%d, ignore_last=%d, j=%d",
                                index, reads, next_index, ignore_last, j);
@@ -3867,9 +3867,9 @@ static int write_memory_progbuf(struct target *target, target_addr_t address,
                        goto error;
 
                /* To write another word, we put it in S1 and execute the program. */
-               unsigned start = (cur_addr - address) / size;
-               for (unsigned i = start; i < count; ++i) {
-                       unsigned offset = size*i;
+               unsigned int start = (cur_addr - address) / size;
+               for (unsigned int i = start; i < count; ++i) {
+                       unsigned int offset = size * i;
                        const uint8_t *t_buffer = buffer + offset;
 
                        uint64_t value = buf_get_u64(t_buffer, 0, 8 * size);
@@ -4160,18 +4160,18 @@ static int select_prepped_harts(struct target *target, bool *use_hasel)
        }
 
        assert(dm->hart_count);
-       unsigned hawindow_count = (dm->hart_count + 31) / 32;
+       unsigned int hawindow_count = (dm->hart_count + 31) / 32;
        uint32_t hawindow[hawindow_count];
 
        memset(hawindow, 0, sizeof(uint32_t) * hawindow_count);
 
        target_list_t *entry;
-       unsigned total_selected = 0;
+       unsigned int total_selected = 0;
        list_for_each_entry(entry, &dm->target_list, list) {
                struct target *t = entry->target;
                struct riscv_info *r = riscv_info(t);
                riscv013_info_t *info = get_info(t);
-               unsigned index = info->index;
+               unsigned int index = info->index;
                LOG_DEBUG("index=%d, coreid=%d, prepped=%d", index, t->coreid, r->prepped);
                r->selected = r->prepped;
                if (r->prepped) {
@@ -4188,7 +4188,7 @@ static int select_prepped_harts(struct target *target, bool *use_hasel)
                return ERROR_OK;
        }
 
-       for (unsigned i = 0; i < hawindow_count; i++) {
+       for (unsigned int i = 0; i < hawindow_count; i++) {
                if (dmi_write(target, DM_HAWINDOWSEL, i) != ERROR_OK)
                        return ERROR_FAIL;
                if (dmi_write(target, DM_HAWINDOW, hawindow[i]) != ERROR_OK)
@@ -4346,7 +4346,7 @@ static enum riscv_halt_reason riscv013_halt_reason(struct target *target)
        return RISCV_HALT_UNKNOWN;
 }
 
-int riscv013_write_debug_buffer(struct target *target, unsigned index, riscv_insn_t data)
+int riscv013_write_debug_buffer(struct target *target, unsigned int index, riscv_insn_t data)
 {
        dm013_info_t *dm = get_dm(target);
        if (!dm)
@@ -4361,7 +4361,7 @@ int riscv013_write_debug_buffer(struct target *target, unsigned index, riscv_ins
        return ERROR_OK;
 }
 
-riscv_insn_t riscv013_read_debug_buffer(struct target *target, unsigned index)
+riscv_insn_t riscv013_read_debug_buffer(struct target *target, unsigned int index)
 {
        uint32_t value;
        dmi_read(target, &value, DM_PROGBUF0 + index);
index 511a3c6c3264c88951af2d98bee84366f86592ef..99f40a7393702a08d72d68ed75713b327926871f 100644 (file)
@@ -476,7 +476,7 @@ static void riscv_free_registers(struct target *target)
                if (target->reg_cache->reg_list) {
                        free(target->reg_cache->reg_list[0].arch_info);
                        /* Free the ones we allocated separately. */
-                       for (unsigned i = GDB_REGNO_COUNT; i < target->reg_cache->num_regs; i++)
+                       for (unsigned int i = GDB_REGNO_COUNT; i < target->reg_cache->num_regs; i++)
                                free(target->reg_cache->reg_list[i].arch_info);
                        for (unsigned int i = 0; i < target->reg_cache->num_regs; i++)
                                free(target->reg_cache->reg_list[i].value);
@@ -1583,7 +1583,7 @@ static int riscv_address_translate(struct target *target,
        if (result != ERROR_OK)
                return result;
 
-       unsigned xlen = riscv_xlen(target);
+       unsigned int xlen = riscv_xlen(target);
        mode = get_field(satp_value, RISCV_SATP_MODE(xlen));
        switch (mode) {
                case SATP_MODE_SV32:
@@ -1927,7 +1927,7 @@ static int riscv_run_algorithm(struct target *target, int num_mem_params,
                                GDB_REGNO_PC,
                                GDB_REGNO_MSTATUS, GDB_REGNO_MEPC, GDB_REGNO_MCAUSE,
                        };
-                       for (unsigned i = 0; i < ARRAY_SIZE(regnums); i++) {
+                       for (unsigned int i = 0; i < ARRAY_SIZE(regnums); i++) {
                                enum gdb_regno regno = regnums[i];
                                riscv_reg_t reg_value;
                                if (riscv_get_register(target, &reg_value, regno) != ERROR_OK)
@@ -2007,8 +2007,8 @@ static int riscv_checksum_memory(struct target *target,
 
        static const uint8_t *crc_code;
 
-       unsigned xlen = riscv_xlen(target);
-       unsigned crc_code_size;
+       unsigned int xlen = riscv_xlen(target);
+       unsigned int crc_code_size;
        if (xlen == 32) {
                crc_code = riscv32_crc_code;
                crc_code_size = sizeof(riscv32_crc_code);
@@ -2187,8 +2187,8 @@ int riscv_openocd_poll(struct target *target)
        int halted_hart = -1;
 
        if (target->smp) {
-               unsigned should_remain_halted = 0;
-               unsigned should_resume = 0;
+               unsigned int should_remain_halted = 0;
+               unsigned int should_resume = 0;
                struct target_list *list;
                foreach_smp_target(list, target->smp_targets) {
                        struct target *t = list->target;
@@ -2436,8 +2436,8 @@ static int parse_ranges(struct list_head *ranges, const char *tcl_arg, const cha
        /* For backward compatibility, allow multiple parameters within one TCL argument, separated by ',' */
        char *arg = strtok(args, ",");
        while (arg) {
-               unsigned low = 0;
-               unsigned high = 0;
+               unsigned int low = 0;
+               unsigned int high = 0;
                char *name = NULL;
 
                char *dash = strchr(arg, '-');
@@ -3055,7 +3055,7 @@ static const struct command_registration riscv_command_handlers[] = {
        COMMAND_REGISTRATION_DONE
 };
 
-static unsigned riscv_xlen_nonconst(struct target *target)
+static unsigned int riscv_xlen_nonconst(struct target *target)
 {
        return riscv_xlen(target);
 }
@@ -3194,7 +3194,7 @@ static int riscv_step_rtos_hart(struct target *target)
 bool riscv_supports_extension(struct target *target, char letter)
 {
        RISCV_INFO(r);
-       unsigned num;
+       unsigned int num;
        if (letter >= 'a' && letter <= 'z')
                num = letter - 'a';
        else if (letter >= 'A' && letter <= 'Z')
@@ -3204,7 +3204,7 @@ bool riscv_supports_extension(struct target *target, char letter)
        return r->misa & BIT(num);
 }
 
-unsigned riscv_xlen(const struct target *target)
+unsigned int riscv_xlen(const struct target *target)
 {
        RISCV_INFO(r);
        return r->xlen;
@@ -3778,7 +3778,7 @@ static struct reg_arch_type riscv_reg_arch_type = {
 };
 
 struct csr_info {
-       unsigned number;
+       unsigned int number;
        const char *name;
 };
 
@@ -3958,7 +3958,7 @@ int riscv_init_registers(struct target *target)
        };
        /* encoding.h does not contain the registers in sorted order. */
        qsort(csr_info, ARRAY_SIZE(csr_info), sizeof(*csr_info), cmp_csr_info);
-       unsigned csr_info_index = 0;
+       unsigned int csr_info_index = 0;
 
        int custom_within_range = 0;
 
@@ -4213,7 +4213,7 @@ int riscv_init_registers(struct target *target)
                } else if (number >= GDB_REGNO_CSR0 && number <= GDB_REGNO_CSR4095) {
                        r->group = "csr";
                        r->feature = &feature_csr;
-                       unsigned csr_number = number - GDB_REGNO_CSR0;
+                       unsigned int csr_number = number - GDB_REGNO_CSR0;
 
                        while (csr_info[csr_info_index].number < csr_number &&
                                        csr_info_index < ARRAY_SIZE(csr_info) - 1) {
@@ -4376,7 +4376,7 @@ int riscv_init_registers(struct target *target)
 
                        range_list_t *range = list_first_entry(&info->expose_custom, range_list_t, list);
 
-                       unsigned custom_number = range->low + custom_within_range;
+                       unsigned int custom_number = range->low + custom_within_range;
 
                        r->group = "custom";
                        r->feature = &feature_custom;
index aba0864e6d0203dd183edc3f94ffc9eb0acfd2a8..6cb2709de81c64d0107571c74ec99b6617a69303 100644 (file)
@@ -61,7 +61,7 @@ enum riscv_halt_reason {
 
 typedef struct {
        struct target *target;
-       unsigned custom_number;
+       unsigned int custom_number;
 } riscv_reg_info_t;
 
 #define RISCV_SAMPLE_BUF_TIMESTAMP_BEFORE      0x80
@@ -90,7 +90,7 @@ typedef struct {
 struct riscv_info {
        unsigned int common_magic;
 
-       unsigned dtm_version;
+       unsigned int dtm_version;
 
        struct command_context *cmd_ctx;
        void *version_specific;
@@ -159,9 +159,9 @@ struct riscv_info {
        int (*halt_go)(struct target *target);
        int (*on_step)(struct target *target);
        enum riscv_halt_reason (*halt_reason)(struct target *target);
-       int (*write_debug_buffer)(struct target *target, unsigned index,
+       int (*write_debug_buffer)(struct target *target, unsigned int index,
                        riscv_insn_t d);
-       riscv_insn_t (*read_debug_buffer)(struct target *target, unsigned index);
+       riscv_insn_t (*read_debug_buffer)(struct target *target, unsigned int index);
        int (*execute_debug_buffer)(struct target *target);
        int (*dmi_write_u64_bits)(struct target *target);
        void (*fill_dmi_write_u64)(struct target *target, char *buf, int a, uint64_t d);
@@ -184,7 +184,7 @@ struct riscv_info {
 
        /* How many harts are attached to the DM that this target is attached to? */
        int (*hart_count)(struct target *target);
-       unsigned (*data_bits)(struct target *target);
+       unsigned int (*data_bits)(struct target *target);
 
        COMMAND_HELPER((*print_info), struct target *target);
 
@@ -239,14 +239,14 @@ typedef struct {
 typedef struct {
        const char *name;
        int level;
-       unsigned va_bits;
-       unsigned pte_shift;
-       unsigned vpn_shift[PG_MAX_LEVEL];
-       unsigned vpn_mask[PG_MAX_LEVEL];
-       unsigned pte_ppn_shift[PG_MAX_LEVEL];
-       unsigned pte_ppn_mask[PG_MAX_LEVEL];
-       unsigned pa_ppn_shift[PG_MAX_LEVEL];
-       unsigned pa_ppn_mask[PG_MAX_LEVEL];
+       unsigned int va_bits;
+       unsigned int pte_shift;
+       unsigned int vpn_shift[PG_MAX_LEVEL];
+       unsigned int vpn_mask[PG_MAX_LEVEL];
+       unsigned int pte_ppn_shift[PG_MAX_LEVEL];
+       unsigned int pte_ppn_mask[PG_MAX_LEVEL];
+       unsigned int pa_ppn_shift[PG_MAX_LEVEL];
+       unsigned int pa_ppn_mask[PG_MAX_LEVEL];
 } virt2phys_info_t;
 
 /* Wall-clock timeout for a command/access. Settable via RISC-V Target commands.*/
@@ -307,7 +307,7 @@ int riscv_openocd_deassert_reset(struct target *target);
 bool riscv_supports_extension(struct target *target, char letter);
 
 /* Returns XLEN for the given (or current) hart. */
-unsigned riscv_xlen(const struct target *target);
+unsigned int riscv_xlen(const struct target *target);
 int riscv_xlen_of_hart(const struct target *target);
 
 /* Sets the current hart, which is the hart that will actually be used when