diff --git a/src/target/aarch64.c b/src/target/aarch64.c index d1ff023d9..9539a6437 100644 --- a/src/target/aarch64.c +++ b/src/target/aarch64.c @@ -1095,7 +1095,7 @@ static int aarch64_post_debug_entry(struct target *target) LOG_DEBUG("System_register: %8.8" PRIx64, aarch64->system_control_reg); aarch64->system_control_reg_curr = aarch64->system_control_reg; - if (armv8->armv8_mmu.armv8_cache.info == -1) { + if (!armv8->armv8_mmu.armv8_cache.info_valid) { armv8_identify_cache(armv8); armv8_read_mpidr(armv8); } diff --git a/src/target/armv8.c b/src/target/armv8.c index 478a72b8d..51337a5d8 100644 --- a/src/target/armv8.c +++ b/src/target/armv8.c @@ -1303,7 +1303,7 @@ COMMAND_HANDLER(armv8_pauth_command) int armv8_handle_cache_info_command(struct command_invocation *cmd, struct armv8_cache_common *armv8_cache) { - if (armv8_cache->info == -1) { + if (!armv8_cache->info_valid) { command_print(cmd, "cache not yet identified"); return ERROR_OK; } @@ -1330,7 +1330,7 @@ int armv8_init_arch_info(struct target *target, struct armv8_common *armv8) armv8->common_magic = ARMV8_COMMON_MAGIC; armv8->armv8_mmu.armv8_cache.l2_cache = NULL; - armv8->armv8_mmu.armv8_cache.info = -1; + armv8->armv8_mmu.armv8_cache.info_valid = false; armv8->armv8_mmu.armv8_cache.flush_all_data_cache = NULL; armv8->armv8_mmu.armv8_cache.display_cache_info = NULL; return ERROR_OK; diff --git a/src/target/armv8.h b/src/target/armv8.h index 51b8b00cd..5bc1719df 100644 --- a/src/target/armv8.h +++ b/src/target/armv8.h @@ -151,7 +151,7 @@ struct armv8_arch_cache { }; struct armv8_cache_common { - int info; + bool info_valid; int loc; uint32_t iminline; uint32_t dminline; diff --git a/src/target/armv8_cache.c b/src/target/armv8_cache.c index 1c251beb9..7bf4dcd47 100644 --- a/src/target/armv8_cache.c +++ b/src/target/armv8_cache.c @@ -214,7 +214,7 @@ static int armv8_handle_inner_cache_info_command(struct command_invocation *cmd, { int cl; - if (armv8_cache->info == -1) { + if (!armv8_cache->info_valid) { command_print(cmd, "cache not yet identified"); return ERROR_OK; } @@ -262,7 +262,7 @@ static int armv8_flush_all_data(struct target *target) int retval = ERROR_FAIL; /* check that armv8_cache is correctly identify */ struct armv8_common *armv8 = target_to_armv8(target); - if (armv8->armv8_mmu.armv8_cache.info == -1) { + if (!armv8->armv8_mmu.armv8_cache.info_valid) { LOG_ERROR("trying to flush un-identified cache"); return retval; } @@ -288,7 +288,7 @@ static int armv8_flush_all_instruction(struct target *target) int retval = ERROR_FAIL; /* check that armv8_cache is correctly identify */ struct armv8_common *armv8 = target_to_armv8(target); - if (armv8->armv8_mmu.armv8_cache.info == -1) { + if (!armv8->armv8_mmu.armv8_cache.info_valid) { LOG_ERROR("trying to flush un-identified cache"); return retval; } @@ -459,7 +459,7 @@ int armv8_identify_cache(struct armv8_common *armv8) if (retval != ERROR_OK) goto done; - armv8->armv8_mmu.armv8_cache.info = 1; + armv8->armv8_mmu.armv8_cache.info_valid = true; /* if no l2 cache initialize l1 data cache flush function function */ if (!armv8->armv8_mmu.armv8_cache.flush_all_data_cache) {