639b49ef9a
audio/adlib: remove limitation of one adlib card
...
Signed-off-by: Hervé Poussineau <hpoussin@reactos.org >
Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org >
Message-id: 20170621043401.19842-3-hpoussin@reactos.org
Signed-off-by: Gerd Hoffmann <kraxel@redhat.com >
2017-07-17 11:09:02 +02:00
c57fbf50e7
audio/fmopl: modify timer callback to give opaque and channel parameters in two arguments
...
Signed-off-by: Hervé Poussineau <hpoussin@reactos.org >
Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org >
Message-id: 20170621043401.19842-2-hpoussin@reactos.org
Signed-off-by: Gerd Hoffmann <kraxel@redhat.com >
2017-07-17 11:09:02 +02:00
8a824e4d74
audio: Rename hw/audio/audio.h to hw/audio/soundhw.h
...
All the functions in hw/audio/audio.h are called "soundhw_*()"
and live in hw/audio/audiohw.c. Rename the header file for
consistency.
Signed-off-by: Eduardo Habkost <ehabkost@redhat.com >
Reviewed-by: David Gibson <david@gibson.dropbear.id.au >
Reviewed-by: Hervé Poussineau <hpoussin@reactos.org >
Message-id: 20170508205735.23444-4-ehabkost@redhat.com
Signed-off-by: Gerd Hoffmann <kraxel@redhat.com >
2017-05-19 10:48:54 +02:00
4c565674a2
audio: Rename audio_init() to soundhw_init()
...
To make it consistent with the remaining soundhw.c functions and
avoid confusion with the audio_init() function in audio/audio.c,
rename audio_init() to soundhw_init().
Signed-off-by: Eduardo Habkost <ehabkost@redhat.com >
Reviewed-by: David Gibson <david@gibson.dropbear.id.au >
Message-id: 20170508205735.23444-3-ehabkost@redhat.com
Signed-off-by: Gerd Hoffmann <kraxel@redhat.com >
2017-05-19 10:48:53 +02:00
ca89f72092
audio: Move arch_init audio code to hw/audio/soundhw.c
...
There's no reason to keep the soundhw table in arch_init.c. Move
that code to a new hw/audio/soundhw.c file.
While moving the code, trivial coding style issues were fixed.
Signed-off-by: Eduardo Habkost <ehabkost@redhat.com >
Reviewed-by: David Gibson <david@gibson.dropbear.id.au >
Reviewed-by: Thomas Huth <thuth@redhat.com >
Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org >
Message-id: 20170508205735.23444-2-ehabkost@redhat.com
Signed-off-by: Gerd Hoffmann <kraxel@redhat.com >
2017-05-19 10:48:53 +02:00
e90f2a8c3e
qdev: Replace cannot_instantiate_with_device_add_yet with !user_creatable
...
cannot_instantiate_with_device_add_yet was introduced by commit
efec3dd631
to replace no_user. It was
supposed to be a temporary measure.
When it was introduced, we had 54
cannot_instantiate_with_device_add_yet=true lines in the code.
Today (3 years later) this number has not shrunk: we now have
57 cannot_instantiate_with_device_add_yet=true lines. I think it
is safe to say it is not a temporary measure, and we won't see
the flag go away soon.
Instead of a long field name that misleads people to believe it
is temporary, replace it a shorter and less misleading field:
user_creatable.
Except for code comments, changes were generated using the
following Coccinelle patch:
@@
expression DC;
@@
(
-DC->cannot_instantiate_with_device_add_yet = false;
+DC->user_creatable = true;
|
-DC->cannot_instantiate_with_device_add_yet = true;
+DC->user_creatable = false;
)
@@
typedef ObjectClass;
expression dc;
identifier class, data;
@@
static void device_class_init(ObjectClass *class, void *data)
{
...
dc->hotpluggable = true;
+dc->user_creatable = true;
...
}
@@
@@
struct DeviceClass {
...
-bool cannot_instantiate_with_device_add_yet;
+bool user_creatable;
...
}
@@
expression DC;
@@
(
-!DC->cannot_instantiate_with_device_add_yet
+DC->user_creatable
|
-DC->cannot_instantiate_with_device_add_yet
+!DC->user_creatable
)
Cc: Alistair Francis <alistair.francis@xilinx.com >
Cc: Laszlo Ersek <lersek@redhat.com >
Cc: Marcel Apfelbaum <marcel@redhat.com >
Cc: Markus Armbruster <armbru@redhat.com >
Cc: Peter Maydell <peter.maydell@linaro.org >
Cc: Thomas Huth <thuth@redhat.com >
Acked-by: Alistair Francis <alistair.francis@xilinx.com >
Reviewed-by: Thomas Huth <thuth@redhat.com >
Reviewed-by: Marcel Apfelbaum <marcel@redhat.com >
Acked-by: Marcel Apfelbaum <marcel@redhat.com >
Signed-off-by: Eduardo Habkost <ehabkost@redhat.com >
Message-Id: <20170503203604.31462-2-ehabkost@redhat.com >
[ehabkost: kept "TODO remove once we're there" comment]
Reviewed-by: Markus Armbruster <armbru@redhat.com >
Signed-off-by: Eduardo Habkost <ehabkost@redhat.com >
2017-05-17 10:37:00 -03:00
9ea5ada76f
audio: Use ARRAY_SIZE from qemu/osdep.h
...
Signed-off-by: Juan Quintela <quintela@redhat.com >
Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org >
Message-id: 20170425223739.6703-27-quintela@redhat.com
Signed-off-by: Gerd Hoffmann <kraxel@redhat.com >
2017-05-04 09:16:05 +02:00
d52ccc0eca
audio: un-export OPLResetChip
...
Signed-off-by: Juan Quintela <quintela@redhat.com >
Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org >
Message-id: 20170425223739.6703-26-quintela@redhat.com
Signed-off-by: Gerd Hoffmann <kraxel@redhat.com >
2017-05-04 09:16:05 +02:00
3fab7b675a
audio: Remove unused typedefs
...
Signed-off-by: Juan Quintela <quintela@redhat.com >
Message-id: 20170425223739.6703-25-quintela@redhat.com
Signed-off-by: Gerd Hoffmann <kraxel@redhat.com >
2017-05-04 09:16:05 +02:00
b3b40917c7
audio: UpdateHandler is not used anymore
...
Signed-off-by: Juan Quintela <quintela@redhat.com >
Message-id: 20170425223739.6703-24-quintela@redhat.com
Signed-off-by: Gerd Hoffmann <kraxel@redhat.com >
2017-05-04 09:16:05 +02:00
e14e09c945
audio: IRQHandler is not used anymore
...
Signed-off-by: Juan Quintela <quintela@redhat.com >
Message-id: 20170425223739.6703-23-quintela@redhat.com
Signed-off-by: Gerd Hoffmann <kraxel@redhat.com >
2017-05-04 09:16:05 +02:00
ade339896b
audio: OPLSetUpdateHandler is not used anywhere
...
Signed-off-by: Juan Quintela <quintela@redhat.com >
Message-id: 20170425223739.6703-22-quintela@redhat.com
Signed-off-by: Gerd Hoffmann <kraxel@redhat.com >
2017-05-04 09:16:05 +02:00
17a1694a56
audio: OPLSetIRQHandler is not used anywhere
...
Signed-off-by: Juan Quintela <quintela@redhat.com >
Message-id: 20170425223739.6703-21-quintela@redhat.com
Signed-off-by: Gerd Hoffmann <kraxel@redhat.com >
2017-05-04 09:16:05 +02:00
135f5ae197
audio: GUSsample is int16_t
...
Signed-off-by: Juan Quintela <quintela@redhat.com >
Message-id: 20170425223739.6703-20-quintela@redhat.com
Signed-off-by: Gerd Hoffmann <kraxel@redhat.com >
2017-05-04 09:16:05 +02:00
4bf7792aae
audio: GUSword is uint16_t
...
Signed-off-by: Juan Quintela <quintela@redhat.com >
Message-id: 20170425223739.6703-19-quintela@redhat.com
Signed-off-by: Gerd Hoffmann <kraxel@redhat.com >
2017-05-04 09:16:05 +02:00
1c742f2b8e
audio: GUSword is uint16_t
...
Signed-off-by: Juan Quintela <quintela@redhat.com >
Message-id: 20170425223739.6703-18-quintela@redhat.com
Signed-off-by: Gerd Hoffmann <kraxel@redhat.com >
2017-05-04 09:16:04 +02:00
222e0356fa
audio: remove GUSchar
...
Signed-off-by: Juan Quintela <quintela@redhat.com >
Message-id: 20170425223739.6703-17-quintela@redhat.com
Signed-off-by: Gerd Hoffmann <kraxel@redhat.com >
2017-05-04 09:16:04 +02:00
0af81c56bf
audio: GUSbyte is uint8_t
...
Signed-off-by: Juan Quintela <quintela@redhat.com >
Message-id: 20170425223739.6703-16-quintela@redhat.com
Signed-off-by: Gerd Hoffmann <kraxel@redhat.com >
2017-05-04 09:16:04 +02:00
9887e22155
audio: Remove unused fields
...
These were used for the remove stuff.
Signed-off-by: Juan Quintela <quintela@redhat.com >
Message-id: 20170425223739.6703-15-quintela@redhat.com
Signed-off-by: Gerd Hoffmann <kraxel@redhat.com >
2017-05-04 09:16:04 +02:00
8f7e2c2cb7
audio: Remove type field
...
It was not used anymore as now there is only one type of devices.
Signed-off-by: Juan Quintela <quintela@redhat.com >
Message-id: 20170425223739.6703-14-quintela@redhat.com
Signed-off-by: Gerd Hoffmann <kraxel@redhat.com >
2017-05-04 09:16:04 +02:00
7852b53acc
audio: Remove Unused OPL_TYPE_*
...
Since we removed the previous unused devices, they are not used anymore.
Signed-off-by: Juan Quintela <quintela@redhat.com >
Message-id: 20170425223739.6703-13-quintela@redhat.com
Signed-off-by: Gerd Hoffmann <kraxel@redhat.com >
2017-05-04 09:16:04 +02:00
8ec734d027
audio: Unfold OPLSAMPLE
...
It was used only once, and now it was always int16_t.
Signed-off-by: Juan Quintela <quintela@redhat.com >
Message-id: 20170425223739.6703-12-quintela@redhat.com
Signed-off-by: Gerd Hoffmann <kraxel@redhat.com >
2017-05-04 09:16:04 +02:00
7f643fb53a
audio: Remove INT32
...
Signed-off-by: Juan Quintela <quintela@redhat.com >
Message-id: 20170425223739.6703-11-quintela@redhat.com
Signed-off-by: Gerd Hoffmann <kraxel@redhat.com >
2017-05-04 09:16:04 +02:00
7bf10b1de2
audio: remove INT16
...
Signed-off-by: Juan Quintela <quintela@redhat.com >
Message-id: 20170425223739.6703-10-quintela@redhat.com
Signed-off-by: Gerd Hoffmann <kraxel@redhat.com >
2017-05-04 09:16:04 +02:00
d2a4a01fa4
audio: Remove INT8
...
Signed-off-by: Juan Quintela <quintela@redhat.com >
Message-id: 20170425223739.6703-9-quintela@redhat.com
Signed-off-by: Gerd Hoffmann <kraxel@redhat.com >
2017-05-04 09:16:04 +02:00
3795f18095
audio: remove UINT32
...
Signed-off-by: Juan Quintela <quintela@redhat.com >
Message-id: 20170425223739.6703-8-quintela@redhat.com
Signed-off-by: Gerd Hoffmann <kraxel@redhat.com >
2017-05-04 09:16:03 +02:00
d8586afd8b
audio: remove UINT16
...
More modernitation.
Signed-off-by: Juan Quintela <quintela@redhat.com >
Message-id: 20170425223739.6703-7-quintela@redhat.com
Signed-off-by: Gerd Hoffmann <kraxel@redhat.com >
2017-05-04 09:16:03 +02:00
4a796e979e
audio: Remove UINT8
...
uint8_t has existed since ..... all this century?
Signed-off-by: Juan Quintela <quintela@redhat.com >
Message-id: 20170425223739.6703-6-quintela@redhat.com
Signed-off-by: Gerd Hoffmann <kraxel@redhat.com >
2017-05-04 09:16:03 +02:00
882ab9d615
audio: YM3812 was always defined
...
So, remove the ifdefs.
Signed-off-by: Juan Quintela <quintela@redhat.com >
Message-id: 20170425223739.6703-5-quintela@redhat.com
Signed-off-by: Gerd Hoffmann <kraxel@redhat.com >
2017-05-04 09:16:03 +02:00
2004429e9b
audio: Remove YM3526 support
...
It was never compiled in.
Signed-off-by: Juan Quintela <quintela@redhat.com >
Message-id: 20170425223739.6703-4-quintela@redhat.com
Signed-off-by: Gerd Hoffmann <kraxel@redhat.com >
2017-05-04 09:16:03 +02:00
b6c6919e21
audio: remove Y8950 configuration
...
Include file has never been on qemu and it has been undefined from the very beginning.
Signed-off-by: Juan Quintela <quintela@redhat.com >
Message-id: 20170425223739.6703-3-quintela@redhat.com
Signed-off-by: Gerd Hoffmann <kraxel@redhat.com >
2017-05-04 09:16:03 +02:00
68883a4078
adlib: Remove support for YMF262
...
Notice that the code was supposed to be in the file ymf262.h, that has
never been on qemu source tree.
Signed-off-by: Juan Quintela <quintela@redhat.com >
Message-id: 20170425223739.6703-2-quintela@redhat.com
Signed-off-by: Gerd Hoffmann <kraxel@redhat.com >
2017-05-04 09:16:03 +02:00
5eaa8e1e0f
hw/audio: convert exit callback in HDACodecDeviceClass to void
...
The exit callback always return 0, convert it to void
Signed-off-by: Zihan Yang <tgnyang@gmail.com >
Message-id: 1493211188-24086-5-git-send-email-tgnyang@gmail.com
Signed-off-by: Gerd Hoffmann <kraxel@redhat.com >
2017-05-04 08:29:01 +02:00
8ac5535145
hw/audio: replace exit with unrealize in hda_codec_device_class_init
...
The exit callback of DeviceClass will be removed in the future, so
convert to unrealize in the init functioin
Signed-off-by: Zihan Yang <tgnyang@gmail.com >
Message-id: 1493211188-24086-4-git-send-email-tgnyang@gmail.com
Signed-off-by: Gerd Hoffmann <kraxel@redhat.com >
2017-05-04 08:29:01 +02:00
11f547e58a
es1370: wire up reset via DeviceClass
...
Instead of using qemu_register_reset().
That way we get proper cleanup for free.
Signed-off-by: Gerd Hoffmann <kraxel@redhat.com >
Message-id: 1484054281-26139-1-git-send-email-kraxel@redhat.com
2017-01-11 09:19:03 +01:00
12351a91da
audio: ac97: add exit function
...
Currently the ac97 device emulation doesn't have a exit function,
hot unplug this device will leak some memory. Add a exit function to
avoid this.
Signed-off-by: Li Qiang <liqiang6-s@360.cn >
Reviewed-by: Marc-André Lureau <marcandre.lureau@redhat.com >
Message-id: 58520052.4825ed0a.27a71.6cae@mx.google.com
Signed-off-by: Gerd Hoffmann <kraxel@redhat.com >
2017-01-11 09:19:03 +01:00
069eb7b2b8
audio: es1370: add exit function
...
Currently the es1370 device emulation doesn't have a exit function,
hot unplug this device will leak some memory. Add a exit function to
avoid this.
Signed-off-by: Li Qiang <liqiang6-s@360.cn >
Reviewed-by: Marc-André Lureau <marcandre.lureau@redhat.com >
Message-id: 585200c9.a968ca0a.1ab80.4c98@mx.google.com
Signed-off-by: Gerd Hoffmann <kraxel@redhat.com >
2017-01-11 09:19:03 +01:00
8becab9523
hw/audio: QOM'ify pl041.c
...
split the old SysBus init function into an instance_init
and Device realize function
Signed-off-by: xiaoqiang zhao <zxq_yx_007@163.com >
Message-id: 20161231011720.3965-3-zxq_yx_007@163.com
Signed-off-by: Gerd Hoffmann <kraxel@redhat.com >
2017-01-11 09:19:03 +01:00
c025d0abce
hw/audio: QOM'ify marvell_88w8618.c
...
split the old SysBus init function into an instance_init
and Device realize function
Signed-off-by: xiaoqiang zhao <zxq_yx_007@163.com >
Message-id: 20161231011720.3965-2-zxq_yx_007@163.com
Signed-off-by: Gerd Hoffmann <kraxel@redhat.com >
2017-01-11 09:19:03 +01:00
d307c28ca9
i2c: Allow I2C devices to NAK start events
...
Add a return value to the event handler. Some I2C devices will
NAK if they have no data, so allow them to do this. This required
the following changes:
Go through all the event handlers and change them to return int
and return 0.
Modify i2c_start_transfer to terminate the transaction on a NAK.
Modify smbus handing to not assert if a NAK occurs on a second
operation, and terminate the transaction and return -1 instead.
Add some information on semantics to I2CSlaveClass.
Signed-off-by: Corey Minyard <cminyard@mvista.com >
Reviewed-by: Peter Maydell <peter.maydell@linaro.org >
Signed-off-by: Peter Maydell <peter.maydell@linaro.org >
2017-01-09 11:40:20 +00:00
04e27c6bb0
migration/pcspk: Add a property to state if pcspk is migrated
...
Allow us to turn migration of pcspk off for compatibility.
Signed-off-by: Dr. David Alan Gilbert <dgilbert@redhat.com >
Message-Id: <20161128133201.16104-2-dgilbert@redhat.com >
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com >
2016-11-28 16:45:12 +01:00
0c0fc2b5fd
audio: intel-hda: check stream entry count during transfer
...
Intel HDA emulator uses stream of buffers during DMA data
transfers. Each entry has buffer length and buffer pointer
position, which are used to derive bytes to 'copy'. If this
length and buffer pointer were to be same, 'copy' could be
set to zero(0), leading to an infinite loop. Add check to
avoid it.
Reported-by: Huawei PSIRT <psirt@huawei.com >
Signed-off-by: Prasad J Pandit <pjp@fedoraproject.org >
Reviewed-by: Stefan Hajnoczi <stefanha@redhat.com >
Message-id: 1476949224-6865-1-git-send-email-ppandit@redhat.com
Signed-off-by: Gerd Hoffmann <kraxel@redhat.com >
2016-10-26 14:51:44 +02:00
39c88f5697
pcspk: adding vmstate for save/restore
...
VMState added by this patch preserves correct
loading of the PC speaker device state.
Signed-off-by: Pavel Dovgalyuk <pavel.dovgaluk@ispras.ru >
Message-Id: <20160915090133.6440.65457.stgit@PASHA-ISP>
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com >
2016-09-15 15:10:06 +02:00
e305a16510
portio: keep references on portio
...
The isa_register_portio_list() function allocates ioports
data/state. Let's keep the reference to this data on some owner. This
isn't enough to fix leaks, but at least, ASAN stops complaining of
direct leaks. Further cleanup would require calling
portio_list_del/destroy().
Signed-off-by: Marc-André Lureau <marcandre.lureau@redhat.com >
Reviewed-by: Paolo Bonzini <pbonzini@redhat.com >
2016-09-08 18:05:21 +04:00
e723b87103
trace-events: fix first line comment in trace-events
...
Documentation is docs/tracing.txt instead of docs/trace-events.txt.
find . -name trace-events -exec \
sed -i "s?See docs/trace-events.txt for syntax documentation.?See docs/tracing.txt for syntax documentation.?" \
{} \;
Signed-off-by: Laurent Vivier <lvivier@redhat.com >
Message-id: 1470669081-17860-1-git-send-email-lvivier@redhat.com
Signed-off-by: Stefan Hajnoczi <stefanha@redhat.com >
2016-08-12 10:36:01 +01:00
175de52487
Clean up decorations and whitespace around header guards
...
Cleaned up with scripts/clean-header-guards.pl.
Signed-off-by: Markus Armbruster <armbru@redhat.com >
Reviewed-by: Richard Henderson <rth@twiddle.net >
2016-07-12 16:20:46 +02:00
2a6a4076e1
Clean up ill-advised or unusual header guards
...
Cleaned up with scripts/clean-header-guards.pl.
Signed-off-by: Markus Armbruster <armbru@redhat.com >
Reviewed-by: Richard Henderson <rth@twiddle.net >
2016-07-12 16:20:46 +02:00
1108b2f8a9
pci: Convert msi_init() to Error and fix callers to check it
...
msi_init() reports errors with error_report(), which is wrong
when it's used in realize().
Fix by converting it to Error.
Fix its callers to handle failure instead of ignoring it.
For those callers who don't handle the failure, it might happen:
when user want msi on, but he doesn't get what he want because of
msi_init fails silently.
cc: Gerd Hoffmann <kraxel@redhat.com >
cc: John Snow <jsnow@redhat.com >
cc: Dmitry Fleytman <dmitry@daynix.com >
cc: Jason Wang <jasowang@redhat.com >
cc: Michael S. Tsirkin <mst@redhat.com >
cc: Hannes Reinecke <hare@suse.de >
cc: Paolo Bonzini <pbonzini@redhat.com >
cc: Alex Williamson <alex.williamson@redhat.com >
cc: Markus Armbruster <armbru@redhat.com >
cc: Marcel Apfelbaum <marcel@redhat.com >
Reviewed-by: Markus Armbruster <armbru@redhat.com >
Signed-off-by: Cao jin <caoj.fnst@cn.fujitsu.com >
Reviewed-by: Michael S. Tsirkin <mst@redhat.com >
Signed-off-by: Michael S. Tsirkin <mst@redhat.com >
Reviewed-by: Hannes Reinecke <hare@suse.com >
2016-07-05 13:14:41 +03:00
c0f2abff73
intel-hda: change msi property type
...
>From uint32 to enum OnOffAuto.
cc: Gerd Hoffmann <kraxel@redhat.com >
cc: Michael S. Tsirkin <mst@redhat.com >
cc: Markus Armbruster <armbru@redhat.com >
cc: Marcel Apfelbaum <marcel@redhat.com >
Reviewed-by: Markus Armbruster <armbru@redhat.com >
Signed-off-by: Cao jin <caoj.fnst@cn.fujitsu.com >
Reviewed-by: Michael S. Tsirkin <mst@redhat.com >
Signed-off-by: Michael S. Tsirkin <mst@redhat.com >
2016-07-05 13:14:41 +03:00
8a0b4de048
pcspk: fix KVM
...
The link property that was added to the pcspk device has the wrong type:
it is only correct for TCG and for KVM's userspace or split irqchip
options. The default KVM option (fully in-kernel irqchip) breaks
because it uses a PIT whose type is a sibling of TYPE_I8254.
Fixes: 873b4d3f05
Tested-by: Peter Lieven <pl@kamp.de >
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com >
Message-id: 1467298657-6588-1-git-send-email-pbonzini@redhat.com
Reviewed-by: Peter Maydell <peter.maydell@linaro.org >
Signed-off-by: Peter Maydell <peter.maydell@linaro.org >
2016-06-30 19:00:02 +01:00