aviralgarg05 opened a new pull request, #3662: URL: https://github.com/apache/nuttx-apps/pull/3662
*Note: Please adhere to [Contributing Guidelines](https://github.com/apache/nuttx/blob/master/CONTRIBUTING.md).* ## Summary Add touchscreen swipe input and loadable-module support to BrickMatch. The framebuffer board is centered below the reserved supervisor strip, touch drags are converted to discrete directions, setup failures are handled, and the game exits cooperatively on `SIGTERM`. Existing GPIO, gesture-sensor, and LED-matrix paths remain selectable. This PR is stacked on the supervised-games PR. For focused review, the BrickMatch-specific change is the final commit [`5e09f0b0d`](https://github.com/aviralgarg05/nuttx-apps/commit/5e09f0b0dee79de2555b9f2af5b2502c3eed1cee). The earlier draft commits are dependencies already under review through #3661; they will disappear as the stack merges. ## Impact - New feature: YES, optional touchscreen input and module operation. - User adaptation: NO for existing input backends. - Build: Adds Kconfig options for touchscreen device, threshold, and offset. - Hardware: NO driver change. - Documentation: YES; Kconfig help documents the input and layout options. - Security: NO new security boundary or privilege change. - Compatibility: Existing backends remain available. - Other: This draft depends on the supervised-games draft. ## 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 RGB565 module build and launch - Touchscreen gesture path exercised - Nonblocking no-sample reads yield instead of busy-spinning; coordinate deltas use 32-bit arithmetic - Process stack usage observed at 4.5 percent - Supervisor `SIGTERM` removed the process cleanly Testing logs before change: ```text BrickMatch has no touchscreen input or supervised module path. ``` Testing logs after change: ```text touchscreen gestures produced discrete game directions brick stack usage: 4.5% supervisor close: process exited after SIGTERM ``` ## PR verification Self-Check - [x] This PR contains one focused BrickMatch input/module 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]
