[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Openvortex-dev] Re: [Alsa-devel] Re: [ALSA - driver 0001138]: errors wh
From: |
Takashi Iwai |
Subject: |
[Openvortex-dev] Re: [Alsa-devel] Re: [ALSA - driver 0001138]: errors when installing au8820 modules |
Date: |
Fri, 29 Jul 2005 17:09:27 +0200 |
User-agent: |
Wanderlust/2.12.0 (Your Wildest Dreams) SEMI/1.14.6 (Maruoka) FLIM/1.14.7 (Sanjō) APEL/10.6 MULE XEmacs/21.5 (beta18) (chestnut) (+CVS-20041021) (i386-suse-linux) |
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