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@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