Re: [RFC PATCH] audio: Make the AudiodevDriver enum conditional

[Markus Armbruster <armbru@redhat.com>]

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 :)