-
Notifications
You must be signed in to change notification settings - Fork 1.4k
samples: zephyr: nrfx: Add test for nRF7120 #26420
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
This PR adds nRF7120 development kit support to the nrfx GPPI integration test sample, extending the Zephyr sample to verify compatibility with Nordic development kits.
- Adds nrf7120dk/nrf7120/cpuapp platform configuration
- Configures GPIO and GPIOTE peripherals for the nRF7120 platform
- Sets up test harness to validate GPPI functionality
Reviewed changes
Copilot reviewed 7 out of 7 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| samples/zephyr/boards/nordic/nrfx/sample.yaml | Defines test configuration for nRF7120 platform with console harness validation |
| samples/zephyr/boards/nordic/nrfx/prj.conf | Configures base project settings including GPIO, logging, and NRFX_GPPI |
| samples/zephyr/boards/nordic/nrfx/boards/nrf7120dk_nrf7120_cpuapp.overlay | Device tree overlay mapping GPIO pins and GPIOTE instance for nRF7120 |
| samples/zephyr/boards/nordic/nrfx/boards/nrf7120dk_nrf7120_cpuapp.conf | Board-specific configuration enabling GPIOTE20 driver |
| samples/zephyr/boards/nordic/nrfx/README.txt | Documentation describing the sample's purpose and source location |
| samples/zephyr/boards/nordic/nrfx/Kconfig | Sources the upstream Zephyr sample's Kconfig |
| samples/zephyr/boards/nordic/nrfx/CMakeLists.txt | Build configuration sourcing main.c from Zephyr sample |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
CI InformationTo view the history of this post, click the 'edited' button above Inputs:Sources:sdk-nrf: PR head: 2269b7edbf8fb6627b2a01b823d7d37f6e2c0673 more detailssdk-nrf:
Github labels
List of changed files detected by CI (9)Outputs:ToolchainVersion: f911d4f4e7 Test Spec & Results: ✅ Success; ❌ Failure; 🟠 Queued; 🟡 Progress; ◻️ Skipped;
|
samples/zephyr/boards/nordic/nrfx/boards/nrf7120dk_nrf7120_cpuapp.overlay
Outdated
Show resolved
Hide resolved
|
You can find the documentation preview for this PR here. |
06730e6 to
bd0227b
Compare
bd0227b to
dc6af33
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
Copilot reviewed 8 out of 8 changed files in this pull request and generated no new comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Add nrf7120dk cpuapp support to nrfx gppi integration test. Signed-off-by: Dhanoo Surasarang <[email protected]>
dc6af33 to
2269b7e
Compare
Add nrf7120dk cpuapp support to nrfx gppi integration test.