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.

Sign up for GitHub

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

Jump to bottom

iio: adc: ad7944: always use 32-bit DMA word for offload #2634

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

iio: adc: ad7944: always use 32-bit DMA word for offload #2634

wants to merge 1 commit into from

Conversation

dlech
Copy link
Collaborator

@dlech dlech commented Oct 25, 2024 •
edited
Loading

PR Description

The HDL team is trying to standardize on using always 32-bit DMA words for SPI offload. Add a new channel_offloads channel configuration to reflect this. Also, since we are adding a new channel configuration, we can move the sampling frequency attribute implementation to use IIO_CHAN_INFO_SAMP_FREQ.

PR Type

  • Bug fix (a change that fixes an issue)
  • New feature (a change that adds new functionality)
  • Breaking change (a change that affects other repos or cause CIs to fail)

PR Checklist

  • I have conducted a self-review of my own code changes
  • I have tested the changes on the relevant hardware
  • I have updated the documentation outside this repo accordingly (if there is the case)

Tested with AD7985.

@dlech
iio: adc: ad7944: always use 32-bit DMA word for offload
74a8c60 
The HDL team is trying to standardize on using always 32-bit DMA words
for SPI offload. Add a new channel_offloads channel configuration to
reflect this. Also, since we are adding a new channel configuration,
we can move the sampling frequency attribute implementation to use
IIO_CHAN_INFO_SAMP_FREQ.

Signed-off-by: David Lechner <dlechner@baylibre.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@nunojsa nunojsa Awaiting requested review from nunojsa nunojsa is a code owner

@mhennerich mhennerich Awaiting requested review from mhennerich mhennerich is a code owner

@dbogdan dbogdan Awaiting requested review from dbogdan dbogdan is a code owner

@btogorean btogorean Awaiting requested review from btogorean btogorean is a code owner

@chegbeli chegbeli Awaiting requested review from chegbeli chegbeli is a code owner

@buha buha Awaiting requested review from buha buha is a code owner

@amiclaus amiclaus Awaiting requested review from amiclaus amiclaus is a code owner

@rbolboac rbolboac Awaiting requested review from rbolboac rbolboac is a code owner

@danmois danmois Awaiting requested review from danmois danmois is a code owner

@CiprianRegus CiprianRegus Awaiting requested review from CiprianRegus CiprianRegus is a code owner

@machschmitt machschmitt Awaiting requested review from machschmitt machschmitt is a code owner

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@dlech