aviralgarg05 opened a new pull request, #3661: URL: https://github.com/apache/nuttx-apps/pull/3661
*Note: Please adhere to [Contributing Guidelines](https://github.com/apache/nuttx/blob/master/CONTRIBUTING.md).* ## Summary Allow CGOL, Match4, and Snake to participate in the supervised loadable-app flow. CGOL gains configurable framebuffer/input paths, framebuffer validation, reserved-strip placement, clean setup failure handling, and cooperative `SIGTERM`. Match4 and Snake become tristate so they can be selected as loadable modules without removing their builtin option. This PR is stacked on the nxstore series. For focused review, the supervised-games change is the final commit [`d7b08fd41`](https://github.com/aviralgarg05/nuttx-apps/commit/d7b08fd4126dcb672259180db2b228f50edd00f1). The earlier draft commits are dependencies already under review in #3642, #3643, and #3660; they will disappear as the stack merges. ## Impact - New feature: YES, optional module/supervisor operation for existing games. - User adaptation: NO for existing builtin configurations. - Build: Kconfig symbols become module-capable. - Hardware: NO driver change. - Documentation: YES; Kconfig help covers the selectable module behavior and device paths. - Security: NO new security boundary or privilege change. - Compatibility: Existing builtin use remains available. - Other: This draft depends on the nxstore and calculator drafts. ## Testing - Build host: macOS 26.5, arm64, GCC 14.2.0. - Target: Xtensa ESP32-S3, Waveshare ESP32-S3-Touch-LCD-7. - `git diff --check` - NuttX `tools/checkpatch.sh` - ESP32-S3 module build - CGOL rendered with 28.9 percent observed stack use and exited on `SIGTERM` - Match4 and Snake reported unavailable LED devices cleanly instead of corrupting state Testing logs before change: ```text CGOL, Match4, and Snake cannot participate in the supervised module flow. ``` Testing logs after change: ```text cgol rendered and exited cooperatively on SIGTERM cgol stack usage: 28.9% match4/snake: unavailable LED devices reported without corrupting state ``` ## PR verification Self-Check - [x] This PR contains one focused module/supervisor change. - [x] I have updated all required description fields above. - [x] I have reviewed and signed the commit. - [x] Local static and hardware checks pass. - [x] This PR adheres to the current contribution and coding guidelines. - [x] My PR is still work in progress. - [ ] My PR is ready for review and can be safely merged. -- 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]
