-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
tests: drivers: gpio: Add loopback test for eGPIO SDP #17173
Conversation
CI InformationTo view the history of this post, clich the 'edited' button above Inputs:Sources:sdk-nrf: PR head: c6a414bed016430dc932db8cd576f2a326689cb0 more detailssdk-nrf:
Github labels
List of changed files detected by CI (12)
Outputs:ToolchainVersion: 3dd8985b56 Test Spec & Results: ✅ Success; ❌ Failure; 🟠 Queued; 🟡 Progress; ◻️ Skipped;
|
b939a46
to
730ea56
Compare
You can find the documentation preview for this PR at this link. It will be updated about 10 minutes after the documentation build succeeds. Note: This comment is automatically posted by the Documentation Publishing GitHub Action. |
This is a copy of |
This file was the basis for my commit. However, eGPIO limitations (currently no input reading) forced changes that make this test incompatible with the original upstream and had to be added as a separate - eGPIO-only test case. |
730ea56
to
e9f0e3c
Compare
ebfddd5
to
e1295fc
Compare
e1295fc
to
6059b02
Compare
@PerMac @katgiadla can you take a look? |
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.
Tags are needed.
Please also add path for this new test into test-spec: https://github.com/nrfconnect/sdk-nrf/blob/main/.github/test-spec.yml#L583 |
tests/drivers/gpio/egpio_basic_api/dts/bindings/test-egpio.yaml
Outdated
Show resolved
Hide resolved
e070b94
to
037d4de
Compare
037d4de
to
99e1f98
Compare
99e1f98
to
98d1c3c
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.
One nit note
tests/drivers/gpio/egpio_basic_api/dts/bindings/test-egpio.yaml
Outdated
Show resolved
Hide resolved
Added a loopback test for the software-defined eGPIO peripheral. The test verifies proper operation of the eGPIO output by using a different processor hardware GPIO input pin. Signed-off-by: Jakub Zymelka <jakub.zymelka@nordicsemi.no>
98d1c3c
to
c6a414b
Compare
@nrfconnect/ncs-test-leads @nrfconnect/ncs-co-verification Ping |
Icbmsg test case will have to be added after this or #17321 is merged |
Added a loopback test for the software-defined eGPIO peripheral.
The test verifies proper operation of the eGPIO output by using
a different processor hardware GPIO input pin.