[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [RFC PATCH] audio: Make the AudiodevDriver enum conditional
From: |
Daniel P . Berrangé |
Subject: |
Re: [RFC PATCH] audio: Make the AudiodevDriver enum conditional |
Date: |
Thu, 8 Jul 2021 16:17:05 +0100 |
User-agent: |
Mutt/2.0.7 (2021-05-04) |
On Thu, Jul 08, 2021 at 04:56:29PM +0200, Markus Armbruster wrote:
> Thomas Huth <thuth@redhat.com> writes:
>
> > On 25/06/2021 10.30, Daniel P. Berrangé wrote:
> >> On Thu, Jun 24, 2021 at 08:37:16PM +0200, Thomas Huth wrote:
> >>> This way, the upper layers like libvirt could have the possibility
> >>> to use QAPI to find out which audio drivers have been enabled during
> >>> compile-time of QEMU.
> >>>
> >>> Signed-off-by: Thomas Huth <thuth@redhat.com>
> >>> ---
> >>> Note: Marked as RFC since it's quite a lot of ifdef'ing here...
> >>> not sure whether I really like it...
> >> Same as my patch here:
> >> https://lists.gnu.org/archive/html/qemu-devel/2021-03/msg00654.html
> >
> > Oh, nice, I guess it means that it's not such a bad idea to introduce
> > all those #ifdefs here :-)
> >
> > Anyway, looks like my patch was incomplete anyway (I just hacked it
> > together after my corresponding DisplayType patch - see
> > https://lists.gnu.org/archive/html/qemu-devel/2021-06/msg06529.html),
> > since it lacks the wiring via a qapi command, so please disregard this
> > RFC PATCH. Are you going to respin your series from March, Daniel?
>
> I reviewed Daniel's series back then, and pointed out a few minor
> issues, mostly in commit messages. I hope that wasn't the reason for it
> getting stuck :)
No, I'm mostly just side-tracked on other work I'm afraid.
Regards,
Daniel
--
|: https://berrange.com -o- https://www.flickr.com/photos/dberrange :|
|: https://libvirt.org -o- https://fstop138.berrange.com :|
|: https://entangle-photo.org -o- https://www.instagram.com/dberrange :|