[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] Re: [PATCH 45/46] Remove AUDIO_DRIVERS define
From: |
Juan Quintela |
Subject: |
[Qemu-devel] Re: [PATCH 45/46] Remove AUDIO_DRIVERS define |
Date: |
Sun, 26 Jul 2009 15:41:49 +0200 |
User-agent: |
Gnus/5.11 (Gnus v5.11) Emacs/22.3 (gnu/linux) |
Anthony Liguori <address@hidden> wrote:
> malc wrote:
>> On Fri, 24 Jul 2009, Juan Quintela wrote:
>>
>>
>>> Signed-off-by: Juan Quintela <address@hidden>
>>> ---
>>> audio/audio.c | 25 ++++++++++++++++++++++++-
>>> configure | 3 ---
>>> 2 files changed, 24 insertions(+), 4 deletions(-)
>>>
>>
>> That's how it used to be before, i see no reason of going back to this
>> style.
>>
>
> Right. The way to get rid of AUDIO_DRIVERS is to convert drvtab[] to
> be a list with an appropriate audio driver registration function.
>
> Each .c file should have an audio_init() method that then registers
> the audio_driver structure. Individual audio drivers can be
> added/removed by simply building or not building the file.
>
> Just like how the block infrastructure works.
>
> Regards,
>
> Anthony Liguori
Not enough. See malc hint (g show 0c58ac1c7)
The order in audio_drivers is important, that means you can have a
different default sound driver.
After some thinking, my current plan is exporting audio_drv_list
to the Makefile, and then generating something similar to AUDIO_DRIVERS.
Notice that the "ordering" problems not only happens with sound drivers,
we have the same problems with graphics console, which one is the
default one? It depends on what you have configured, and changing the
default order is far from trivial.
Later, Juan.
- [Qemu-devel] [PATCH 43/46] rename NEEDS_LIBSUNMATH to CONFIG_NEEDS_LIBSUNMATH, (continued)
- [Qemu-devel] [PATCH 43/46] rename NEEDS_LIBSUNMATH to CONFIG_NEEDS_LIBSUNMATH, Juan Quintela, 2009/07/24
- [Qemu-devel] [PATCH 44/46] rename USE_NPTL to CONFIG_USE_NPTL, Juan Quintela, 2009/07/24
- [Qemu-devel] [PATCH 46/46] Generate config-host.h from config-host.mak, Juan Quintela, 2009/07/24
- [Qemu-devel] [PATCH 45/46] Remove AUDIO_DRIVERS define, Juan Quintela, 2009/07/24
- Re: [Qemu-devel] [PATCH 45/46] Remove AUDIO_DRIVERS define, Anthony Liguori, 2009/07/26
- [Qemu-devel] Re: [PATCH 45/46] Remove AUDIO_DRIVERS define,
Juan Quintela <=
- Re: [Qemu-devel] Re: [PATCH 45/46] Remove AUDIO_DRIVERS define, Avi Kivity, 2009/07/26
- [Qemu-devel] Re: [PATCH 45/46] Remove AUDIO_DRIVERS define, Jan Kiszka, 2009/07/26
- [Qemu-devel] Re: [PATCH 45/46] Remove AUDIO_DRIVERS define, Avi Kivity, 2009/07/26
- [Qemu-devel] Re: [PATCH 45/46] Remove AUDIO_DRIVERS define, malc, 2009/07/26
- [Qemu-devel] Re: [PATCH 45/46] Remove AUDIO_DRIVERS define, Jan Kiszka, 2009/07/26
- [Qemu-devel] Re: [PATCH 45/46] Remove AUDIO_DRIVERS define, malc, 2009/07/26
- Re: [Qemu-devel] Re: [PATCH 45/46] Remove AUDIO_DRIVERS define, Anthony Liguori, 2009/07/26
- [Qemu-devel] Re: [PATCH 45/46] Remove AUDIO_DRIVERS define, Anthony Liguori, 2009/07/26
[Qemu-devel] [PATCH 21/46] rename HAVE_FNMATCH_H to CONFIG_FNMATCH, Juan Quintela, 2009/07/24
[Qemu-devel] [PATCH 02/46] Remove SP_CFLAGS and SP_LDFLAGS, Juan Quintela, 2009/07/24