jerpelea opened a new pull request, #16451: URL: https://github.com/apache/nuttx/pull/16451
## Summary 0001 - this patch fixes LED constants for breadxavr board. The code was developed based on other AVR boards, LED constants were taken from board that apparently has multiple status LEDs and used on a board that only has one. The functionality that was not working properly is blinking LED on panic, which works correctly now 0002 - while digging around in the code trying to figure out how PANIC() works, I noticed a bug in up_saveusercontext function that is called from _assert. The function makes use of USER_SAVE macro but that macro is designed to be only a first half of the context switch. It is unsuitable to be used standalone like it was in up_saveusercontext, it pops return address from the stack and does not return, meaning that the function also did not return. The patch adds what is missing and what would otherwise be done by the second half of the context switch. Tested by compiling and verifying the disassembly - the function no longer falls through to the next function in the program memory, push/pop instructions are balanced and stack contents preserved 0003 - typo fix 0004 - this patch adds --print-memory-usage flag to the linker if GCC compiler is used. Memory usage information is then output after linking. Can't test other supported compilers but they should not be affected since the patch only takes effect if CONFIG_ARCH_TOOLCHAIN_GCC is set ## Impact AVR ## Testing CI -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
