From b5c952e6b64b12970e747875b0bc894b57a5eca2 Mon Sep 17 00:00:00 2001 From: Joel Sherrill Date: Fri, 10 Oct 2025 13:03:05 -0500 Subject: [PATCH] arm/rtl22xx: Address unused parameter warnings Add "(void) param;" annotation to address unused parameter warnings. Found with GCC's warning -Wunused-parameter. --- bsps/arm/rtl22xx/clock/clockdrv.c | 12 ++++++++++++ bsps/arm/rtl22xx/console/uart.c | 13 +++++++++++++ bsps/arm/rtl22xx/irq/irq.c | 17 +++++++++++++++++ 3 files changed, 42 insertions(+) diff --git a/bsps/arm/rtl22xx/clock/clockdrv.c b/bsps/arm/rtl22xx/clock/clockdrv.c index 2d9fc1e07c..c44c770879 100644 --- a/bsps/arm/rtl22xx/clock/clockdrv.c +++ b/bsps/arm/rtl22xx/clock/clockdrv.c @@ -30,11 +30,15 @@ static rtems_timecounter_simple lpc22xx_tc; static uint32_t lpc22xx_tc_get(rtems_timecounter_simple *tc) { + (void) tc; + return T0TC; } static bool lpc22xx_tc_is_pending(rtems_timecounter_simple *tc) { + (void) tc; + return (T0IR & 0x1) != 0; } @@ -54,6 +58,8 @@ static uint32_t lpc22xx_tc_get_timecount(struct timecounter *tc) */ static void lpc22xx_tc_at_tick(rtems_timecounter_simple *tc) { + (void) tc; + if (!(T0IR & 0x01)) return; T0IR = 0x01; @@ -135,6 +141,8 @@ rtems_irq_connect_data clock_isr_data = { */ static void clock_isr_on(const rtems_irq_connect_data *unused) { + (void) unused; + T0IR&=0x01; } @@ -145,6 +153,8 @@ static void clock_isr_on(const rtems_irq_connect_data *unused) */ static void clock_isr_off(const rtems_irq_connect_data *unused) { + (void) unused; + T0IR=0x00; } @@ -156,6 +166,8 @@ static void clock_isr_off(const rtems_irq_connect_data *unused) */ static int clock_isr_is_on(const rtems_irq_connect_data *irq) { + (void) irq; + return T0IR & 0x01; /* MR0 mask */ } diff --git a/bsps/arm/rtl22xx/console/uart.c b/bsps/arm/rtl22xx/console/uart.c index 7a723dedef..531ed21e37 100644 --- a/bsps/arm/rtl22xx/console/uart.c +++ b/bsps/arm/rtl22xx/console/uart.c @@ -116,6 +116,10 @@ console_tbl Console_Configuration_Ports[] = { */ static int uart_first_open(int major, int minor, void *arg) { + (void) major; + (void) minor; + (void) arg; + return 0; } @@ -126,6 +130,10 @@ static int uart_first_open(int major, int minor, void *arg) */ static int uart_last_close(int major, int minor, void *arg) { + (void) major; + (void) minor; + (void) arg; + return 0; } @@ -194,6 +202,8 @@ static ssize_t uart_write(int minor, const char *buf, size_t len) /* Set up the UART. */ static void uart_init(int minor) { + (void) minor; + #if 0 //init will be done in bspstart.c int baud=6; int mode =0x03; @@ -241,6 +251,9 @@ static void uart_write_polled(int minor, char c) /* This is for setting baud rate, bits, etc. */ static int uart_set_attributes(int minor, const struct termios *t) { + (void) minor; + (void) t; + return 0; } diff --git a/bsps/arm/rtl22xx/irq/irq.c b/bsps/arm/rtl22xx/irq/irq.c index f0df99a51a..40aad8f5ed 100644 --- a/bsps/arm/rtl22xx/irq/irq.c +++ b/bsps/arm/rtl22xx/irq/irq.c @@ -33,6 +33,9 @@ rtems_status_code bsp_interrupt_get_attributes( rtems_interrupt_attributes *attributes ) { + (void) vector; + (void) attributes; + return RTEMS_SUCCESSFUL; } @@ -41,6 +44,8 @@ rtems_status_code bsp_interrupt_is_pending( bool *pending ) { + (void) vector; + bsp_interrupt_assert(bsp_interrupt_is_valid_vector(vector)); bsp_interrupt_assert(pending != NULL); *pending = false; @@ -49,12 +54,16 @@ rtems_status_code bsp_interrupt_is_pending( rtems_status_code bsp_interrupt_raise(rtems_vector_number vector) { + (void) vector; + bsp_interrupt_assert(bsp_interrupt_is_valid_vector(vector)); return RTEMS_UNSATISFIED; } rtems_status_code bsp_interrupt_clear(rtems_vector_number vector) { + (void) vector; + bsp_interrupt_assert(bsp_interrupt_is_valid_vector(vector)); return RTEMS_UNSATISFIED; } @@ -64,6 +73,8 @@ rtems_status_code bsp_interrupt_vector_is_enabled( bool *enabled ) { + (void) vector; + bsp_interrupt_assert(bsp_interrupt_is_valid_vector(vector)); bsp_interrupt_assert(enabled != NULL); *enabled = false; @@ -89,6 +100,9 @@ rtems_status_code bsp_interrupt_set_priority( uint32_t priority ) { + (void) vector; + (void) priority; + bsp_interrupt_assert(bsp_interrupt_is_valid_vector(vector)); return RTEMS_UNSATISFIED; } @@ -98,6 +112,9 @@ rtems_status_code bsp_interrupt_get_priority( uint32_t *priority ) { + (void) vector; + (void) priority; + bsp_interrupt_assert(bsp_interrupt_is_valid_vector(vector)); bsp_interrupt_assert(priority != NULL); return RTEMS_UNSATISFIED;