|Subject:||Re: [PATCH] audio: Add sndio backend|
|Date:||Sat, 13 Nov 2021 15:40:39 -0500|
|User-agent:||Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:95.0) Gecko/20100101 Thunderbird/95.0|
On 11/8/2021 8:03 AM, Christian Schoenebeck wrote:
Ah, I was not aware of that. I usually include it but it was an oversight this time.On Sonntag, 7. November 2021 06:19:26 CET Brad Smith wrote:audio: Add sndio backend Add a sndio backend.Hi Brad!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.). The C code is from Alexandre Ratchov <email@example.com> and the rest of the bits are from me.A Signed-off-by: line is mandatory for all QEMU patches: https://wiki.qemu.org/Contribute/SubmitAPatch
Also, it should be clear from the patches who did what exactly, either by splitting the patches up and assigning the respective authors accordingly, or by making the person with the most relevant work the patch author and describing in the commit log additional authors and what they have added/ changed, along with their Signed-off-by: line: Signed-off-by: Alexandre Ratchov <firstname.lastname@example.org> [Brad Smith: - Added foo - Some other change] Signed-off-by: Brad Smith <email@example.com>
I think I'll go with this.
http://git.kernel.org/cgit/linux/kernel/git/torvalds/linux.git/tree/ Documentation/SubmittingPatches? id=f6f94e2ab1b33f0082ac22d71f66385a60d8157f#n297 Please CC those involved authors.
I did not add anything here as I figured it implies a certain level of obligation. His time available varies quite a bit (especially at the current time) and I wasn't sure if it's--- audio/audio.c | 1 + audio/audio_template.h | 2 + audio/meson.build | 1 + audio/sndioaudio.c | 555 +++++++++++++++++++++++++++++++++++++++++ meson.build | 7 + meson_options.txt | 4 +- qapi/audio.json | 25 +- qemu-options.hx | 8 + tests/vm/freebsd | 3 + 9 files changed, 604 insertions(+), 2 deletions(-)An additional subsection for this backend should be added to MAINTAINERS.
appropriate listing him.
create mode 100644 audio/sndioaudio.c diff --git a/audio/audio.c b/audio/audio.c index 54a153c0ef..bad1ceb69e 100644 --- a/audio/audio.c +++ b/audio/audio.c @@ -2005,6 +2005,7 @@ void audio_create_pdos(Audiodev *dev) CASE(OSS, oss, Oss); CASE(PA, pa, Pa); CASE(SDL, sdl, Sdl); + CASE(SNDIO, sndio, ); CASE(SPICE, spice, ); CASE(WAV, wav, ); diff --git a/audio/audio_template.h b/audio/audio_template.h index c6714946aa..ecc5a0bc6d 100644 --- a/audio/audio_template.h +++ b/audio/audio_template.h @@ -337,6 +337,8 @@ AudiodevPerDirectionOptions *glue(audio_get_pdo_, TYPE)(Audiodev *dev) return qapi_AudiodevPaPerDirectionOptions_base(dev->u.pa.TYPE); case AUDIODEV_DRIVER_SDL: return qapi_AudiodevSdlPerDirectionOptions_base(dev->u.sdl.TYPE); + case AUDIODEV_DRIVER_SNDIO: + return dev->u.sndio.TYPE; case AUDIODEV_DRIVER_SPICE: return dev->u.spice.TYPE; case AUDIODEV_DRIVER_WAV: diff --git a/audio/meson.build b/audio/meson.build index 462533bb8c..e24c86e7e6 100644 --- a/audio/meson.build +++ b/audio/meson.build @@ -17,6 +17,7 @@ foreach m : [ ['pa', pulse, files('paaudio.c')], ['sdl', sdl, files('sdlaudio.c')], ['jack', jack, files('jackaudio.c')], + ['sndio', sndio, files('sndioaudio.c')], ['spice', spice, files('spiceaudio.c')] ] if m.found() diff --git a/audio/sndioaudio.c b/audio/sndioaudio.c new file mode 100644 index 0000000000..204af07781 --- /dev/null +++ b/audio/sndioaudio.c @@ -0,0 +1,555 @@ +/* + * Copyright (c) 2019 Alexandre Ratchov <firstname.lastname@example.org> + *It is quite common for new source files in QEMU to have an authors list section in the header here like: * Autors: * Alexandre Ratchov <email@example.com>
I was looking through the tree and all of the examples I came across were using this with a Copyright for a company as opposed to an individual. What would be the
That way scripts/get_maintainer.pl can suggest those people as well in case they are not explicitly listed in MAINTAINERS. Does not seem to be mandatory for QEMU though. Just saying.
|[Prev in Thread]||Current Thread||[Next in Thread]|