[alsa-devel] [PATCH] Gallant SC-6000 driver (4th version)

Takashi Iwai tiwai at suse.de
Fri Sep 7 13:11:21 CEST 2007


At Thu, 06 Sep 2007 23:16:39 +0200,
Rene Herman wrote:
> 
> On 09/06/2007 11:12 PM, Krzysztof Helt wrote:
> 
> > On Thu, 06 Sep 2007 12:12:42 +0200
> > Rene Herman <rene.herman at gmail.com> wrote:
> > 
> >> On 09/06/2007 08:49 AM, Krzysztof Helt wrote:
> >>
> >>> Actually, the parport_pc driver is loaded by the udev by default, so the
> >>> IRQ7 cannot be requested (I tested - IRQ7 request failed even if I had 
> >>> not used the printer).
> >> Not everybody uses udev. But more importantly, only if the hardware in 
> >> question is discoverable. [...]
> >>
> >> And _then_ there's the bit about the ISA probing being a huge gaping race by 
> >> design. There's really no defending that massive junk. All ISA autoprobing 
> >> is bad engineering and should be ripped out and shot, period. It's broken 
> >> both by design and implementation. The solution to making things nicer is 
> >> not autoprobing but not using legacy ISA -- and basically noone is other 
> >> than for entertainment purposes.
> >>
> > 
> > I am quite sure we won't agree on this one. My reason (to keep autodection)
> > is sheer laziness of giving two numbers. Even giving them does not make
> > user free of problems how to avoid irqs and dmas with installed devices 
> > but not visible in the /proc/interrupts file).
> 
> No, but then at least you aren't hiding the problem by promising the user to 
> figure it all out automatically. And as said, /etc/modprobe.conf works...
> 
> > I would like to ask Takashi to decide on this issue. I will done as he says.
> 
> Oh, I've not much doubt he'll not mind it terribly. Still many drivers in 
> sound/isa that do probing. Doesn't change that it's all counter-productive, 
> broken junk though!

The codes used for this auto-probing is relatively small and easy, so
as Rene expected, I don't mind about it :)

The criterion is only whether it's _really_ useful and works without
hitch.  We keep the old code as is just because it was worked, and we
shouldn't break it.  The concept is not perfect, but if it practically
works, I have no objection.

> > Will this driver be included? I found a bug in probing - if something is wrong
> > after the SB region is requested it won't be released. So the next version is
> > needed to be done anyway (or a small patch later).
> 
> Still testing with snd-galaxy here and haven't really looked at integrating 
> them yet. Will do, but I believe I saw some differences stand out already.

I'm for merging it -- at least, if it's confirmed to work on the
latest kernel (and the bug is fixed).  Let's put all them in a pot
before 1.0.15 final. We'll cook better after that.


thanks,

Takashi


More information about the Alsa-devel mailing list