How to enable multiple peripheral interrupts ?

[stdefeber]

Hello,

I have multiple interrupt sources, being uart, wdt and a timer.

The timer interrupts works. However when adding the UART0 RX interrupt, it is not triggering an ISR.

In my uart_init and timer_int function I register the ISR as follows:

neorv32_rte_handler_install(UART0_RX_RTE_ID, uart_rx_isr)

and

neorv32_rte_handler_install(GPTMR_RTE_ID, timer_isr)

In my init_core function:

bool init_core(void)
{
  bool init_ok = false;


  // setup NEORV32 runtime environment (for trap handling)
  neorv32_rte_setup();

  // enable interrupt
  neorv32_cpu_csr_set(CSR_MIE, 1 << GPTMR_FIRQ_ENABLE | 1 << UART0_RX_FIRQ_ENABLE);   // enable GPTMR FIRQ and UART0_RX_FIRQ_ENABLE channel
                                                          //
  neorv32_cpu_csr_set(CSR_MSTATUS, 1 << CSR_MSTATUS_MIE); // enable machine-mode interrupts

Is this the correct way ?

best regards

Simon

[stnolting]

Hey there!

I have multiple interrupt sources, being uart, wdt and a timer.

Watchdog interrupt?? Which version of the core are you using?

In my uart_init and timer_int function I register the ISR as follows:

Basically, this is correct. You can use the "raw" RTE IDs:

neorv32/sw/lib/include/neorv32_rte.h

Lines 21 to 56 in b3418fd

	/**********************************************************************//**
	* NEORV32 runtime environment trap IDs.
	**************************************************************************/
	/**@{*/
	/**< Trap ID enumeration */
	enum NEORV32_RTE_TRAP_enum {
	RTE_TRAP_I_ACCESS = 0, /**< Instruction access fault */
	RTE_TRAP_I_ILLEGAL = 1, /**< Illegal instruction */
	RTE_TRAP_I_MISALIGNED = 2, /**< Instruction address misaligned */
	RTE_TRAP_BREAKPOINT = 3, /**< Breakpoint (EBREAK instruction) */
	RTE_TRAP_L_MISALIGNED = 4, /**< Load address misaligned */
	RTE_TRAP_L_ACCESS = 5, /**< Load access fault */
	RTE_TRAP_S_MISALIGNED = 6, /**< Store address misaligned */
	RTE_TRAP_S_ACCESS = 7, /**< Store access fault */
	RTE_TRAP_UENV_CALL = 8, /**< Environment call from user mode (ECALL instruction) */
	RTE_TRAP_MENV_CALL = 9, /**< Environment call from machine mode (ECALL instruction) */
	RTE_TRAP_MSI = 10, /**< Machine software interrupt */
	RTE_TRAP_MTI = 11, /**< Machine timer interrupt */
	RTE_TRAP_MEI = 12, /**< Machine external interrupt */
	RTE_TRAP_FIRQ_0 = 13, /**< Fast interrupt channel 0 */
	RTE_TRAP_FIRQ_1 = 14, /**< Fast interrupt channel 1 */
	RTE_TRAP_FIRQ_2 = 15, /**< Fast interrupt channel 2 */
	RTE_TRAP_FIRQ_3 = 16, /**< Fast interrupt channel 3 */
	RTE_TRAP_FIRQ_4 = 17, /**< Fast interrupt channel 4 */
	RTE_TRAP_FIRQ_5 = 18, /**< Fast interrupt channel 5 */
	RTE_TRAP_FIRQ_6 = 19, /**< Fast interrupt channel 6 */
	RTE_TRAP_FIRQ_7 = 20, /**< Fast interrupt channel 7 */
	RTE_TRAP_FIRQ_8 = 21, /**< Fast interrupt channel 8 */
	RTE_TRAP_FIRQ_9 = 22, /**< Fast interrupt channel 9 */
	RTE_TRAP_FIRQ_10 = 23, /**< Fast interrupt channel 10 */
	RTE_TRAP_FIRQ_11 = 24, /**< Fast interrupt channel 11 */
	RTE_TRAP_FIRQ_12 = 25, /**< Fast interrupt channel 12 */
	RTE_TRAP_FIRQ_13 = 26, /**< Fast interrupt channel 13 */
	RTE_TRAP_FIRQ_14 = 27, /**< Fast interrupt channel 14 */
	RTE_TRAP_FIRQ_15 = 28 /**< Fast interrupt channel 15 */
	};

Or you can use the more convenient device-specific aliases like the one you are using:

neorv32/sw/lib/include/neorv32.h

Line 113 in b3418fd

#define UART1_RX_RTE_ID RTE_TRAP_FIRQ_4 /**< RTE entry code (#NEORV32_RTE_TRAP_enum) */

In my init_core function:

Your interrupt-enabled code (neorv32_cpu_csr_set(CSR_MIE, ...);) also looks fine.

Is this the correct way ?

That looks good!

Maybe there is a problem with the device-specific interrupt configuration?

For example, if you want to trigger the UART0 interrupt when the RX FIFO becomes not-empty:

// enable IRQ if RX FIFO not empty
neorv32_uart0_setup(BAUD_RATE, 1 << UART_CTRL_IRQ_RX_NEMPTY);

[stdefeber]

Thanks for your quick response.

Brackets, brackets, brackets :(

I changed it to:

neorv32_cpu_csr_set((CSR_MIE, 1 << GPTMR_FIRQ_ENABLE) | (1 << UART0_RX_FIRQ_ENABLE)); 

Which resulted in correct behaviour.

[stnolting]

Brackets, brackets, brackets :(

🙈 😄

Which resulted in correct behaviour.

That's great to hear!