diff options
author | Martin Kletzander <mkletzan@redhat.com> | 2023-10-02 16:27:57 +0200 |
---|---|---|
committer | Paolo Bonzini <pbonzini@redhat.com> | 2023-10-03 10:29:40 +0200 |
commit | cb94ff5f80c537e43c04fa4f071f1df784255310 (patch) | |
tree | 18907d34d44c8e344ea2174c5bd80e6fa8dc5169 /hw/audio/ac97.c | |
parent | 9dcb64c96073e9898105cdbef6553cfef0fadcdb (diff) | |
download | qemu-cb94ff5f80c537e43c04fa4f071f1df784255310.zip qemu-cb94ff5f80c537e43c04fa4f071f1df784255310.tar.gz qemu-cb94ff5f80c537e43c04fa4f071f1df784255310.tar.bz2 |
audio: propagate Error * out of audio_init
Starting from audio_driver_init, propagate errors via Error ** so that
audio_init_audiodevs can simply pass &error_fatal, and AUD_register_card
can signal faiure.
Signed-off-by: Martin Kletzander <mkletzan@redhat.com>
[Reworked the audio/audio.c parts, while keeping Martin's hw/ changes. - Paolo]
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
Diffstat (limited to 'hw/audio/ac97.c')
-rw-r--r-- | hw/audio/ac97.c | 6 |
1 files changed, 5 insertions, 1 deletions
diff --git a/hw/audio/ac97.c b/hw/audio/ac97.c index c2a5ce0..6a7a2dc 100644 --- a/hw/audio/ac97.c +++ b/hw/audio/ac97.c @@ -1273,6 +1273,10 @@ static void ac97_realize(PCIDevice *dev, Error **errp) AC97LinkState *s = AC97(dev); uint8_t *c = s->dev.config; + if (!AUD_register_card ("ac97", &s->card, errp)) { + return; + } + /* TODO: no need to override */ c[PCI_COMMAND] = 0x00; /* pcicmd pci command rw, ro */ c[PCI_COMMAND + 1] = 0x00; @@ -1306,7 +1310,7 @@ static void ac97_realize(PCIDevice *dev, Error **errp) "ac97-nabm", 256); pci_register_bar(&s->dev, 0, PCI_BASE_ADDRESS_SPACE_IO, &s->io_nam); pci_register_bar(&s->dev, 1, PCI_BASE_ADDRESS_SPACE_IO, &s->io_nabm); - AUD_register_card("ac97", &s->card); + ac97_on_reset(DEVICE(s)); } |