From c5ab69eaedb062153f33beb181d30ab835a61253 Mon Sep 17 00:00:00 2001 From: Joel Sherrill Date: Fri, 10 Oct 2025 14:06:54 -0500 Subject: [PATCH] bsps/powerpc/gen8x33: Address unused parameter warnings Add "(void) param;" annotation to address unused parameter warnings. Found with GCC's warning -Wunused-parameter. --- bsps/powerpc/gen83xx/dev/mpc83xx_i2cdrv.c | 3 +++ bsps/powerpc/gen83xx/dev/mpc83xx_spidrv.c | 3 +++ bsps/powerpc/gen83xx/include/tm27.h | 3 +++ bsps/powerpc/gen83xx/irq/irq.c | 19 +++++++++++++++++++ bsps/powerpc/gen83xx/spi/spi_init.c | 13 +++++++++++++ bsps/powerpc/gen83xx/start/bspstart.c | 3 +++ 6 files changed, 44 insertions(+) diff --git a/bsps/powerpc/gen83xx/dev/mpc83xx_i2cdrv.c b/bsps/powerpc/gen83xx/dev/mpc83xx_i2cdrv.c index 44d238e859..aae259c975 100644 --- a/bsps/powerpc/gen83xx/dev/mpc83xx_i2cdrv.c +++ b/bsps/powerpc/gen83xx/dev/mpc83xx_i2cdrv.c @@ -248,6 +248,7 @@ static void mpc83xx_i2c_irq_on_off | | \*=========================================================================*/ { + (void) irq_conn_data; } @@ -270,6 +271,8 @@ static int mpc83xx_i2c_irq_isOn | TRUE, if enabled | \*=========================================================================*/ { + (void) irq_conn_data; + return (TRUE); } diff --git a/bsps/powerpc/gen83xx/dev/mpc83xx_spidrv.c b/bsps/powerpc/gen83xx/dev/mpc83xx_spidrv.c index f96be67644..440e94d669 100644 --- a/bsps/powerpc/gen83xx/dev/mpc83xx_spidrv.c +++ b/bsps/powerpc/gen83xx/dev/mpc83xx_spidrv.c @@ -270,6 +270,7 @@ static void mpc83xx_spi_irq_on_off | | \*=========================================================================*/ { + (void) irq_conn_data; } @@ -292,6 +293,8 @@ static int mpc83xx_spi_irq_isOn | TRUE, if enabled | \*=========================================================================*/ { + (void) irq_conn_data; + return (TRUE); } diff --git a/bsps/powerpc/gen83xx/include/tm27.h b/bsps/powerpc/gen83xx/include/tm27.h index 95cca5fea2..f56338231d 100644 --- a/bsps/powerpc/gen83xx/include/tm27.h +++ b/bsps/powerpc/gen83xx/include/tm27.h @@ -46,6 +46,9 @@ static rtems_interrupt_handler tm27_interrupt_handler; static int tm27_exception_handler( BSP_Exception_frame *frame, unsigned number) { + (void) frame; + (void) number; + (*tm27_interrupt_handler)( NULL); return 0; diff --git a/bsps/powerpc/gen83xx/irq/irq.c b/bsps/powerpc/gen83xx/irq/irq.c index f82d9a352e..74c74778ee 100644 --- a/bsps/powerpc/gen83xx/irq/irq.c +++ b/bsps/powerpc/gen83xx/irq/irq.c @@ -395,6 +395,9 @@ rtems_status_code bsp_interrupt_get_attributes( rtems_interrupt_attributes *attributes ) { + (void) vector; + (void) attributes; + return RTEMS_SUCCESSFUL; } @@ -403,6 +406,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; @@ -411,12 +416,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; } @@ -426,6 +435,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; @@ -481,6 +492,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; } @@ -490,6 +504,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; @@ -651,6 +668,8 @@ static int mpc83xx_exception_handler( unsigned exception_number ) { + (void) frame; + return BSP_irq_handle_at_ipic( exception_number); } diff --git a/bsps/powerpc/gen83xx/spi/spi_init.c b/bsps/powerpc/gen83xx/spi/spi_init.c index 0455c225b0..9243e77347 100644 --- a/bsps/powerpc/gen83xx/spi/spi_init.c +++ b/bsps/powerpc/gen83xx/spi/spi_init.c @@ -76,6 +76,8 @@ static rtems_status_code bsp_spi_sel_addr { #if defined( MPC83XX_BOARD_MPC8313ERDB) + (void) bh; + (void) rw; /* Check address */ if (addr > 0) { @@ -86,6 +88,8 @@ static rtems_status_code bsp_spi_sel_addr mpc83xx.gpio [0].gpdat &= ~0x20000000; #elif defined( MPC83XX_BOARD_MPC8349EAMDS) + (void) bh; + (void) rw; /* * check device address for valid range @@ -101,6 +105,8 @@ static rtems_status_code bsp_spi_sel_addr mpc83xx.gpio[0].gpdat &= ~(1 << (31- 0)); #elif defined( MPC83XX_BOARD_HSC_CM01) + (void) bh; + (void) rw; /* * check device address for valid range @@ -125,6 +131,10 @@ static rtems_status_code bsp_spi_sel_addr */ mpc83xx.gpio[0].gpdat |= (1 << (31- 27)); +#else + (void) bh; + (void) addr; + (void) rw; #endif return RTEMS_SUCCESSFUL; @@ -148,6 +158,7 @@ static rtems_status_code bsp_spi_send_start_dummy | o = ok or error code | \*=========================================================================*/ { + (void) bh; #if defined( MPC83XX_BOARD_MPC8313ERDB) @@ -193,6 +204,8 @@ static rtems_status_code bsp_spi_send_stop | o = ok or error code | \*=========================================================================*/ { + (void) bh; + #if defined(DEBUG) printk("bsp_spi_send_stop called... "); #endif diff --git a/bsps/powerpc/gen83xx/start/bspstart.c b/bsps/powerpc/gen83xx/start/bspstart.c index 4b61d58be9..e5f22d51e1 100644 --- a/bsps/powerpc/gen83xx/start/bspstart.c +++ b/bsps/powerpc/gen83xx/start/bspstart.c @@ -59,6 +59,9 @@ uint32_t bsp_clicks_per_usec; /* Default decrementer exception handler */ static int mpc83xx_decrementer_exception_handler( BSP_Exception_frame *frame, unsigned number) { + (void) frame; + (void) number; + ppc_set_decrementer_register(UINT32_MAX); return 0;