Gedare Bloom started a new discussion on bsps/riscv/esp32/irq/irq_c3.c: https://gitlab.rtems.org/rtems/rtos/rtems/-/merge_requests/1160#note_147635 > + __asm__ volatile ("fence o, i" : : : "memory"); > + riscv_interrupt_enable(cookie); > + > + /* clear all mappings */ > + for ( uint8_t vec = 1; vec < RISCV_MAXIMUM_EXTERNAL_INTERRUPTS; vec++ ) { > + bsp_interrupt_vector_disable( vec ); > + } > +} > + > +bool bsp_interrupt_is_valid_vector(rtems_vector_number vector) > +{ > + if ( vector == 0 ) { > + return false; > + } > + > + return vector < (rtems_vector_number) BSP_INTERRUPT_VECTOR_COUNT; It would be clearer to keep using `RISCV_MAXIMUM_EXTERNAL_INTERRUPTS` here. And double-check if it should be `<=` again. -- View it on GitLab: https://gitlab.rtems.org/rtems/rtos/rtems/-/merge_requests/1160#note_147635 You're receiving this email because of your account on gitlab.rtems.org.
_______________________________________________ bugs mailing list [email protected] http://lists.rtems.org/mailman/listinfo/bugs
