OSDN Git Service

ALSA: ens137x: Allocate resources with device-managed APIs
authorTakashi Iwai <tiwai@suse.de>
Thu, 15 Jul 2021 07:58:38 +0000 (09:58 +0200)
committerTakashi Iwai <tiwai@suse.de>
Mon, 19 Jul 2021 14:16:44 +0000 (16:16 +0200)
This patch converts the resource management in PCI esn137x drivers
with devres as a clean up.  Each manual resource management is
converted with the corresponding devres helper, the devres helper is
used for the DMA buffer page allocations, and the card object release
is managed now via card->private_free instead of a lowlevel
snd_device.

This should give no user-visible functional changes.

Link: https://lore.kernel.org/r/20210715075941.23332-17-tiwai@suse.de
Signed-off-by: Takashi Iwai <tiwai@suse.de>
sound/pci/ens1370.c

index 728b69d..2651f0c 100644 (file)
@@ -414,7 +414,7 @@ struct ensoniq {
        unsigned int spdif_stream;
 
 #ifdef CHIP1370
-       struct snd_dma_buffer dma_bug;
+       struct snd_dma_buffer *dma_bug;
 #endif
 
 #ifdef SUPPORT_JOYSTICK
@@ -1872,11 +1872,11 @@ static void snd_ensoniq_proc_init(struct ensoniq *ensoniq)
 
  */
 
-static int snd_ensoniq_free(struct ensoniq *ensoniq)
+static void snd_ensoniq_free(struct snd_card *card)
 {
+       struct ensoniq *ensoniq = card->private_data;
+
        snd_ensoniq_free_gameport(ensoniq);
-       if (ensoniq->irq < 0)
-               goto __hw_end;
 #ifdef CHIP1370
        outl(ES_1370_SERR_DISABLE, ES_REG(ensoniq, CONTROL));   /* switch everything off */
        outl(0, ES_REG(ensoniq, SERIAL));       /* clear serial interface */
@@ -1884,24 +1884,6 @@ static int snd_ensoniq_free(struct ensoniq *ensoniq)
        outl(0, ES_REG(ensoniq, CONTROL));      /* switch everything off */
        outl(0, ES_REG(ensoniq, SERIAL));       /* clear serial interface */
 #endif
-       pci_set_power_state(ensoniq->pci, PCI_D3hot);
-      __hw_end:
-#ifdef CHIP1370
-       if (ensoniq->dma_bug.area)
-               snd_dma_free_pages(&ensoniq->dma_bug);
-#endif
-       if (ensoniq->irq >= 0)
-               free_irq(ensoniq->irq, ensoniq);
-       pci_release_regions(ensoniq->pci);
-       pci_disable_device(ensoniq->pci);
-       kfree(ensoniq);
-       return 0;
-}
-
-static int snd_ensoniq_dev_free(struct snd_device *device)
-{
-       struct ensoniq *ensoniq = device->device_data;
-       return snd_ensoniq_free(ensoniq);
 }
 
 #ifdef CHIP1371
@@ -1935,7 +1917,7 @@ static void snd_ensoniq_chip_init(struct ensoniq *ensoniq)
        outl(ensoniq->ctrl, ES_REG(ensoniq, CONTROL));
        outl(ensoniq->sctrl, ES_REG(ensoniq, SERIAL));
        outl(ES_MEM_PAGEO(ES_PAGE_ADC), ES_REG(ensoniq, MEM_PAGE));
-       outl(ensoniq->dma_bug.addr, ES_REG(ensoniq, PHANTOM_FRAME));
+       outl(ensoniq->dma_bug->addr, ES_REG(ensoniq, PHANTOM_FRAME));
        outl(0, ES_REG(ensoniq, PHANTOM_COUNT));
 #else
        outl(ensoniq->ctrl, ES_REG(ensoniq, CONTROL));
@@ -2032,51 +2014,35 @@ static SIMPLE_DEV_PM_OPS(snd_ensoniq_pm, snd_ensoniq_suspend, snd_ensoniq_resume
 #endif /* CONFIG_PM_SLEEP */
 
 static int snd_ensoniq_create(struct snd_card *card,
-                             struct pci_dev *pci,
-                             struct ensoniq **rensoniq)
+                             struct pci_dev *pci)
 {
-       struct ensoniq *ensoniq;
+       struct ensoniq *ensoniq = card->private_data;
        int err;
-       static const struct snd_device_ops ops = {
-               .dev_free =     snd_ensoniq_dev_free,
-       };
 
-       *rensoniq = NULL;
-       err = pci_enable_device(pci);
+       err = pcim_enable_device(pci);
        if (err < 0)
                return err;
-       ensoniq = kzalloc(sizeof(*ensoniq), GFP_KERNEL);
-       if (ensoniq == NULL) {
-               pci_disable_device(pci);
-               return -ENOMEM;
-       }
        spin_lock_init(&ensoniq->reg_lock);
        mutex_init(&ensoniq->src_mutex);
        ensoniq->card = card;
        ensoniq->pci = pci;
        ensoniq->irq = -1;
        err = pci_request_regions(pci, "Ensoniq AudioPCI");
-       if (err < 0) {
-               kfree(ensoniq);
-               pci_disable_device(pci);
+       if (err < 0)
                return err;
-       }
        ensoniq->port = pci_resource_start(pci, 0);
-       if (request_irq(pci->irq, snd_audiopci_interrupt, IRQF_SHARED,
-                       KBUILD_MODNAME, ensoniq)) {
+       if (devm_request_irq(&pci->dev, pci->irq, snd_audiopci_interrupt,
+                            IRQF_SHARED, KBUILD_MODNAME, ensoniq)) {
                dev_err(card->dev, "unable to grab IRQ %d\n", pci->irq);
-               snd_ensoniq_free(ensoniq);
                return -EBUSY;
        }
        ensoniq->irq = pci->irq;
        card->sync_irq = ensoniq->irq;
 #ifdef CHIP1370
-       if (snd_dma_alloc_pages(SNDRV_DMA_TYPE_DEV, &pci->dev,
-                               16, &ensoniq->dma_bug) < 0) {
-               dev_err(card->dev, "unable to allocate space for phantom area - dma_bug\n");
-               snd_ensoniq_free(ensoniq);
-               return -EBUSY;
-       }
+       ensoniq->dma_bug =
+               snd_devm_alloc_pages(&pci->dev, SNDRV_DMA_TYPE_DEV, 16);
+       if (!ensoniq->dma_bug)
+               return -ENOMEM;
 #endif
        pci_set_master(pci);
        ensoniq->rev = pci->revision;
@@ -2099,17 +2065,10 @@ static int snd_ensoniq_create(struct snd_card *card,
                ensoniq->cssr |= ES_1371_ST_AC97_RST;
 #endif
 
+       card->private_free = snd_ensoniq_free;
        snd_ensoniq_chip_init(ensoniq);
 
-       err = snd_device_new(card, SNDRV_DEV_LOWLEVEL, ensoniq, &ops);
-       if (err < 0) {
-               snd_ensoniq_free(ensoniq);
-               return err;
-       }
-
        snd_ensoniq_proc_init(ensoniq);
-
-       *rensoniq = ensoniq;
        return 0;
 }
 
@@ -2360,47 +2319,35 @@ static int snd_audiopci_probe(struct pci_dev *pci,
                return -ENOENT;
        }
 
-       err = snd_card_new(&pci->dev, index[dev], id[dev], THIS_MODULE,
-                          0, &card);
+       err = snd_devm_card_new(&pci->dev, index[dev], id[dev], THIS_MODULE,
+                               sizeof(*ensoniq), &card);
        if (err < 0)
                return err;
+       ensoniq = card->private_data;
 
-       err = snd_ensoniq_create(card, pci, &ensoniq);
-       if (err < 0) {
-               snd_card_free(card);
+       err = snd_ensoniq_create(card, pci);
+       if (err < 0)
                return err;
-       }
-       card->private_data = ensoniq;
 
 #ifdef CHIP1370
        err = snd_ensoniq_1370_mixer(ensoniq);
-       if (err < 0) {
-               snd_card_free(card);
+       if (err < 0)
                return err;
-       }
 #endif
 #ifdef CHIP1371
        err = snd_ensoniq_1371_mixer(ensoniq, spdif[dev], lineio[dev]);
-       if (err < 0) {
-               snd_card_free(card);
+       if (err < 0)
                return err;
-       }
 #endif
        err = snd_ensoniq_pcm(ensoniq, 0);
-       if (err < 0) {
-               snd_card_free(card);
+       if (err < 0)
                return err;
-       }
        err = snd_ensoniq_pcm2(ensoniq, 1);
-       if (err < 0) {
-               snd_card_free(card);
+       if (err < 0)
                return err;
-       }
        err = snd_ensoniq_midi(ensoniq, 0);
-       if (err < 0) {
-               snd_card_free(card);
+       if (err < 0)
                return err;
-       }
 
        snd_ensoniq_create_gameport(ensoniq, dev);
 
@@ -2414,26 +2361,18 @@ static int snd_audiopci_probe(struct pci_dev *pci,
                ensoniq->irq);
 
        err = snd_card_register(card);
-       if (err < 0) {
-               snd_card_free(card);
+       if (err < 0)
                return err;
-       }
 
        pci_set_drvdata(pci, card);
        dev++;
        return 0;
 }
 
-static void snd_audiopci_remove(struct pci_dev *pci)
-{
-       snd_card_free(pci_get_drvdata(pci));
-}
-
 static struct pci_driver ens137x_driver = {
        .name = KBUILD_MODNAME,
        .id_table = snd_audiopci_ids,
        .probe = snd_audiopci_probe,
-       .remove = snd_audiopci_remove,
        .driver = {
                .pm = SND_ENSONIQ_PM_OPS,
        },