Hi Alexandre,

sndio is the native API used by OpenBSD, although it has been ported to
other *BSD's and Linux (packages for Ubuntu, Debian, Void, Arch, etc.).

Signed-off-by: Brad Smith <b...@comstyle.com>
Signed-off-by: Alexandre Ratchov <a...@caoua.org>
---

Thank you for the reviews and all the comments. Here's a second diff
with all the suggested changes:

- Replace ISC license by SPDX-License-Identifier header
- Fix units (milli- vs micro-) in comment about SNDIO_LATENCY_US
- Drop outdated comment about the "size" argument of sndio_get_buffer_out()
- Fix AUDIO_FORMAT_U32 handling (missing "break" statement)
- Set {read,write] methods to audio_generic_{read,write} (fixes craches)
- Check if backend is enabled in sndio_poll_event()
- Use https://sndio.org in description
- Mark options as available after 7.0 release (instead of 6.2)
- Describe sndio-specific options (dev, latency) in qemu-options.hx
- Add myself as reviewer to MAINTAINERS
- Style fixes: no space after function names, use 4-space indent
- Don't use "return foo()" if foo() returns void
- Include backend to audio_drivers_priority[]

Tested on OpenBSD, works as expected!

  MAINTAINERS            |   5 +
  audio/audio.c          |   1 +
  audio/audio_template.h |   2 +
  audio/meson.build      |   1 +
  audio/sndioaudio.c     | 555 +++++++++++++++++++++++++++++++++++++++++
  meson.build            |   9 +-
  meson_options.txt      |   4 +-
  qapi/audio.json        |  25 +-
  qemu-options.hx        |  16 ++
  tests/vm/freebsd       |   3 +
  10 files changed, 618 insertions(+), 3 deletions(-)
  create mode 100644 audio/sndioaudio.c

Tested on Linux.

Reviewed-by: Volker Rümelin <vr_q...@t-online.de>
Tested-by: Volker Rümelin <vr_q...@t-online.de>


Reply via email to