[v2] audio: es1370: add exit function

Commit Message

Li Qiang Dec. 14, 2016, 3:03 a.m. UTC

From: Li Qiang <[e-mail]>

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 <[e-mail]>
---

Changes since the v1:
close the streams

 hw/audio/es1370.c | 19 +++++++++++++++++++
 1 file changed, 19 insertions(+)

Patch

diff --git a/hw/audio/es1370.c b/hw/audio/es1370.c
index 8449b5f..8fc79fe 100644
--- a/hw/audio/es1370.c
+++ b/hw/audio/es1370.c
@@ -1041,6 +1041,24 @@  static void es1370_realize(PCIDevice *dev, Error **errp)
     es1370_reset (s);
 }
 
+static void es1370_exit(PCIDevice *dev)
+{
+    ES1370State *s = ES1370(dev);
+    int i;
+
+    if (s->adc_voice) {
+        AUD_close_in (&s->card, s->adc_voice);
+    }
+
+    for (i = 0; i < 2; ++i) {
+        if (s->dac_voice[i]) {
+            AUD_close_out (&s->card, s->dac_voice[i]);
+        }
+    }
+
+    AUD_remove_card(&s->card);
+}
+
 static int es1370_init (PCIBus *bus)
 {
     pci_create_simple (bus, -1, TYPE_ES1370);
@@ -1053,6 +1071,7 @@  static void es1370_class_init (ObjectClass *klass, void *data)
     PCIDeviceClass *k = PCI_DEVICE_CLASS (klass);
 
     k->realize = es1370_realize;
+    k->exit = es1370_exit;
     k->vendor_id = PCI_VENDOR_ID_ENSONIQ;
     k->device_id = PCI_DEVICE_ID_ENSONIQ_ES1370;
     k->class_id = PCI_CLASS_MULTIMEDIA_AUDIO;