Make OpenOCD build using -Og.
With -Og gcc doesn't perform as many optimizations, and as a result warns about some code that it wouldn't otherwise warn about. These fixes all assign values to otherwise uninitialized variables. Change-Id: I9a6ea5eadd73673891ecfec568b8b00d78b596a5 Signed-off-by: Tim Newsome <tim@sifive.com> Reviewed-on: http://openocd.zylin.com/3779 Tested-by: jenkins Reviewed-by: Tomas Vanek <vanekt@fbl.cz> Reviewed-by: Freddie Chopin <freddie.chopin@gmail.com>gitignore-build
parent
f6b8a4ea65
commit
ca19c82d94
|
@ -1340,7 +1340,7 @@ static int kinetis_make_ram_ready(struct target *target)
|
|||
static int kinetis_write_sections(struct flash_bank *bank, const uint8_t *buffer,
|
||||
uint32_t offset, uint32_t count)
|
||||
{
|
||||
int result;
|
||||
int result = ERROR_OK;
|
||||
struct kinetis_flash_bank *kinfo = bank->driver_priv;
|
||||
uint8_t *buffer_aligned = NULL;
|
||||
/*
|
||||
|
|
|
@ -230,7 +230,8 @@ static int niietcm4_uopstatus_check(struct flash_bank *bank)
|
|||
static int niietcm4_dump_uflash_page(struct flash_bank *bank, uint32_t *dump, int page_num, int mem_type)
|
||||
{
|
||||
struct target *target = bank->target;
|
||||
int i, retval;
|
||||
int i;
|
||||
int retval = ERROR_OK;
|
||||
|
||||
uint32_t uflash_cmd;
|
||||
if (mem_type == INFO_MEM_TYPE)
|
||||
|
@ -265,7 +266,8 @@ static int niietcm4_dump_uflash_page(struct flash_bank *bank, uint32_t *dump, in
|
|||
static int niietcm4_load_uflash_page(struct flash_bank *bank, uint32_t *dump, int page_num, int mem_type)
|
||||
{
|
||||
struct target *target = bank->target;
|
||||
int i, retval;
|
||||
int i;
|
||||
int retval = ERROR_OK;
|
||||
|
||||
uint32_t uflash_cmd;
|
||||
if (mem_type == INFO_MEM_TYPE)
|
||||
|
|
|
@ -998,11 +998,6 @@ static int xmc4xxx_flash_unprotect(struct flash_bank *bank, int32_t level)
|
|||
uint32_t addr;
|
||||
int res;
|
||||
|
||||
if ((level < 0) || (level > 1)) {
|
||||
LOG_ERROR("Invalid user level. Must be 0-1");
|
||||
return ERROR_FAIL;
|
||||
}
|
||||
|
||||
switch (level) {
|
||||
case 0:
|
||||
addr = UCB0_BASE;
|
||||
|
@ -1010,6 +1005,9 @@ static int xmc4xxx_flash_unprotect(struct flash_bank *bank, int32_t level)
|
|||
case 1:
|
||||
addr = UCB1_BASE;
|
||||
break;
|
||||
default:
|
||||
LOG_ERROR("Invalid user level. Must be 0-1");
|
||||
return ERROR_FAIL;
|
||||
}
|
||||
|
||||
res = xmc4xxx_erase_sector(bank, addr, true);
|
||||
|
|
|
@ -2667,7 +2667,7 @@ static int xscale_analyze_trace(struct target *target, struct command_context *c
|
|||
struct xscale_common *xscale = target_to_xscale(target);
|
||||
struct xscale_trace_data *trace_data = xscale->trace.data;
|
||||
int i, retval;
|
||||
uint32_t breakpoint_pc;
|
||||
uint32_t breakpoint_pc = 0;
|
||||
struct arm_instruction instruction;
|
||||
uint32_t current_pc = 0;/* initialized when address determined */
|
||||
|
||||
|
|
Loading…
Reference in New Issue