All of lore.kernel.org
 help / color / mirror / Atom feed
From: Rene Herman <rene.herman@gmail.com>
To: Krzysztof Helt <krzysztof.h1@gmail.com>
Cc: ALSA devel <alsa-devel@alsa-project.org>,
	Ram Kromberg <ramkromberg@mail.com>
Subject: Re: [PATCH] New Aztech Sound Galaxy driver.
Date: Tue, 11 Sep 2007 21:21:16 +0200	[thread overview]
Message-ID: <46E6EAAC.1070804@gmail.com> (raw)
In-Reply-To: <20070911210704.e738bd61.krzysztof.h1@gmail.com>

On 09/11/2007 09:07 PM, Krzysztof Helt wrote:

>> 5. I38-MMSN852 = AZT2316R + CS4231A-KL, with EEPROM
>>
> 
> I got this one. It has also AZT3320 chip.

Yep -- hardware midi, hooked up to the onboard MPU-401. Sounds really thin, 
but it's okay to play with.

> 1. There is a problem with freeing resources in the probe function. If the probe
> ends with error, removal of the driver does not release requested regions

Yes, dumb cut & paste error from when it was split off into its own 
function; please change the four "return err;" exits in (that version of) 
snd_galaxy_probe() with "goto error;"

> 2. After a while I found the free irq (it took quite long with this 
> manual settings). The driver loaded, the /proc/asound/* files showed 
> GALAXY something but the alsamixer ended with message that there was no
> mixer device.

It took quite long for you to look at /proc/interrupts and see which ones of 
four possible IRQs was unclaimed (the only thing autoprobing does)? Strange.

I don't see the mixer problem here, with the same card. Might it be a 
testing fluke caused by the unloading bug above?

Rene.

  reply	other threads:[~2007-09-11 19:21 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-09-07 23:22 [PATCH] New Aztech Sound Galaxy driver Rene Herman
2007-09-08 13:20 ` Krzysztof Helt
2007-09-11 19:07 ` Krzysztof Helt
2007-09-11 19:21   ` Rene Herman [this message]
2007-09-12 18:31     ` Krzysztof Helt
2007-09-13 17:56       ` Rene Herman
  -- strict thread matches above, loose matches on Subject: below --
2007-09-20 12:20 Rene Herman
2007-09-20 12:51 ` Takashi Iwai
2007-09-22  6:06   ` Rene Herman
2007-09-24 23:59     ` Takashi Iwai
2007-09-23 15:43 ` Krzysztof Helt
2007-09-24 19:19   ` Rene Herman

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=46E6EAAC.1070804@gmail.com \
    --to=rene.herman@gmail.com \
    --cc=alsa-devel@alsa-project.org \
    --cc=krzysztof.h1@gmail.com \
    --cc=ramkromberg@mail.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.