This is an automated email from Gerrit. "Antonio Borneo <borneo.anto...@gmail.com>" just uploaded a new patch set to Gerrit, which you can find at https://review.openocd.org/c/openocd/+/9066
-- gerrit commit 3c9f4cce29490060fc3ee4400502baaf7c8d37be Author: Antonio Borneo <borneo.anto...@gmail.com> Date: Fri Aug 1 17:28:40 2025 +0200 flash: at91sam7: align format strings to types Remove the cast and use the correct conversion specifier. Change-Id: Idd9fae8cb8858e1f2f098544eb2eaa80bf0c5597 Signed-off-by: Antonio Borneo <borneo.anto...@gmail.com> diff --git a/src/flash/nor/at91sam7.c b/src/flash/nor/at91sam7.c index a124e49df2..093f7ea3e3 100644 --- a/src/flash/nor/at91sam7.c +++ b/src/flash/nor/at91sam7.c @@ -277,7 +277,7 @@ static void at91sam7_set_flash_mode(struct flash_bank *bank, int mode) if (at91sam7_info->mck_freq > 30000000ul) fws = 1; - LOG_DEBUG("fmcn[%i]: %i", bank->bank_number, (int)(fmcn)); + LOG_DEBUG("fmcn[%u]: %" PRIu32, bank->bank_number, fmcn); fmr = fmcn << 16 | fws << 8; target_write_u32(target, mc_fmr[bank->bank_number], fmr); } @@ -291,11 +291,11 @@ static uint32_t at91sam7_wait_status_busy(struct flash_bank *bank, uint32_t wait while ((!((status = at91sam7_get_flash_status(bank->target, bank->bank_number)) & waitbits)) && (timeout-- > 0)) { - LOG_DEBUG("status[%i]: 0x%" PRIx32, (int)bank->bank_number, status); + LOG_DEBUG("status[%u]: 0x%" PRIx32, bank->bank_number, status); alive_sleep(1); } - LOG_DEBUG("status[%i]: 0x%" PRIx32, bank->bank_number, status); + LOG_DEBUG("status[%u]: 0x%" PRIx32, bank->bank_number, status); if (status & 0x0C) { LOG_ERROR("status register: 0x%" PRIx32, status); @@ -319,7 +319,7 @@ static int at91sam7_flash_command(struct flash_bank *bank, uint8_t cmd, uint16_t fcr = (0x5A << 24) | ((pagen&0x3FF) << 8) | cmd; target_write_u32(target, mc_fcr[bank->bank_number], fcr); - LOG_DEBUG("Flash command: 0x%" PRIx32 ", flash bank: %i, page number: %u", + LOG_DEBUG("Flash command: 0x%" PRIx32 ", flash bank: %u, page number: %" PRIu16, fcr, bank->bank_number + 1, pagen); --