From 68e65bf5a6d6d83638ed48e53df2974a62298b30 Mon Sep 17 00:00:00 2001 From: Joel Sherrill Date: Fri, 10 Oct 2025 14:08:48 -0500 Subject: [PATCH] bsps/powerpc/mpc8260ads: Address unused parameter warnings Add "(void) param;" annotation to address unused parameter warnings. Found with GCC's warning -Wunused-parameter. --- bsps/powerpc/mpc8260ads/console/console.c | 18 ++++++++++++++++++ bsps/powerpc/mpc8260ads/dev/console-generic.c | 12 ++++++++++++ bsps/powerpc/mpc8260ads/irq/irq.c | 19 +++++++++++++++++++ 3 files changed, 49 insertions(+) diff --git a/bsps/powerpc/mpc8260ads/console/console.c b/bsps/powerpc/mpc8260ads/console/console.c index ab3e455a12..7e343cdaa4 100644 --- a/bsps/powerpc/mpc8260ads/console/console.c +++ b/bsps/powerpc/mpc8260ads/console/console.c @@ -131,6 +131,8 @@ static rtems_status_code do_poll_read( void * arg ) { + (void) major; + rtems_libio_rw_args_t *rw_args = arg; int c; @@ -171,6 +173,8 @@ static rtems_status_code do_poll_write( void * arg ) { + (void) major; + rtems_libio_rw_args_t *rw_args = arg; uint32_t i; char cr ='\r'; @@ -219,6 +223,10 @@ rtems_device_driver console_initialize( void *arg ) { + (void) major; + (void) minor; + (void) arg; + rtems_status_code status; rtems_device_minor_number console_minor; @@ -355,6 +363,10 @@ rtems_device_driver console_open( #endif #else /* UARTS_USE_TERMIOS != 1 */ + (void) major; + (void) minor; + (void) arg; + /* no termios -- default to polled I/O */ sc = RTEMS_SUCCESSFUL; #endif /* UARTS_USE_TERMIOS != 1 */ @@ -372,6 +384,9 @@ rtems_device_driver console_close( void *arg ) { + (void) major; + (void) arg; + if ( minor > NUM_PORTS-1 ) return RTEMS_INVALID_NUMBER; @@ -433,6 +448,9 @@ rtems_device_driver console_control( void *arg ) { + (void) major; + (void) arg; + if ( minor > NUM_PORTS-1 ) return RTEMS_INVALID_NUMBER; diff --git a/bsps/powerpc/mpc8260ads/dev/console-generic.c b/bsps/powerpc/mpc8260ads/dev/console-generic.c index e74f98f55a..8ccf081efc 100644 --- a/bsps/powerpc/mpc8260ads/dev/console-generic.c +++ b/bsps/powerpc/mpc8260ads/dev/console-generic.c @@ -335,6 +335,8 @@ m8xx_uart_setAttributes( static void m8xx_scc1_interrupt_handler (rtems_irq_hdl_param unused) { + (void) unused; + /* * Buffer received? */ @@ -377,6 +379,8 @@ m8xx_scc1_interrupt_handler (rtems_irq_hdl_param unused) static void m8xx_scc2_interrupt_handler (rtems_irq_hdl_param unused) { + (void) unused; + /* * Buffer received? */ @@ -419,6 +423,8 @@ m8xx_scc2_interrupt_handler (rtems_irq_hdl_param unused) static void m8xx_scc3_interrupt_handler (rtems_irq_hdl_param unused) { + (void) unused; + /* * Buffer received? */ @@ -462,6 +468,8 @@ m8xx_scc3_interrupt_handler (rtems_irq_hdl_param unused) static void m8xx_scc4_interrupt_handler (rtems_irq_hdl_param unused) { + (void) unused; + /* * Buffer received? */ @@ -504,6 +512,8 @@ m8xx_scc4_interrupt_handler (rtems_irq_hdl_param unused) static void m8xx_smc1_interrupt_handler (rtems_irq_hdl_param unused) { + (void) unused; + /* * Buffer received? */ @@ -546,6 +556,8 @@ m8xx_smc1_interrupt_handler (rtems_irq_hdl_param unused) static void m8xx_smc2_interrupt_handler (rtems_irq_hdl_param unused) { + (void) unused; + /* * Buffer received? */ diff --git a/bsps/powerpc/mpc8260ads/irq/irq.c b/bsps/powerpc/mpc8260ads/irq/irq.c index 9a2cf1522d..ce784b2414 100644 --- a/bsps/powerpc/mpc8260ads/irq/irq.c +++ b/bsps/powerpc/mpc8260ads/irq/irq.c @@ -229,6 +229,8 @@ volatile unsigned int maxLoop = 0; */ static int C_dispatch_irq_handler (BSP_Exception_frame *frame, unsigned excNum) { + (void) frame; + register unsigned int irq; #if 0 register unsigned oldMask; /* old siu pic masks */ @@ -349,6 +351,9 @@ rtems_status_code bsp_interrupt_get_attributes( rtems_interrupt_attributes *attributes ) { + (void) vector; + (void) attributes; + return RTEMS_SUCCESSFUL; } @@ -357,6 +362,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; @@ -365,12 +372,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; } @@ -380,6 +391,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; @@ -419,6 +432,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; } @@ -428,6 +444,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;