[alsa-devel] [PATCH] Gallant SC-6000 driver (4th version)
rene.herman at gmail.com
Thu Sep 6 23:16:39 CEST 2007
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!
> 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.
More information about the Alsa-devel