Skip to content

fmcomms5: port streaming to libiio v1 multi-buffer API#141

Merged
dNechita merged 1 commit intostaging/libiio1-supportfrom
libiio-v1-support
Apr 14, 2026
Merged

fmcomms5: port streaming to libiio v1 multi-buffer API#141
dNechita merged 1 commit intostaging/libiio1-supportfrom
libiio-v1-support

Conversation

@999Marius
Copy link
Copy Markdown
Collaborator

PR Description

Port fmcomms5 streaming to the libiio 1.0 multi-buffer API.

Changes

  • Replace iio_device_create_buffer() with iio_device_get_buffer() in ad9361_fmcomms5_phase_sync.c and test/fmcomms5_sync_test.c
  • Remove iio_buffer_destroy()
  • Move channels mask from iio_device_create_buffer() to iio_buffer_create_stream(), for the new API design
  • Fix null check on buffer: iio_err()-> !rxbuf since iio_device_get_buffer() returns NULL, not a pointer-encoded error

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 commented new code, particularly complex or unclear areas
  • I have checked that I did not introduce new warnings or errors (CI output)
  • I have checked that components that use libiio did not get broken
  • I have updated the documentation accordingly (GitHub Pages, READMEs, etc)

iio_device_create_buffer() and iio_buffer_destroy() are gone in libiio v1.
The buffer is got with iio_device_get_buffer().
iio_buffer_create_stream() now takes the channels mask directly.

Signed-off-by: Marius Lucacel <marius.lucacel@analog.com>
Copy link
Copy Markdown

@dNechita dNechita left a comment

Choose a reason for hiding this comment

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

Looks good!

@dNechita dNechita merged commit 6b69b5e into staging/libiio1-support Apr 14, 2026
1 of 3 checks passed
@dNechita dNechita deleted the libiio-v1-support branch April 14, 2026 10:06
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