Re: Re: [ALSA - driver 0001138]: errors when installing au8820 modules

[Takashi Iwai <tiwai@suse.de>]

At Fri, 29 Jul 2005 22:43:18 +0800,
Raymond wrote:
> 
> Takashi Iwai wrote:
> > At Wed, 08 Jun 2005 17:06:39 +0800,
> > Raymond wrote:
> > 
> > 
> >>Is there any reason to make this change ?
> >>
> >>alsa-kernel/pci/au88x0/au88x0.c
> >>@@ -144,15 +144,18 @@
> >>         // check PCI availability (DMA).
> >>         if ((err = pci_enable_device(pci)) < 0)
> >>                 return err;
> >>-       if (!pci_dma_supported(pci, VORTEX_DMA_MASK)) {
> >>+        if (pci_set_dma_mask(pci, VORTEX_DMA_MASK) < 0 ||
> >>+               pci_set_consistent_dma_mask(pci, VORTEX_DMA_MASK) < 0) {
> >>                 printk(KERN_ERR "error to set DMA mask\n");
> >>+               pci_disable_device(pci);
> >>                 return -ENXIO;
> >>         }
> >>-       pci_set_dma_mask(pci, VORTEX_DMA_MASK);
> > 
> > 
> > This is clean up.
> > 
> > 
> >>         chip = kcalloc(1, sizeof(*chip), GFP_KERNEL);
> >>-       if (chip == NULL)
> >>+       if (chip == NULL) {
> >>+               pci_disable_device(pci);
> >>                 return -ENOMEM;
> >>+       }
> > 
> > 
> > This fixes the PCI state in the error path.
> > 
> > 
> >>         chip->card = card;
> >>
> >>@@ -202,6 +205,8 @@
> >>                 goto alloc_out;
> >>         }
> >>
> >>+       snd_card_set_dev(card, &pci->dev);
> >>+
> >>         *rchip = chip;
> > 
> > 
> > And this sets up the missing sysfs stuff.
> > 
> > 
> 
> It seem that these two patches for Kernel 2.6 were lost when au88x0 
> migrate from alsa-driver to alsa-kernel.

You're right.  It seems dropped at that time accidentally.

Could you care to post a patch to fix them?


thanks,

Takashi


-------------------------------------------------------
SF.Net email is Sponsored by the Better Software Conference & EXPO September
19-22, 2005 * San Francisco, CA * Development Lifecycle Practices
Agile & Plan-Driven Development * Managing Projects & Teams * Testing & QA
Security * Process Improvement & Measurement * http://www.sqe.com/bsce5sf