qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
From: Peter Maydell <peter.maydell@linaro.org>
To: "Kővágó, Zoltán" <dirty.ice.hu@gmail.com>
Cc: Gerd Hoffmann <kraxel@redhat.com>,
	QEMU Developers <qemu-devel@nongnu.org>,
	Maxim Levitsky <mlevitsk@redhat.com>
Subject: Re: [Qemu-devel] [PATCH 0/3] Audio: misc fixes for "Audio 20190821 patches", part two
Date: Thu, 17 Oct 2019 13:07:35 +0100	[thread overview]
Message-ID: <CAFEAcA_yHtDR=bjVR3brqsTeYoae7_ixADOS2ve+qQOyhoQo-w@mail.gmail.com> (raw)
In-Reply-To: <cover.1568157545.git.DirtY.iCE.hu@gmail.com>

Hi -- not sure what's happened with this patchset, but
can we at least get the first patch (which fixes some
coverity errors) in, even if the other two need more
consideration?

(cc'ing Gerd as audio maintainer.)

thanks
-- PMM


On Wed, 11 Sep 2019 at 00:27, Kővágó, Zoltán <dirty.ice.hu@gmail.com> wrote:
>
> Hi,
>
> This series contains some random fixes for the "Audio 20190821 patches":
> a coverity bugfix and pulseaudio connection/stream names fix.
>
> Since there wasn't a clear consensus about naming the pa streams, I've
> split it into two commits: the first time just uses the audiodev id
> unconditionally, while the last commit adds a new option to qapi to
> override it.  This way we can easily drop the last commit if it turns
> out to be unnecessary.
>
> Regards,
> Zoltan
>
>
> Kővágó, Zoltán (3):
>   audio: fix parameter dereference before NULL check
>   audio: paaudio: fix connection and stream name
>   audio: paaudio: ability to specify stream name
>
>  audio/audio_template.h | 7 +++++--
>  audio/paaudio.c        | 9 ++++++---
>  qapi/audio.json        | 6 ++++++
>  3 files changed, 17 insertions(+), 5 deletions(-)


      parent reply	other threads:[~2019-10-17 12:09 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2019-09-10 23:26 [Qemu-devel] [PATCH 0/3] Audio: misc fixes for "Audio 20190821 patches", part two Kővágó, Zoltán
2019-09-10 23:26 ` [Qemu-devel] [PATCH 1/3] audio: fix parameter dereference before NULL check Kővágó, Zoltán
2019-09-10 23:26 ` [Qemu-devel] [PATCH 2/3] audio: paaudio: fix connection and stream name Kővágó, Zoltán
2019-09-10 23:26 ` [Qemu-devel] [PATCH 3/3] audio: paaudio: ability to specify " Kővágó, Zoltán
2019-09-23 13:17   ` Markus Armbruster
2019-10-17 12:07 ` Peter Maydell [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to='CAFEAcA_yHtDR=bjVR3brqsTeYoae7_ixADOS2ve+qQOyhoQo-w@mail.gmail.com' \
    --to=peter.maydell@linaro.org \
    --cc=dirty.ice.hu@gmail.com \
    --cc=kraxel@redhat.com \
    --cc=mlevitsk@redhat.com \
    --cc=qemu-devel@nongnu.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).