diff options
| author | Sebastian Huber <sebastian.huber@embedded-brains.de> | 2017-06-19 14:09:28 +0200 |
|---|---|---|
| committer | Sebastian Huber <sebastian.huber@embedded-brains.de> | 2017-06-20 07:42:25 +0200 |
| commit | c6810c824c9445fdc44ae5c0092f9ae5355a1420 (patch) | |
| tree | 72441065d38fefdef1008118d9bf51bebf71ad05 /c/src/lib/libbsp/powerpc/tqm8xx | |
| parent | bsps/powerpc: Fix warning (diff) | |
| download | rtems-c6810c824c9445fdc44ae5c0092f9ae5355a1420.tar.bz2 | |
bsps: Improve interrupt vector enable/disable API
Change bsp_interrupt_vector_enable() and bsp_interrupt_vector_disable()
to not return a status code. Add bsp_interrupt_assert() and use it to
validate the vector number in the vector enable/disable implementations.
Diffstat (limited to 'c/src/lib/libbsp/powerpc/tqm8xx')
| -rw-r--r-- | c/src/lib/libbsp/powerpc/tqm8xx/irq/irq.c | 32 |
1 files changed, 14 insertions, 18 deletions
diff --git a/c/src/lib/libbsp/powerpc/tqm8xx/irq/irq.c b/c/src/lib/libbsp/powerpc/tqm8xx/irq/irq.c index 8066a7f43b..2a94c27d1b 100644 --- a/c/src/lib/libbsp/powerpc/tqm8xx/irq/irq.c +++ b/c/src/lib/libbsp/powerpc/tqm8xx/irq/irq.c @@ -63,30 +63,26 @@ static rtems_status_code bsp_irq_enable_at_CPM(rtems_vector_number irqnum) return RTEMS_SUCCESSFUL; } -rtems_status_code bsp_interrupt_vector_enable( rtems_vector_number irqnum) +void bsp_interrupt_vector_enable(rtems_vector_number vector) { - if (BSP_IS_CPM_IRQ(irqnum)) { - bsp_irq_enable_at_CPM(irqnum); - return RTEMS_SUCCESSFUL; - } - else if (BSP_IS_SIU_IRQ(irqnum)) { - bsp_irq_enable_at_SIU(irqnum); - return RTEMS_SUCCESSFUL; + bsp_interrupt_assert(bsp_interrupt_is_valid_vector(vector)); + + if (BSP_IS_CPM_IRQ(vector)) { + bsp_irq_enable_at_CPM(vector); + } else if (BSP_IS_SIU_IRQ(vector)) { + bsp_irq_enable_at_SIU(vector); } - return RTEMS_INVALID_ID; } -rtems_status_code bsp_interrupt_vector_disable( rtems_vector_number irqnum) +void bsp_interrupt_vector_disable(rtems_vector_number vector) { - if (BSP_IS_CPM_IRQ(irqnum)) { - bsp_irq_disable_at_CPM(irqnum); - return RTEMS_SUCCESSFUL; - } - else if (BSP_IS_SIU_IRQ(irqnum)) { - bsp_irq_disable_at_SIU(irqnum); - return RTEMS_SUCCESSFUL; + bsp_interrupt_assert(bsp_interrupt_is_valid_vector(vector)); + + if (BSP_IS_CPM_IRQ(vector)) { + bsp_irq_disable_at_CPM(vector); + } else if (BSP_IS_SIU_IRQ(vector)) { + bsp_irq_disable_at_SIU(vector); } - return RTEMS_INVALID_ID; } /* |