Skip to content
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: adc_accuracy #76088

Merged

Conversation

nordic-babu
Copy link
Contributor

@nordic-babu nordic-babu commented Jul 19, 2024

Add overlays and enable test for nrf52840dk/nrf54h20dk/nrf54l15dk.

@zephyrbot zephyrbot added the area: ADC Analog-to-Digital Converter (ADC) label Jul 19, 2024
@zephyrbot zephyrbot requested review from anangl and decsny July 19, 2024 08:03
@nordic-babu nordic-babu force-pushed the extend-adc-ref branch 2 times, most recently from f5f00d8 to ab32d06 Compare July 19, 2024 10:03
decsny
decsny previously approved these changes Jul 19, 2024
nordic-babu added a commit to nordic-babu/sdk-zephyr that referenced this pull request Jul 23, 2024
Add overlay and enable test for nrf52840dk.

Upstream PR: zephyrproject-rtos/zephyr#76088

Signed-off-by: Bartlomiej Buczek <[email protected]>
nordic-babu added a commit to nordic-babu/sdk-zephyr that referenced this pull request Jul 23, 2024
Add overlay and enable test for nrf54h20dk.

Upstream PR: zephyrproject-rtos/zephyr#76088

Signed-off-by: Bartlomiej Buczek <[email protected]>
nordic-babu added a commit to nordic-babu/sdk-zephyr that referenced this pull request Jul 23, 2024
Add overlay and enable test for nrf54l15pdk.

Upstream PR: zephyrproject-rtos/zephyr#76088

Signed-off-by: Bartlomiej Buczek <[email protected]>
nordic-babu added a commit to nordic-babu/sdk-zephyr that referenced this pull request Jul 23, 2024
…r nrf54.

It will increase test coverage for adc driver, no test uses
ADC_GAIN_2_3 setting yet.

Upstream PR: zephyrproject-rtos/zephyr#76088

Signed-off-by: Bartlomiej Buczek <[email protected]>
nordicjm pushed a commit to nrfconnect/sdk-zephyr that referenced this pull request Jul 23, 2024
Add overlay and enable test for nrf52840dk.

Upstream PR: zephyrproject-rtos/zephyr#76088

Signed-off-by: Bartlomiej Buczek <[email protected]>
nordicjm pushed a commit to nrfconnect/sdk-zephyr that referenced this pull request Jul 23, 2024
Add overlay and enable test for nrf54h20dk.

Upstream PR: zephyrproject-rtos/zephyr#76088

Signed-off-by: Bartlomiej Buczek <[email protected]>
nordicjm pushed a commit to nrfconnect/sdk-zephyr that referenced this pull request Jul 23, 2024
Add overlay and enable test for nrf54l15pdk.

Upstream PR: zephyrproject-rtos/zephyr#76088

Signed-off-by: Bartlomiej Buczek <[email protected]>
nordicjm pushed a commit to nrfconnect/sdk-zephyr that referenced this pull request Jul 23, 2024
…r nrf54.

It will increase test coverage for adc driver, no test uses
ADC_GAIN_2_3 setting yet.

Upstream PR: zephyrproject-rtos/zephyr#76088

Signed-off-by: Bartlomiej Buczek <[email protected]>
@nordic-babu nordic-babu dismissed stale reviews from bjarki-andreasen and decsny via a2ced2f July 23, 2024 10:47
@nordic-babu
Copy link
Contributor Author

had to parametrize expected accuracy, need other value for some nrf boards to pass on our internal setups

rlubos pushed a commit to rlubos/sdk-zephyr that referenced this pull request Oct 22, 2024
Add overlay and enable test for nrf54h20dk.

Upstream PR: zephyrproject-rtos/zephyr#76088

Signed-off-by: Bartlomiej Buczek <[email protected]>
(cherry picked from commit 9cfc59e)
rlubos pushed a commit to rlubos/sdk-zephyr that referenced this pull request Oct 22, 2024
Add overlay and enable test for nrf54l15pdk.

Upstream PR: zephyrproject-rtos/zephyr#76088

Signed-off-by: Bartlomiej Buczek <[email protected]>
(cherry picked from commit 5ca3d32)
rlubos pushed a commit to rlubos/sdk-zephyr that referenced this pull request Oct 22, 2024
…r nrf54.

It will increase test coverage for adc driver, no test uses
ADC_GAIN_2_3 setting yet.

Upstream PR: zephyrproject-rtos/zephyr#76088

Signed-off-by: Bartlomiej Buczek <[email protected]>
(cherry picked from commit 8b6e30f)
rlubos pushed a commit to rlubos/sdk-zephyr that referenced this pull request Oct 22, 2024
…uracy

On nrf boards expected accuracy from ref voltage is 64 instead of 32.

Upstream PR: zephyrproject-rtos/zephyr#76088

Signed-off-by: Bartlomiej Buczek <[email protected]>
(cherry picked from commit 3107c7a)
rlubos pushed a commit to rlubos/sdk-zephyr that referenced this pull request Oct 22, 2024
sampling

Calibrate adc before taking measurement.

Upstream PR: zephyrproject-rtos/zephyr#76088

Signed-off-by: Bartlomiej Buczek <[email protected]>
(cherry picked from commit 0e4d6d5)
@carlescufi
Copy link
Member

@nordic-babu please rebase

Add overlay and enable test for nrf52840dk.

Signed-off-by: Bartlomiej Buczek <[email protected]>
Add overlay and enable test for nrf54h20dk.

Signed-off-by: Bartlomiej Buczek <[email protected]>
@nordic-babu nordic-babu force-pushed the extend-adc-ref branch 2 times, most recently from 58e7e51 to 3eaa712 Compare October 22, 2024 12:22
@@ -21,5 +21,5 @@ tests:
- ek_ra8m1
- frdm_mcxc242
- nrf52840dk/nrf52840
- nrf54l15pdk/nrf54l15/cpuapp
- nrf54l15dk/nrf54l15/cpuapp
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Wrong commit, This should be corrected in tests: adc_accuracy: add overlay for nrf54l15dk..

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

change moved to correct commit

Add overlay and enable test for nrf54l15dk.

Signed-off-by: Bartlomiej Buczek <[email protected]>
It will increase test coverage for adc driver, no test uses
ADC_GAIN_2_3 setting yet.

Signed-off-by: Bartlomiej Buczek <[email protected]>
anangl
anangl previously approved these changes Oct 22, 2024
carlescufi
carlescufi previously approved these changes Oct 22, 2024
On nrf boards expected accuracy from ref voltage is 64 instead of 32.

Signed-off-by: Bartlomiej Buczek <[email protected]>
Calibrate adc before taking measurement.

Signed-off-by: Bartlomiej Buczek <[email protected]>
@nordic-babu
Copy link
Contributor Author

nordic-babu commented Oct 23, 2024

Aligned some boards overlays to fix CI

57300 pushed a commit to 57300/sdk-zephyr that referenced this pull request Oct 23, 2024
…r nrf54.

Upstream PR: zephyrproject-rtos/zephyr#76088

It will increase test coverage for adc driver, no test uses
ADC_GAIN_2_3 setting yet.

Signed-off-by: Bartlomiej Buczek <[email protected]>
(cherry picked from commit 71ae6ff6f10497e816fcf2da05ee0887dd5452df)
57300 pushed a commit to 57300/sdk-zephyr that referenced this pull request Oct 23, 2024
…r nrf54.

Upstream PR: zephyrproject-rtos/zephyr#76088

It will increase test coverage for adc driver, no test uses
ADC_GAIN_2_3 setting yet.

Signed-off-by: Bartlomiej Buczek <[email protected]>
(cherry picked from commit 71ae6ff6f10497e816fcf2da05ee0887dd5452df)
57300 pushed a commit to 57300/sdk-zephyr that referenced this pull request Oct 23, 2024
…r nrf54.

Upstream PR: zephyrproject-rtos/zephyr#76088

It will increase test coverage for adc driver, no test uses
ADC_GAIN_2_3 setting yet.

Signed-off-by: Bartlomiej Buczek <[email protected]>
(cherry picked from commit 71ae6ff6f10497e816fcf2da05ee0887dd5452df)
@henrikbrixandersen henrikbrixandersen merged commit dfbcea7 into zephyrproject-rtos:main Oct 23, 2024
17 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: ADC Analog-to-Digital Converter (ADC)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

10 participants