diff options
author | Sebastian Huber <sebastian.huber@embedded-brains.de> | 2021-07-27 09:58:43 +0200 |
---|---|---|
committer | Sebastian Huber <sebastian.huber@embedded-brains.de> | 2021-07-27 10:03:19 +0200 |
commit | c7b4eca7fa3e311c67d5dd1e62e301b01ed45271 (patch) | |
tree | 8580e41fd1bac88af64654bde7d4bf0b340ce173 /bsps/powerpc/mpc8260ads/irq/irq.c | |
parent | validation: Test rtems_interrupt_handler_iterate() (diff) | |
download | rtems-c7b4eca7fa3e311c67d5dd1e62e301b01ed45271.tar.bz2 |
bsps/irq: bsp_interrupt_facility_initialize()
Do not return a status code in bsp_interrupt_facility_initialize() since this
leads to unreachable code in bsp_interrupt_initialize(). Use RTEMS_DEBUG
assertions in bsp_interrupt_facility_initialize() if necessary.
Diffstat (limited to 'bsps/powerpc/mpc8260ads/irq/irq.c')
-rw-r--r-- | bsps/powerpc/mpc8260ads/irq/irq.c | 16 |
1 files changed, 7 insertions, 9 deletions
diff --git a/bsps/powerpc/mpc8260ads/irq/irq.c b/bsps/powerpc/mpc8260ads/irq/irq.c index 9b73979efa..70cb5acf9b 100644 --- a/bsps/powerpc/mpc8260ads/irq/irq.c +++ b/bsps/powerpc/mpc8260ads/irq/irq.c @@ -398,21 +398,19 @@ rtems_status_code bsp_interrupt_vector_disable( rtems_vector_number irqnum) return RTEMS_SUCCESSFUL; } -rtems_status_code bsp_interrupt_facility_initialize() +void bsp_interrupt_facility_initialize() { + rtems_status_code sc; + /* Install exception handler */ - if (ppc_exc_set_handler( ASM_EXT_VECTOR, C_dispatch_irq_handler)) { - return RTEMS_IO_ERROR; - } - if (ppc_exc_set_handler( ASM_DEC_VECTOR, C_dispatch_irq_handler)) { - return RTEMS_IO_ERROR; - } + sc = ppc_exc_set_handler( ASM_EXT_VECTOR, C_dispatch_irq_handler); + _Assert_Unused_variable_equals( sc, RTEMS_SUCCESSFUL); + sc = ppc_exc_set_handler( ASM_DEC_VECTOR, C_dispatch_irq_handler); + _Assert_Unused_variable_equals( sc, RTEMS_SUCCESSFUL); /* Fill in priority masks */ compute_SIU_IvectMask_from_prio(); /* Initialize the interrupt controller */ BSP_CPM_irq_init(); - - return RTEMS_SUCCESSFUL; } |