gustavonihei opened a new pull request, #8954:
URL: https://github.com/apache/nuttx/pull/8954

   ## Summary
   
   This PR intends to simplify the error handling of CPU interrupt allocation 
for Espressif's chis supported within `risc-v/espressif` tree.
   
   Instead of propagating an `-ENOMEM` error value, `esp_setup_irq` now adopts 
a fail-fast approach and simply panics. CPU interrupt allocation errors are 
unrecoverable, so the reliability of the execution becomes compromised.
   
   ## Impact
   
   Serial driver no longer performs the error checking, resulting is less code.
   
   ## Testing
   
   Verification in internal CI that all available defconfig within 
`boards/risc-v/espressif` boot properly.
   
   Also forced the error by hardcoding `esp_cpuint_alloc` return value as 
`-ENOMEM`:
   
   ```
   esp_setup_irq: Unable to allocate CPU interrupt for source=1
   _assert: Current Version: NuttX  10.4.0 12fda1da1c-dirty Apr  3 2023 
15:29:42 risc-v
   _assert: Assertion failed panic: at file: chip/esp_irq.c:454 task: Idle Task 
0x42003f5a
   ```
   
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscr...@nuttx.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to