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

Libiio v1 support ad9361 #507

Merged
merged 4 commits into from
Dec 6, 2024
Merged

Conversation

dNechita
Copy link
Contributor

@dNechita dNechita commented Dec 5, 2024

PR Description

This re-enabled the 'fmcomms2' plugin and updates it to use libiio v1.x API.
Additionally, it adds some helpers for iio attributes reading and writing. The purpose of the helpers is to reduce duplicated code and improve code readability.

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 followed the coding standards and guidelines
  • I have conducted a self-review of my own code changes
  • I have commented new code, particularly complex or unclear areas
  • I have checked in CI output that no new warnings/errors got introduced
  • I have updated documentation accordingly (GitHub Pages, READMEs, etc)

These helpers are useful when only one action is done on an attribute.
For usecases where several read/write operations are done on an
attribute it doesn't make sense to use this hepers.

Signed-off-by: Dan <dan.nechita@analog.com>
Signed-off-by: Dan <dan.nechita@analog.com>
Signed-off-by: Dan <dan.nechita@analog.com>
Signed-off-by: Dan <dan.nechita@analog.com>
@dNechita dNechita marked this pull request as ready for review December 5, 2024 13:02
@dNechita dNechita merged commit bf5e9f3 into libiio-v1-support Dec 6, 2024
2 of 4 checks passed
@dNechita dNechita deleted the libiio-v1-support-ad9361 branch December 6, 2024 09:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants