Message-ID: <4204845E.10606@free.fr>
Date: Sat, 05 Feb 2005 09:31:26 +0100
From: matthieu castet <castet.matthieu@free.fr>
User-Agent: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.7.5) Gecko/20050105 Debian/1.7.5-1
MIME-Version: 1.0
To: Andrew Morton <akpm@osdl.org>
Cc: Adam Belay <ambx1@neo.rr.com>, linux-kernel@vger.kernel.org,
       vojtech@suse.cz
Subject: Re: [patch] ns558 bug
References: <4203D476.4040706@free.fr>	<20050205004311.GA7998@neo.rr.com> <20050204190614.6cfd68ce.akpm@osdl.org>
In-Reply-To: <20050204190614.6cfd68ce.akpm@osdl.org>
Content-Type: text/plain; charset=us-ascii; format=flowed
Content-Transfer-Encoding: 7bit
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 1142
Lines: 35

Hi,

Andrew Morton wrote:
> ambx1@neo.rr.com (Adam Belay) wrote:
> 
>>On Fri, Feb 04, 2005 at 09:00:54PM +0100, matthieu castet wrote:
>> > Hi,
>> > 
>> > this patch is based on http://bugzilla.kernel.org/show_bug.cgi?id=2962 
>> > patch from adam belay.
>> > 
>> > It solve a oops when pnp_register_driver(&ns558_pnp_driver) failed.
>> > 
>> > Please apply this patch.
>> > 
>> > Matthieu
>>
>> I remember writing a version of this patch a while ago.  The current behavior
>> is broken because it shouldn't be considered a failure if the driver doesn't
>> find any devices.
Why ?
If isa detection failed, the modules could not find any new isa devices.
If the pnp detection find no device, the module is load.
If it is the kernel modules behaviour, lot's of modules are broken...

If you want that, why not simply alway "return 0" in the __init like it 
is done in the bugzilla patch ?


Matthieu
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/