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

[Backport v3.4-branch] drivers: lpadc: fix ADC command chaining #60287

Closed
wants to merge 1 commit into from

Conversation

zephyrbot
Copy link
Collaborator

Backport f4356f3 from #60111.

When reading multiple ADC channel in parallel, an ADC command chain will be
build. This is similar to a linked list, as every command references the
next command.

Before this patch every ADC command after the first, would always reference
this initial command. So that during execution only two commands (the last
and first) would be executed which resulted in readout of only two analog
values. As Zephyr expected more to come in, the `read_adc` function would
block endlessly.

The patch fixes the behaviour and allows a correct chain to build up.

Signed-off-by: Elisabeth Friedrich <auxsys@flowerpot.me>
(cherry picked from commit f4356f3)
@zephyrbot zephyrbot requested a review from anangl as a code owner July 12, 2023 07:26
@zephyrbot zephyrbot added Backport Backport PR and backport failure issues platform: NXP NXP area: ADC Analog-to-Digital Converter (ADC) labels Jul 12, 2023
@danieldegrasse
Copy link
Collaborator

duplicate of #60278, which I created because the original PR does not have an associated issue

@nashif nashif deleted the backport-60111-to-v3.4-branch branch August 15, 2023 17:09
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) Backport Backport PR and backport failure issues platform: NXP NXP
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

5 participants