Skip to content

Commit be99a22

Browse files
committed
Fix some warnings
Former-commit-id: 6db0177
1 parent 5e637f0 commit be99a22

File tree

4 files changed

+12
-11
lines changed

4 files changed

+12
-11
lines changed

cores/arduino/FspTimer.h

Lines changed: 4 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -55,13 +55,12 @@ class GPTimer {
5555
ext_cfg.gtior_setting.gtior = 0U;
5656

5757
cfg.p_extend = &ext_cfg;
58-
58+
59+
memset(&ctrl, 0, sizeof(gpt_instance_ctrl_t));
5960
}
60-
61-
gpt_instance_ctrl_t ctrl = { 0 };
62-
gpt_extended_cfg_t ext_cfg = { 0 };
63-
6461

62+
gpt_instance_ctrl_t ctrl;
63+
gpt_extended_cfg_t ext_cfg;
6564
};
6665

6766
class AGTimer {

cores/arduino/Serial.cpp

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -326,4 +326,5 @@ size_t UART::write_raw(uint8_t* c, size_t len) {
326326
R_BSP_SoftwareDelay(1, BSP_DELAY_UNITS_MILLISECONDS);
327327
i++;
328328
}
329+
return len;
329330
}

cores/arduino/cm_backtrace/cm_backtrace.c

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -262,7 +262,7 @@ static void dump_stack(uint32_t stack_start_addr, size_t stack_size, uint32_t *s
262262
}
263263
cmb_println(print_info[PRINT_THREAD_STACK_INFO]);
264264
for (; (uint32_t) stack_pointer < stack_start_addr + stack_size; stack_pointer++) {
265-
cmb_println(" addr: %08x data: %08x", stack_pointer, *stack_pointer);
265+
cmb_println(" addr: %08x data: %08x", (unsigned int)stack_pointer, (unsigned int)*stack_pointer);
266266
}
267267
cmb_println("====================================");
268268
}
@@ -397,7 +397,7 @@ void cm_backtrace_assert(uint32_t sp) {
397397
uint32_t cur_stack_pointer = cmb_get_sp();
398398
#endif
399399

400-
cmb_println("");
400+
cmb_println("\n");
401401
cm_backtrace_firmware_info();
402402

403403
#ifdef CMB_USING_OS_PLATFORM
@@ -584,7 +584,7 @@ void cm_backtrace_fault(uint32_t fault_handler_lr, uint32_t fault_handler_sp) {
584584

585585
on_fault = true;
586586

587-
cmb_println("");
587+
cmb_println("\n");
588588
cm_backtrace_firmware_info();
589589

590590
#ifdef CMB_USING_OS_PLATFORM

cores/arduino/main.cpp

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -36,7 +36,7 @@ void unsecure_registers() {
3636
uint32_t interrupt_security_state[BSP_ICU_VECTOR_MAX_ENTRIES / BSP_PRV_BITS_PER_WORD];
3737
memset(&interrupt_security_state, UINT8_MAX, sizeof(interrupt_security_state));
3838

39-
for (uint32_t i = 0U; i < BSP_ICU_VECTOR_MAX_ENTRIES; i++)
39+
for (size_t i = 0U; i < BSP_ICU_VECTOR_MAX_ENTRIES; i++)
4040
{
4141
/* This is a secure vector. Clear the associated bit. */
4242
uint32_t index = i / BSP_PRV_BITS_PER_WORD;
@@ -47,7 +47,7 @@ void unsecure_registers() {
4747
/* The Secure Attribute managed within the ARM CPU NVIC must match the security attribution of IELSEn
4848
* (Reference section 13.2.9 in the RA6M4 manual R01UH0890EJ0050). */
4949
uint32_t volatile * p_icusarg = &R_CPSCU->ICUSARG;
50-
for (uint32_t i = 0U; i < BSP_ICU_VECTOR_MAX_ENTRIES / BSP_PRV_BITS_PER_WORD; i++)
50+
for (size_t i = 0U; i < BSP_ICU_VECTOR_MAX_ENTRIES / BSP_PRV_BITS_PER_WORD; i++)
5151
{
5252
p_icusarg[i] = interrupt_security_state[i];
5353
NVIC->ITNS[i] = interrupt_security_state[i];
@@ -78,7 +78,7 @@ void arduino_main(void)
7878

7979
__disable_irq();
8080
irq_vector_table = (volatile uint32_t *)APPLICATION_VECTOR_TABLE_ADDRESS_RAM;
81-
int _i;
81+
size_t _i;
8282
for (_i=0; _i<BSP_CORTEX_VECTOR_TABLE_ENTRIES; _i++) {
8383
*(irq_vector_table + _i) = (uint32_t)__VECTOR_TABLE[_i];
8484
}
@@ -116,6 +116,7 @@ void arduino_main(void)
116116
#ifdef BACKTRACE_SUPPORT
117117
/* for printf compatibility */
118118
extern "C" __attribute__((weak)) int _write (int fhdl, const void *buf, size_t count) {
119+
(void)fhdl;
119120
Serial.write_raw((uint8_t*)buf, count);
120121
}
121122
#endif

0 commit comments

Comments
 (0)