From 0aa3e14695407f3abc443f12e56a2bc04d191853 Mon Sep 17 00:00:00 2001 From: Joel Sherrill Date: Fri, 10 Oct 2025 14:08:25 -0500 Subject: [PATCH] bsps/powerpc/mpc55xxevb: Address unused parameter warnings Add "(void) param;" annotation to address unused parameter warnings. Found with GCC's warning -Wunused-parameter. --- bsps/powerpc/mpc55xxevb/clock/clock-config.c | 12 ++++++++++++ bsps/powerpc/mpc55xxevb/console/console-esci.c | 5 +++++ bsps/powerpc/mpc55xxevb/console/console-generic.c | 14 ++++++++++++++ bsps/powerpc/mpc55xxevb/console/console-linflex.c | 5 +++++ bsps/powerpc/mpc55xxevb/dev/dspi.c | 4 ++++ bsps/powerpc/mpc55xxevb/start/edma.c | 4 ++++ bsps/powerpc/mpc55xxevb/start/idle-thread.c | 2 ++ bsps/powerpc/mpc55xxevb/start/irq.c | 12 ++++++++++++ 8 files changed, 58 insertions(+) diff --git a/bsps/powerpc/mpc55xxevb/clock/clock-config.c b/bsps/powerpc/mpc55xxevb/clock/clock-config.c index d48367ae1e..e93c394faf 100644 --- a/bsps/powerpc/mpc55xxevb/clock/clock-config.c +++ b/bsps/powerpc/mpc55xxevb/clock/clock-config.c @@ -49,11 +49,15 @@ static rtems_timecounter_simple mpc55xx_tc; static uint32_t mpc55xx_tc_get(rtems_timecounter_simple *tc) { + (void) tc; + return EMIOS.CH [MPC55XX_CLOCK_EMIOS_CHANNEL].CCNTR.R; } static bool mpc55xx_tc_is_pending(rtems_timecounter_simple *tc) { + (void) tc; + return EMIOS.CH [MPC55XX_CLOCK_EMIOS_CHANNEL].CSR.B.FLAG != 0; } @@ -68,6 +72,8 @@ static uint32_t mpc55xx_tc_get_timecount(struct timecounter *tc) static void mpc55xx_tc_at_tick(rtems_timecounter_simple *tc) { + (void) tc; + union EMIOS_CSR_tag csr = MPC55XX_ZERO_FLAGS; csr.B.FLAG = 1; EMIOS.CH [MPC55XX_CLOCK_EMIOS_CHANNEL].CSR.R = csr.R; @@ -162,11 +168,15 @@ static void mpc55xx_clock_initialize(void) static uint32_t mpc55xx_tc_get(rtems_timecounter_simple *tc) { + (void) tc; + return PIT_RTI.CHANNEL [MPC55XX_CLOCK_PIT_CHANNEL].CVAL.R; } static bool mpc55xx_tc_is_pending(rtems_timecounter_simple *tc) { + (void) tc; + return PIT_RTI.CHANNEL [MPC55XX_CLOCK_PIT_CHANNEL].TFLG.B.TIF != 0; } @@ -181,6 +191,8 @@ static uint32_t mpc55xx_tc_get_timecount(struct timecounter *tc) static void mpc55xx_tc_at_tick(rtems_timecounter_simple *tc) { + (void) tc; + volatile PIT_RTI_CHANNEL_tag *channel = &PIT_RTI.CHANNEL [MPC55XX_CLOCK_PIT_CHANNEL]; PIT_RTI_TFLG_32B_tag tflg = { .B = { .TIF = 1 } }; diff --git a/bsps/powerpc/mpc55xxevb/console/console-esci.c b/bsps/powerpc/mpc55xxevb/console/console-esci.c index 52cde2e46d..7fac495bb0 100644 --- a/bsps/powerpc/mpc55xxevb/console/console-esci.c +++ b/bsps/powerpc/mpc55xxevb/console/console-esci.c @@ -273,6 +273,8 @@ static int mpc55xx_esci_set_attributes(int minor, const struct termios *t) static int mpc55xx_esci_first_open(int major, int minor, void *arg) { + (void) major; + rtems_status_code sc = RTEMS_SUCCESSFUL; int rv = 0; mpc55xx_esci_context *self = console_generic_get_context(minor); @@ -310,6 +312,9 @@ static int mpc55xx_esci_first_open(int major, int minor, void *arg) static int mpc55xx_esci_last_close(int major, int minor, void* arg) { + (void) major; + (void) arg; + mpc55xx_esci_context *self = console_generic_get_context(minor); mpc55xx_esci_interrupts_disable(self); diff --git a/bsps/powerpc/mpc55xxevb/console/console-generic.c b/bsps/powerpc/mpc55xxevb/console/console-generic.c index 7b8d492530..5a1f45970b 100644 --- a/bsps/powerpc/mpc55xxevb/console/console-generic.c +++ b/bsps/powerpc/mpc55xxevb/console/console-generic.c @@ -85,6 +85,8 @@ rtems_device_driver console_initialize( void *arg ) { + (void) arg; + rtems_status_code sc = RTEMS_SUCCESSFUL; const console_generic_info *info_table = console_generic_info_table; rtems_device_minor_number count = console_generic_info_count; @@ -146,6 +148,9 @@ rtems_device_driver console_close( void *arg ) { + (void) major; + (void) minor; + return rtems_termios_close(arg); } @@ -155,6 +160,9 @@ rtems_device_driver console_read( void *arg ) { + (void) major; + (void) minor; + return rtems_termios_read(arg); } @@ -164,6 +172,9 @@ rtems_device_driver console_write( void *arg ) { + (void) major; + (void) minor; + return rtems_termios_write(arg); } @@ -173,6 +184,9 @@ rtems_device_driver console_control( void *arg ) { + (void) major; + (void) minor; + return rtems_termios_ioctl(arg); } diff --git a/bsps/powerpc/mpc55xxevb/console/console-linflex.c b/bsps/powerpc/mpc55xxevb/console/console-linflex.c index 7d4f04fe68..c50ff76364 100644 --- a/bsps/powerpc/mpc55xxevb/console/console-linflex.c +++ b/bsps/powerpc/mpc55xxevb/console/console-linflex.c @@ -255,6 +255,8 @@ static int mpc55xx_linflex_set_attributes(int minor, const struct termios *t) static int mpc55xx_linflex_first_open(int major, int minor, void *arg) { + (void) major; + rtems_status_code sc = RTEMS_SUCCESSFUL; int rv = 0; mpc55xx_linflex_context *self = console_generic_get_context(minor); @@ -326,6 +328,9 @@ static int mpc55xx_linflex_first_open(int major, int minor, void *arg) static int mpc55xx_linflex_last_close(int major, int minor, void* arg) { + (void) major; + (void) arg; + rtems_status_code sc = RTEMS_SUCCESSFUL; mpc55xx_linflex_context *self = console_generic_get_context(minor); volatile LINFLEX_tag *regs = self->regs; diff --git a/bsps/powerpc/mpc55xxevb/dev/dspi.c b/bsps/powerpc/mpc55xxevb/dev/dspi.c index 2b168ba9af..83cd19615b 100644 --- a/bsps/powerpc/mpc55xxevb/dev/dspi.c +++ b/bsps/powerpc/mpc55xxevb/dev/dspi.c @@ -293,11 +293,15 @@ static rtems_status_code mpc55xx_dspi_send_start( rtems_libi2c_bus_t *bus) static rtems_status_code mpc55xx_dspi_send_stop( rtems_libi2c_bus_t *bus) { + (void) bus; + return RTEMS_SUCCESSFUL; } static rtems_status_code mpc55xx_dspi_send_addr( rtems_libi2c_bus_t *bus, uint32_t addr, int rw) { + (void) rw; + mpc55xx_dspi_bus_entry *e = (mpc55xx_dspi_bus_entry *) bus; union DSPI_SR_tag sr = MPC55XX_ZERO_FLAGS; diff --git a/bsps/powerpc/mpc55xxevb/start/edma.c b/bsps/powerpc/mpc55xxevb/start/edma.c index 6f86400c11..8b1aff267a 100644 --- a/bsps/powerpc/mpc55xxevb/start/edma.c +++ b/bsps/powerpc/mpc55xxevb/start/edma.c @@ -73,6 +73,8 @@ static unsigned edma_channel_index_of_tcd(volatile struct tcd_t *edma_tcd) static volatile struct EDMA_tag *edma_get_regs_by_module(unsigned module) { #if EDMA_MODULE_COUNT == 1 + (void) module; + return &EDMA; #elif EDMA_MODULE_COUNT == 2 return module == 0 ? &EDMA_A : &EDMA_B; @@ -114,6 +116,8 @@ static void edma_interrupt_handler(void *arg) static void edma_interrupt_error_handler(void *arg) { + (void) arg; + rtems_chain_control *chain = &edma_channel_chain; rtems_chain_node *node = rtems_chain_first(chain); uint32_t error_channels [] = { diff --git a/bsps/powerpc/mpc55xxevb/start/idle-thread.c b/bsps/powerpc/mpc55xxevb/start/idle-thread.c index b73759e25a..3df7e17997 100644 --- a/bsps/powerpc/mpc55xxevb/start/idle-thread.c +++ b/bsps/powerpc/mpc55xxevb/start/idle-thread.c @@ -39,6 +39,8 @@ void *bsp_idle_thread(uintptr_t arg) { + (void) arg; + while (true) { mpc55xx_wait_for_interrupt(); } diff --git a/bsps/powerpc/mpc55xxevb/start/irq.c b/bsps/powerpc/mpc55xxevb/start/irq.c index 8569505f18..c6b11561a0 100644 --- a/bsps/powerpc/mpc55xxevb/start/irq.c +++ b/bsps/powerpc/mpc55xxevb/start/irq.c @@ -123,6 +123,8 @@ rtems_status_code mpc55xx_interrupt_handler_install( void bsp_interrupt_dispatch(uintptr_t exception_number) { + (void) exception_number; + /* Acknowledge interrupt request */ rtems_vector_number vector = INTC.IACKR.B.INTVEC; @@ -163,6 +165,8 @@ rtems_status_code bsp_interrupt_get_attributes( rtems_interrupt_attributes *attributes ) { + (void) vector; + attributes->maximum_priority = MPC55XX_INTC_DISABLED_PRIORITY; attributes->can_get_priority = true; attributes->can_set_priority = true; @@ -174,6 +178,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; @@ -182,12 +188,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; } @@ -197,6 +207,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;