[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Openvortex-dev] Re: [ALSA - driver 0001138]: errors when installing
From: |
Igor Kovalenko |
Subject: |
Re: [Openvortex-dev] Re: [ALSA - driver 0001138]: errors when installing au8820 modules |
Date: |
Thu, 09 Jun 2005 14:17:05 +0400 |
User-agent: |
Mail/News Client 1.0+ (X11/20050324) |
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.
>>
>>
>
> So these changes are not neccesary(critical) for
> au88x0_64_bits_compatible.patch (AMD64 and X86_64 ) , just normal ALSA
> cleanup.
>
Actually without snd_card_set_dev() you will get oops shortly because device
pointer won't be available to driver.
DMA mask cleanup is to follow general guidelines about consistent mapping.
--
Kind regards,
Igor V. Kovalenko
- Re: [Openvortex-dev] Re: Re: [ALSA - driver 0001138]: errors when installing au8820 modules, (continued)
- Re: [Openvortex-dev] Re: Re: [ALSA - driver 0001138]: errors when installing au8820 modules, Raymond, 2005/06/08
- Re: [Alsa-devel] Re: [Openvortex-dev] Re: Re: [ALSA - driver 0001138]: errors when installing au8820 modules, Takashi Iwai, 2005/06/08
- Re: [Openvortex-dev] Re: Re: [ALSA - driver 0001138]: errors when installing au8820 modules, Igor Kovalenko, 2005/06/08
- Re: [Openvortex-dev] Re: Re: [ALSA - driver 0001138]: errors when installing au8820 modules, Raymond, 2005/06/09
- Re: [Openvortex-dev] Re: Re: [ALSA - driver 0001138]: errors when installing au8820 modules, Raymond, 2005/06/28
- [Openvortex-dev] Re: [ALSA - driver 0001138]: errors when installing au8820 modules, Raymond, 2005/06/08
- [Openvortex-dev] Re: [Alsa-devel] Re: [ALSA - driver 0001138]: errors when installing au8820 modules, Alien, 2005/06/08
- [Openvortex-dev] Re: [ALSA - driver 0001138]: errors when installing au8820 modules, Raymond, 2005/06/09
- [Openvortex-dev] Re: [Alsa-devel] Re: [ALSA - driver 0001138]: errors when installing au8820 modules, Takashi Iwai, 2005/06/08
- [Openvortex-dev] Re: [ALSA - driver 0001138]: errors when installing au8820 modules, Raymond, 2005/06/09
- Re: [Openvortex-dev] Re: [ALSA - driver 0001138]: errors when installing au8820 modules,
Igor Kovalenko <=
- Message not available
- Message not available
- [Openvortex-dev] Re: [ALSA - driver 0001138]: errors when installing au8820 modules, Raymond, 2005/06/10