diff options
-rw-r--r-- | nuttx/Documentation/NuttxPortingGuide.html | 6 | ||||
-rw-r--r-- | nuttx/arch/arm/include/str71x/irq.h | 2 | ||||
-rw-r--r-- | nuttx/arch/arm/src/str71x/str71x_irq.c | 4 | ||||
-rw-r--r-- | nuttx/arch/arm/src/str71x/str71x_timerisr.c | 2 | ||||
-rw-r--r-- | nuttx/arch/sh/src/common/up_doirq.c | 2 | ||||
-rw-r--r-- | nuttx/arch/sh/src/common/up_internal.h | 6 | ||||
-rw-r--r-- | nuttx/arch/sh/src/sh1/sh1_head.S | 2 | ||||
-rw-r--r-- | nuttx/arch/sh/src/sh1/sh1_irq.c | 44 | ||||
-rw-r--r-- | nuttx/arch/sh/src/sh1/sh1_serial.c | 22 | ||||
-rw-r--r-- | nuttx/arch/sh/src/sh1/sh1_timerisr.c | 2 | ||||
-rw-r--r-- | nuttx/configs/README.txt | 6 | ||||
-rw-r--r-- | nuttx/configs/olimex-strp711/ostest/defconfig | 8 | ||||
-rw-r--r-- | nuttx/configs/us7032evb1/ostest/defconfig | 8 | ||||
-rw-r--r-- | nuttx/include/nuttx/arch.h | 20 |
14 files changed, 68 insertions, 66 deletions
diff --git a/nuttx/Documentation/NuttxPortingGuide.html b/nuttx/Documentation/NuttxPortingGuide.html index 4aee251ae..a471ac72a 100644 --- a/nuttx/Documentation/NuttxPortingGuide.html +++ b/nuttx/Documentation/NuttxPortingGuide.html @@ -1327,6 +1327,12 @@ The system can be re-made subsequently by just typing <code>make</code>. For use in C code</li> <li><code>CONFIG_ENDIAN_BIG</code>: Define if big endian (default is little endian).</li> + <li><code>CONFIG_ARCH_NOINTC</code>: + Define if the architecture does not support an interrupt controller + or otherwise cannot support APIs like up_enable_irq() and up_disable_irq().</li> + <li><code>CONFIG_ARCH_IRQPRIO</code>: + Define if the architecture suports prioritizaton of interrupts and the + up_prioritize_irq() API.</li> </ul> <p> diff --git a/nuttx/arch/arm/include/str71x/irq.h b/nuttx/arch/arm/include/str71x/irq.h index d68b3779f..ecb1dcc95 100644 --- a/nuttx/arch/arm/include/str71x/irq.h +++ b/nuttx/arch/arm/include/str71x/irq.h @@ -110,8 +110,6 @@ extern "C" { * Public Functions ************************************************************************************/ -EXTERN int up_irqpriority(int irq, ubyte priority); /* Set interrupt priority (0-15) */ - #undef EXTERN #ifdef __cplusplus } diff --git a/nuttx/arch/arm/src/str71x/str71x_irq.c b/nuttx/arch/arm/src/str71x/str71x_irq.c index 2093ecc7f..1c243f460 100644 --- a/nuttx/arch/arm/src/str71x/str71x_irq.c +++ b/nuttx/arch/arm/src/str71x/str71x_irq.c @@ -169,14 +169,14 @@ void up_maskack_irq(int irq) } /**************************************************************************** - * Name: up_irqpriority + * Name: up_prioritize_irq * * Description: * set interrupt priority * ****************************************************************************/ -int up_irqpriority(int irq, ubyte priority) +int up_prioritize_irq(int irq, int priority) { uint32 addr; uint32 reg32; diff --git a/nuttx/arch/arm/src/str71x/str71x_timerisr.c b/nuttx/arch/arm/src/str71x/str71x_timerisr.c index 571d67de2..f6b709e60 100644 --- a/nuttx/arch/arm/src/str71x/str71x_timerisr.c +++ b/nuttx/arch/arm/src/str71x/str71x_timerisr.c @@ -184,7 +184,7 @@ void up_timerinit(void) /* Set the IRQ interrupt priority */ - up_irqpriority(STR71X_IRQ_SYSTIMER, 1); + up_prioritize_irq(STR71X_IRQ_SYSTIMER, 1); /* Attach the timer interrupt vector */ diff --git a/nuttx/arch/sh/src/common/up_doirq.c b/nuttx/arch/sh/src/common/up_doirq.c index 6b0d2c0fc..01aa21500 100644 --- a/nuttx/arch/sh/src/common/up_doirq.c +++ b/nuttx/arch/sh/src/common/up_doirq.c @@ -81,7 +81,7 @@ void up_doirq(int irq, uint32* regs) current_regs = regs; - /* Mask and acknowledge the interrupt */ + /* Mask and acknowledge the interrupt (if supported by the chip) */ up_maskack_irq(irq); diff --git a/nuttx/arch/sh/src/common/up_internal.h b/nuttx/arch/sh/src/common/up_internal.h index 31f016074..8cb3ee2dd 100644 --- a/nuttx/arch/sh/src/common/up_internal.h +++ b/nuttx/arch/sh/src/common/up_internal.h @@ -171,9 +171,13 @@ extern void up_wdtinit(void); extern void up_timerinit(void); -/* Defined in up_irq.c */ +/* Defined in chip-specific logic if CONFIG_ARCH_NOINTC is not set */ +#ifndef CONFIG_ARCH_NOINTC extern void up_maskack_irq(int irq); +#else +# define up_maskack_irq(irq) +#endif /* Defined in board/up_leds.c */ diff --git a/nuttx/arch/sh/src/sh1/sh1_head.S b/nuttx/arch/sh/src/sh1/sh1_head.S index 6ff129493..82e8c7175 100644 --- a/nuttx/arch/sh/src/sh1/sh1_head.S +++ b/nuttx/arch/sh/src/sh1/sh1_head.S @@ -232,7 +232,7 @@ __start0: add #4, r0 /* R0: Address of next byte to clear in BSS */ cmp/ge r0, r1 /* End of BSS? */ bt 3b /* Loop until the end of BSS */ - nop /* Delay slot + nop /* Delay slot */ /* Configure the uart so that we can get debug output as soon * as possible. diff --git a/nuttx/arch/sh/src/sh1/sh1_irq.c b/nuttx/arch/sh/src/sh1/sh1_irq.c index f56f642d5..71a43f5f2 100644 --- a/nuttx/arch/sh/src/sh1/sh1_irq.c +++ b/nuttx/arch/sh/src/sh1/sh1_irq.c @@ -88,54 +88,14 @@ void up_irqinitialize(void) } /**************************************************************************** - * Name: up_disable_irq - * - * Description: - * Disable the IRQ specified by 'irq' - * - ****************************************************************************/ - -void up_disable_irq(int irq) -{ -#warning "To be provided" -} - -/**************************************************************************** - * Name: up_enable_irq - * - * Description: - * Enable the IRQ specified by 'irq' - * - ****************************************************************************/ - -void up_enable_irq(int irq) -{ -#warning "To be provided" -} - -/**************************************************************************** - * Name: up_maskack_irq - * - * Description: - * Mask the IRQ and acknowledge it - * - ****************************************************************************/ - -void up_maskack_irq(int irq) -{ -#warning "To be provided" -} - -/**************************************************************************** - * Name: up_irqpriority + * Name: up_prioritize_irq * * Description: * set interrupt priority * ****************************************************************************/ -#warning "Should this be supported?" -void up_irqpriority(int irq, ubyte priority) +void up_prioritize_irq(int irq, int priority) { #warning "To be provided" } diff --git a/nuttx/arch/sh/src/sh1/sh1_serial.c b/nuttx/arch/sh/src/sh1/sh1_serial.c index cfdfbe349..7578132b2 100644 --- a/nuttx/arch/sh/src/sh1/sh1_serial.c +++ b/nuttx/arch/sh/src/sh1/sh1_serial.c @@ -425,24 +425,16 @@ static int up_attach(struct uart_dev_s *dev) /* Attach the RDR full IRQ */ - ret = irq_attach(priv->irq + , up_interrupt); + ret = irq_attach(priv->irq + SH1_RXI_IRQ_OFFSET, up_interrupt); if (ret == OK) { - /* Enable the interrupt - */ + /* Attach the TDR empty IRQ */ - up_enable_irq(priv->irq); - } - - /* Enable the RDR full and TDR empty interrupts at the interupt controller - * (RX and TX interrupts are still disabled in the SCI) - */ - - if (ret == OK) - { - - up_enable_irq(priv->irq); - up_enable_irq(priv->irq); + ret = irq_attach(priv->irq + SH1_TXI_IRQ_OFFSET, up_interrupt); + if (ret < 0) + { + (void)irq_detach(priv->irq + SH1_RXI_IRQ_OFFSET); + } } return ret; diff --git a/nuttx/arch/sh/src/sh1/sh1_timerisr.c b/nuttx/arch/sh/src/sh1/sh1_timerisr.c index d3bf495a1..874e25647 100644 --- a/nuttx/arch/sh/src/sh1/sh1_timerisr.c +++ b/nuttx/arch/sh/src/sh1/sh1_timerisr.c @@ -104,7 +104,7 @@ void up_timerinit(void) /* Set the IRQ interrupt priority */ - up_irqpriority(STR71X_IRQ_SYSTIMER, 1); + up_prioritize_irq(STR71X_IRQ_SYSTIMER, 1); /* Attach the timer interrupt vector */ diff --git a/nuttx/configs/README.txt b/nuttx/configs/README.txt index bedbc6ae7..123a90171 100644 --- a/nuttx/configs/README.txt +++ b/nuttx/configs/README.txt @@ -126,6 +126,12 @@ defconfig -- This is a configuration file similar to the Linux CONFIG_ARCH_BOARD_name - For use in C code CONFIG_ENDIAN_BIG - define if big endian (default is little endian) + CONFIG_ARCH_NOINTC - define if the architecture does not + support an interrupt controller or otherwise cannot support + APIs like up_enable_irq() and up_disable_irq(). + CONFIG_ARCH_IRQPRIO + Define if the architecture suports prioritizaton of interrupts + and the up_prioritize_irq() API. Some architectures require a description of the RAM configuration: diff --git a/nuttx/configs/olimex-strp711/ostest/defconfig b/nuttx/configs/olimex-strp711/ostest/defconfig index 1ff1ad1c0..0d629c2e1 100644 --- a/nuttx/configs/olimex-strp711/ostest/defconfig +++ b/nuttx/configs/olimex-strp711/ostest/defconfig @@ -46,6 +46,12 @@ # the board that supports the particular chip or SoC. # CONFIG_ENDIAN_BIG - define if big endian (default is little endian) # CONFIG_ARCH_BOARD_name - for use in C code +# CONFIG_ARCH_NOINTC - define if the architecture does not +# support an interrupt controller or otherwise cannot support +# APIs like up_enable_irq() and up_disable_irq(). +# CONFIG_ARCH_IRQPRIO +# Define if the architecture suports prioritizaton of interrupts +# and the up_prioritize_irq() API. # CONFIG_BOARD_LOOPSPERMSEC - for delay loops # CONFIG_ARCH_LEDS - Use LEDs to show state. Unique to Olimex STR-P711 # CONFIG_ARCH_BUTTONS - Support reading buttons. Unique to Olimex STR-P711 @@ -62,6 +68,8 @@ CONFIG_ARCH_CHIP=str71x CONFIG_ARCH_STR71X=y CONFIG_ARCH_BOARD=olimex-strp711 CONFIG_ARCH_BOARD_OLIMEX_STRP711=y +CONFIG_ARCH_NOINTC=n +CONFIG_ARCH_IRQPRIO=y CONFIG_BOARD_LOOPSPERMSEC=3270 CONFIG_ARCH_LEDS=y CONFIG_ARCH_BUTTONS=y diff --git a/nuttx/configs/us7032evb1/ostest/defconfig b/nuttx/configs/us7032evb1/ostest/defconfig index 6510a5029..fa2ac1c48 100644 --- a/nuttx/configs/us7032evb1/ostest/defconfig +++ b/nuttx/configs/us7032evb1/ostest/defconfig @@ -46,6 +46,12 @@ # the board that supports the particular chip or SoC. # CONFIG_ENDIAN_BIG - define if big endian (default is little endian) # CONFIG_ARCH_BOARD_name - for use in C code +# CONFIG_ARCH_NOINTC - define if the architecture does not +# support an interrupt controller or otherwise cannot support +# APIs like up_enable_irq() and up_disable_irq(). +# CONFIG_ARCH_IRQPRIO +# Define if the architecture suports prioritizaton of interrupts +# and the up_prioritize_irq() API. # CONFIG_ENDIAN_BIG - Define for big-endian operation # CONFIG_BOARD_LOOPSPERMSEC - for delay loops # CONFIG_ARCH_LEDS - Use LEDs to show state. Unique to Olimex STR-P711 @@ -65,6 +71,8 @@ CONFIG_ARCH_SH7032=y CONFIG_ARCH_BOARD=us7032evb1 CONFIG_ARCH_BOARD_US7032EVB1=y CONFIG_ENDIAN_BIG=y +CONFIG_ARCH_NOINTC=y +CONFIG_ARCH_IRQPRIO=y CONFIG_BOARD_LOOPSPERMSEC=3270 CONFIG_ARCH_LEDS=y CONFIG_ARCH_BUTTONS=y diff --git a/nuttx/include/nuttx/arch.h b/nuttx/include/nuttx/arch.h index 40667b077..e2e8958ea 100644 --- a/nuttx/include/nuttx/arch.h +++ b/nuttx/include/nuttx/arch.h @@ -394,7 +394,9 @@ EXTERN boolean up_interrupt_context(void); * ****************************************************************************/ +#ifndef CONFIG_ARCH_NOINTC EXTERN void up_enable_irq(int irq); +#endif /**************************************************************************** * Name: up_disable_irq @@ -410,7 +412,25 @@ EXTERN void up_enable_irq(int irq); * ****************************************************************************/ +#ifndef CONFIG_ARCH_NOINTC EXTERN void up_disable_irq(int irq); +#endif + + +/**************************************************************************** + * Name: up_prioritize_irq + * + * Description: + * Set the priority of an IRQ. + * + * Since this API is not supported on all architectures, it should be + * avoided in common implementations where possible. + * + ****************************************************************************/ + +#ifdef CONFIG_ARCH_IRQPRIO +EXTERN int up_prioritize_irq(int irq, int priority); +#endif /**************************************************************************** * These are standard interfaces that are exported by the OS |